Rosen Penev <rosenp@xxxxxxxxx> wrote: > This follows the pattern with other USB Wifi drivers. There is nothing > special being done in the _init and _exit functions here. Simplifies and > saves some lines of code. > > Signed-off-by: Rosen Penev <rosenp@xxxxxxxxx> Fails to compile: In file included from ./include/linux/device.h:32, from drivers/net/wireless/purelifi/plfxlc/usb.c:8: drivers/net/wireless/purelifi/plfxlc/usb.c:871:19: error: conflicting types for 'driver_init'; have 'int(void)' 871 | module_usb_driver(driver); | ^~~~~~ ./include/linux/device/driver.h:258:19: note: in definition of macro 'module_driver' 258 | static int __init __driver##_init(void) \ | ^~~~~~~~ drivers/net/wireless/purelifi/plfxlc/usb.c:871:1: note: in expansion of macro 'module_usb_driver' 871 | module_usb_driver(driver); | ^~~~~~~~~~~~~~~~~ ./include/linux/device/driver.h:241:6: note: previous declaration of 'driver_init' with type 'void(void)' 241 | void driver_init(void); | ^~~~~~~~~~~ make[7]: *** [scripts/Makefile.build:243: drivers/net/wireless/purelifi/plfxlc/usb.o] Error 1 make[6]: *** [scripts/Makefile.build:480: drivers/net/wireless/purelifi/plfxlc] Error 2 make[5]: *** [scripts/Makefile.build:480: drivers/net/wireless/purelifi] Error 2 make[4]: *** [scripts/Makefile.build:480: drivers/net/wireless] Error 2 make[3]: *** [scripts/Makefile.build:480: drivers/net] Error 2 make[2]: *** [scripts/Makefile.build:480: drivers] Error 2 make[1]: *** [/home/kvalo/projects/personal/wireless-drivers/src/wireless-next/Makefile:2032: .] Error 2 make: *** [Makefile:234: __sub-make] Error 2 Patch set to Changes Requested. -- https://patchwork.kernel.org/project/linux-wireless/patch/20230826200929.9756-2-rosenp@xxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches