[PATCH 0/6] gpio: batch #2: remove modular code from non-modular drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



For GPIO, I've divided up the the audit of modular usage in non-modular
drivers into three categories to ease review and limit the batch size.
Group #1 has been submitted and merged ; this group here is group #2.

The breakdown of the three groups is as follows:

1) just replacement of modular macros with their non-modular equivalents
   that CPP would have inserted anyway ; this means runtime equivalence
   and actually also binary equivalence.

2) as per #1 but also with the removal of unused/orphaned __exit functions
   that could never be called/exercised.  This also maintains runtime
   equivalence, but since the unused __exit function is gone, there is a
   reduction in the object file size and hence not binary equivalence, eg:
       before: -rw-rw-r-- 1 8828 drivers/gpio/gpio-rc5t583.o
       after:  -rw-rw-r-- 1 7396 drivers/gpio/gpio-rc5t583.o

3) as per #2 but also with the removal of a ".remove" function that is
   hooked into the driver struct.   This ".remove" function would of
   course not be called from the __exit function since that is never run.
   However in theory, someone could have triggered it via sysfs unbind,
   even though there isn't a sensible use case for doing so.  So to cover
   that possibility, we've also disabled sysfs unbind in these drivers.

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.

Interestingly enough, none of the drivers patched here were using the
module_init() binding ; they all used subsys_initcall() instead, which
I think reinforces point #4 above - i.e. nobody writes a new driver
from scratch.

Build tested for x86-64, arm and arm64 on today's linux-next to ensure
no silly typos crept in.

Paul.
---

Cc: Alexandre Courbot <gnurou@xxxxxxxxx>
Cc: Graeme Gregory <gg@xxxxxxxxxxxxxxx>
Cc: Gregory Bean <gbean@xxxxxxxxxxxxxx>
Cc: Hanumath Prasad <hanumath.prasad@xxxxxxxxxxxxxx>
Cc: Jorge Eduardo Candelaria <jedu@xxxxxxxxxxxxxxx>
Cc: Laxman Dewangan <ldewangan@xxxxxxxxxx>
Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
Cc: Rabin Vincent <rabin.vincent@xxxxxxxxxxxxxx>
Cc: linux-gpio@xxxxxxxxxxxxxxx

Paul Gortmaker (6):
  drivers/gpio: make gpio-rc5t583.c explicitly non-modular
  drivers/gpio: make gpio-tc3589x.c explicitly non-modular
  drivers/gpio: make gpio-sx150x.c explicitly non-modular
  drivers/gpio: make gpio-palmas.c explicitly non-modular
  drivers/gpio: make gpio-tps65910.c explicitly non-modular
  drivers/gpio: make gpio-tps6586x.c explicitly non-modular

 drivers/gpio/gpio-palmas.c   | 13 +------------
 drivers/gpio/gpio-rc5t583.c  | 12 ------------
 drivers/gpio/gpio-sx150x.c   | 15 ++++-----------
 drivers/gpio/gpio-tc3589x.c  | 11 -----------
 drivers/gpio/gpio-tps6586x.c | 13 +------------
 drivers/gpio/gpio-tps65910.c | 16 ++--------------
 6 files changed, 8 insertions(+), 72 deletions(-)

-- 
2.6.1

--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SPI]     [Linux Kernel]     [Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [Linux for Samsung SOC]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [Linux MIPS]     [Yosemite Campsites]

  Powered by Linux