linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/6] memory: remove modular usage from non-modular drivers
@ 2016-06-17  0:37 Paul Gortmaker
  2016-06-17  0:37 ` [PATCH 1/6] memory: atmel-sdramc: make it explicitly non-modular Paul Gortmaker
                   ` (5 more replies)
  0 siblings, 6 replies; 14+ messages in thread
From: Paul Gortmaker @ 2016-06-17  0:37 UTC (permalink / raw)
  To: linux-kernel
  Cc: Paul Gortmaker, Alexandre Belloni, Alexandre Courbot,
	Boris Brezillon, Ezequiel Garcia, Jason Cooper,
	Jean-Jacques Hiblot, Krzysztof Kozlowski, Kukjin Kim,
	Nicolas Ferre, Pankaj Dubey, Roger Quadros, Stephen Warren,
	Thierry Reding, Thomas Petazzoni, Tony Lindgren, linux-omap,
	linux-samsung-soc, linux-tegra

It seems that drivers/memory/ doesn't have a single maintainer, so please
feel free to pick up just one or two patches from this as appropriate.

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.

Changes seen here cover the following categories:

  -just replacement of modular macros with their non-modular
   equivalents that CPP would have inserted anyway

  -the removal of including module.h ; replaced with init.h
   as required based on whether the file already had it.

  -the removal of any/all unused/orphaned __exit functions
   that would never be called/exercised.

  -the removal of any ".remove" functions that were hooked into
   the driver struct.   This ".remove" function would of
   course not be called from the __exit function since that was
   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.

There are no initcall level changes here; everything continues to
use the level of initcall it had before.  So no risk of regressions
from that exists here.

Build tested for several different key arch on a recent linux-next
tree to ensure no silly typos crept in.

 FAQ: Why not make it tristate?
 ------------------------------
Upon detecting a non-modular driver making modular references, I don't
immediately convert them to tristate, since it increases functionality
that I can't readily test, and it may not have a sensible use case (e.g.
in the case of core arch support relating to timer ticks or similar.)
So instead the modular references are removed w/o changing the existing
functionality by default.

However there is no reason the original author or an interested user
with the capability to test can't nominate a driver to be tristate
either as the original intent, or as a functional and tested use case.

Paul.
---

Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Alexandre Courbot <gnurou@gmail.com>
Cc: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Jean-Jacques Hiblot <jjhiblot@traphandler.com>
Cc: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Cc: Kukjin Kim <kgene@kernel.org>
Cc: Nicolas Ferre <nicolas.ferre@atmel.com>
Cc: Pankaj Dubey <pankaj.dubey@samsung.com>
Cc: Roger Quadros <rogerq@ti.com>
Cc: Stephen Warren <swarren@wwwdotorg.org>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Tony Lindgren <tony@atomide.com>
Cc: linux-omap@vger.kernel.org
Cc: linux-samsung-soc@vger.kernel.org
Cc: linux-tegra@vger.kernel.org

Paul Gortmaker (6):
  memory: atmel-sdramc: make it explicitly non-modular
  memory: mvebu-devbus: make it explicitly non-modular
  memory: omap-gpmc: make it explicitly non-modular
  memory: tegra20-mc: make it explicitly non-modular
  memory: samsung/exynos-srom: make it explicitly non-modular
  memory: atmel-ebi: make it explicitly non-modular

 drivers/memory/atmel-ebi.c           |  9 ++-------
 drivers/memory/atmel-sdramc.c        | 11 ++++-------
 drivers/memory/mvebu-devbus.c        | 11 ++++-------
 drivers/memory/omap-gpmc.c           | 10 ----------
 drivers/memory/samsung/exynos-srom.c | 21 +++------------------
 drivers/memory/tegra20-mc.c          | 11 ++++-------
 6 files changed, 17 insertions(+), 56 deletions(-)

-- 
2.8.4

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2016-06-21 15:34 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-17  0:37 [PATCH 0/6] memory: remove modular usage from non-modular drivers Paul Gortmaker
2016-06-17  0:37 ` [PATCH 1/6] memory: atmel-sdramc: make it explicitly non-modular Paul Gortmaker
2016-06-17  7:30   ` Nicolas Ferre
2016-06-21 15:17   ` Alexandre Belloni
2016-06-17  0:37 ` [PATCH 2/6] memory: mvebu-devbus: " Paul Gortmaker
2016-06-17  0:37 ` [PATCH 3/6] memory: omap-gpmc: " Paul Gortmaker
2016-06-17  7:26   ` Roger Quadros
2016-06-17  0:37 ` [PATCH 4/6] memory: tegra20-mc: " Paul Gortmaker
2016-06-17  0:37 ` [PATCH 5/6] memory: samsung/exynos-srom: " Paul Gortmaker
2016-06-20  7:50   ` Krzysztof Kozlowski
2016-06-17  0:37 ` [PATCH 6/6] memory: atmel-ebi: " Paul Gortmaker
2016-06-17  7:35   ` Nicolas Ferre
2016-06-17  7:38     ` Boris Brezillon
2016-06-21 15:18   ` Alexandre Belloni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).