On 31/10/2016 at 14:55:24 -0400, Paul Gortmaker wrote : > My ongoing audit looking for non-modular code that needlessly uses > modular macros (vs. built-in equivalents) and/or has dead code > relating to module unloading that can never be executed led to the > creation of these rtc related commits. > > For anyone new to the underlying goal of this cleanup, we are trying to > not use module support for code that can never be built as a module since: > > (1) it is easy to accidentally write unused module_exit and remove code > (2) it can be misleading when reading the source, thinking it can be > modular when the Makefile and/or Kconfig prohibit it > (3) it requires the include of the module.h header file which in turn > includes nearly everything else, thus adding to CPP overhead. > (4) it gets copied/replicated into other code and spreads like weeds. > > Build tested on current linux-next (sparc32) to ensure no silly typos > or implicit include issues that would break compilation crept in. > > --- > > Cc: Alessandro Zummo <a.zummo@xxxxxxxxxxxx> > Cc: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxxxxxxxxx> > Cc: "David S. Miller" <davem@xxxxxxxxxxxxx> > Cc: rtc-linux@xxxxxxxxxxxxxxxx > Cc: sparclinux@xxxxxxxxxxxxxxx > > Paul Gortmaker (3): > rtc: make rtc-lib explicitly non-modular > rtc: sparc: make starfire explicitly non-modular > rtc: sparc: make sun4v explicitly non-modular > > drivers/rtc/rtc-lib.c | 4 +--- > drivers/rtc/rtc-starfire.c | 10 ++++------ > drivers/rtc/rtc-sun4v.c | 10 ++++------ > 3 files changed, 9 insertions(+), 15 deletions(-) > Applied, thanks. -- Alexandre Belloni, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
