Hi Finn, On Sat, Jun 2, 2018 at 5:27 AM, Finn Thain <[email protected]> wrote: > The pmu_init() function has the __init qualifier, but the ops struct > that holds a pointer to it does not. This causes a build warning. > The driver works fine because the pointer is only dereferenced early. > > The function is so small that there's negligible benefit from using > the __init qualifier. Remove it to fix the warning, consistent with > the other ADB drivers.
Some other ADB subdriver .init() and .probe() functions aren't that small. But with the current scheme using adb_drivers_list[], they cannot be __init. Probably the long term fix is to change the ADB subsystem from the centralized approach of letting adb_init() call all subdrivers, to making the subdrivers platform drivers registering with the ADB core. > Tested-by: Stan Johnson <[email protected]> > Signed-off-by: Finn Thain <[email protected]> Anyway: Reviewed-by: Geert Uytterhoeven <[email protected]> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-m68k" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
