From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: linux-kernel@vger.kernel.org
Cc: Paul Gortmaker <paul.gortmaker@windriver.com>,
Alexandre Courbot <gnurou@gmail.com>,
Ashwini Ghuge <aghuge@nvidia.com>,
bcm-kernel-feedback-list@broadcom.com,
Eric Anholt <eric@anholt.net>,
Florian Fainelli <f.fainelli@gmail.com>,
Jon Mason <jonmason@broadcom.com>,
Linus Walleij <linus.walleij@linaro.org>,
Pritesh Raithatha <praithatha@nvidia.com>,
Ray Jui <rjui@broadcom.com>,
Scott Branden <sbranden@broadcom.com>,
Sherman Yin <syin@broadcom.com>,
Stefan Wahren <stefan.wahren@i2se.com>,
Stephen Warren <swarren@wwwdotorg.org>,
Thierry Reding <thierry.reding@gmail.com>,
Thomas Abraham <thomas.ab@samsung.com>,
Tomasz Figa <tomasz.figa@gmail.com>,
linux-gpio@vger.kernel.org, linux-rpi-kernel@lists.infradead.org,
linux-samsung-soc@v
Subject: [PATCH 0/3] pinctrl: bcm/samsung/tegra: make bool code non-modular
Date: Mon, 22 May 2017 16:56:45 -0400 [thread overview]
Message-ID: <20170522205648.23545-1-paul.gortmaker@windriver.com> (raw)
This is the last of the pinctrl cleanups I have in my queue. So
hopefully once these are in and any current but out-of-tree drivers
are submitted, there won't be any more examples of non-modular code
using modular infrastructure for people to inadvertently copy off of.
Nothing really new here -- if the Kconfig and/or the Makefile does
not allow building the code as modular, then we don't need to be
using module_xyz() functions and/or MODULE_XYZ macros.
Here we audit and remove such instances in three subdirectories of
the pinctrl subsystem: bcm, samsung, and tegra, which allows us
to have a positive net removal of unused code as per the diffstat.
Rather than do one commit per driver, we take advantage of the
fact that the drivers are grouped already and bundle the changes
per family, since the changes are not really deeply technical.
Build tested for allmodconfig on linux-next for arm, arm64; also
explicitly compiled each file for both arch to ensure nobody got
missed on the allmodconfig. (there are missing renesas_sdhi syms
in today's linux-next arm but that is not caused by these changes.)
Linus - thanks for your patience in merging all these to date.
Paul.
---
Cc: Alexandre Courbot <gnurou@gmail.com>
Cc: Ashwini Ghuge <aghuge@nvidia.com>
Cc: bcm-kernel-feedback-list@broadcom.com
Cc: Eric Anholt <eric@anholt.net>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Jon Mason <jonmason@broadcom.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Pritesh Raithatha <praithatha@nvidia.com>
Cc: Ray Jui <rjui@broadcom.com>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Sherman Yin <syin@broadcom.com>
Cc: Stefan Wahren <stefan.wahren@i2se.com>
Cc: Stephen Warren <swarren@wwwdotorg.org>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Thomas Abraham <thomas.ab@samsung.com>
Cc: Tomasz Figa <tomasz.figa@gmail.com>
Cc: linux-gpio@vger.kernel.org
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-samsung-soc@vger.kernel.org
Cc: linux-tegra@vger.kernel.org
Paul Gortmaker (3):
pinctrl: samsung: clean up modular vs. non-modular distinctions
pinctrl: tegra: clean up modular vs. non-modular distinctions
pinctrl: bcm: clean up modular vs. non-modular distinctions
drivers/pinctrl/bcm/pinctrl-bcm281xx.c | 16 ++++++++--------
drivers/pinctrl/bcm/pinctrl-bcm2835.c | 20 +++-----------------
drivers/pinctrl/bcm/pinctrl-cygnus-mux.c | 11 +++++------
drivers/pinctrl/samsung/pinctrl-exynos.c | 1 -
drivers/pinctrl/samsung/pinctrl-exynos5440.c | 15 +++------------
drivers/pinctrl/samsung/pinctrl-s3c24xx.c | 2 +-
drivers/pinctrl/samsung/pinctrl-s3c64xx.c | 2 +-
drivers/pinctrl/samsung/pinctrl-samsung.c | 13 +------------
drivers/pinctrl/tegra/pinctrl-tegra.c | 1 -
drivers/pinctrl/tegra/pinctrl-tegra114.c | 11 ++++-------
drivers/pinctrl/tegra/pinctrl-tegra124.c | 11 ++++-------
drivers/pinctrl/tegra/pinctrl-tegra20.c | 11 ++++-------
drivers/pinctrl/tegra/pinctrl-tegra210.c | 9 ++-------
drivers/pinctrl/tegra/pinctrl-tegra30.c | 11 ++++-------
14 files changed, 40 insertions(+), 94 deletions(-)
--
2.11.0
next reply other threads:[~2017-05-22 20:56 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-22 20:56 Paul Gortmaker [this message]
2017-05-22 20:56 ` [PATCH 1/3] pinctrl: samsung: clean up modular vs. non-modular distinctions Paul Gortmaker
2017-05-23 14:51 ` Krzysztof Kozlowski
2017-05-23 15:54 ` Paul Gortmaker
2017-05-29 8:24 ` Linus Walleij
2017-05-22 20:56 ` [PATCH 3/3] pinctrl: bcm: " Paul Gortmaker
2017-05-23 20:11 ` Stefan Wahren
2017-05-23 22:15 ` Scott Branden
2017-05-24 0:12 ` Paul Gortmaker
2017-05-24 0:25 ` Florian Fainelli
2017-05-24 0:37 ` Scott Branden
2017-05-24 0:42 ` Florian Fainelli
2017-05-24 0:34 ` Scott Branden
2017-05-29 8:31 ` Linus Walleij
2017-05-29 16:35 ` Scott Branden
2017-05-30 7:41 ` Linus Walleij
[not found] ` <20170522205648.23545-1-paul.gortmaker-CWA4WttNNZF54TAoqtyWWQ@public.gmane.org>
2017-05-22 20:56 ` [PATCH 2/3] pinctrl: tegra: " Paul Gortmaker
2017-05-29 8:25 ` Linus Walleij
2017-05-29 8:33 ` [PATCH 0/3] pinctrl: bcm/samsung/tegra: make bool code non-modular Linus Walleij
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170522205648.23545-1-paul.gortmaker@windriver.com \
--to=paul.gortmaker@windriver.com \
--cc=aghuge@nvidia.com \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=eric@anholt.net \
--cc=f.fainelli@gmail.com \
--cc=gnurou@gmail.com \
--cc=jonmason@broadcom.com \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rpi-kernel@lists.infradead.org \
--cc=linux-samsung-soc@v \
--cc=praithatha@nvidia.com \
--cc=rjui@broadcom.com \
--cc=sbranden@broadcom.com \
--cc=stefan.wahren@i2se.com \
--cc=swarren@wwwdotorg.org \
--cc=syin@broadcom.com \
--cc=thierry.reding@gmail.com \
--cc=thomas.ab@samsung.com \
--cc=tomasz.figa@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).