From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: linux-kernel@vger.kernel.org
Cc: "Paul Gortmaker" <paul.gortmaker@windriver.com>,
"Antoine Ténart" <antoine.tenart@free-electrons.com>,
"Hongzhou Yang" <hongzhou.yang@mediatek.com>,
"Jisheng Zhang" <jszhang@marvell.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Masahiro Yamada" <yamada.masahiro@socionext.com>,
"Sebastian Hesselbarth" <sebastian.hesselbarth@gmail.com>,
"Thomas Hebb" <tommyhebb@gmail.com>,
"Thomas Petazzoni" <thomas.petazzoni@free-electrons.com>,
"Viresh Kumar" <vireshk@kernel.org>,
spear-devel@list.st.com, linux-gpio@vger.kernel.org
Subject: [PATCH 0/3] pinctrl: berlin/mvebu/spear: make bool code non-modular
Date: Mon, 6 Feb 2017 03:03:49 -0500 [thread overview]
Message-ID: <20170206080352.17808-1-paul.gortmaker@windriver.com> (raw)
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: berlin, mvebu, and spear, which allows us
to have a good 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, i386
and x86_64.
An automated mapping from the object file to the Makefile and then
finally to the Kconfig variable (unless obj-y) and type follows as
a double check; run on the current code base in linux-next today.
Paul.
--
drivers/pinctrl/berlin/Makefile:obj-$(CONFIG_PINCTRL_BERLIN_BG2) += berlin-bg2.o
drivers/pinctrl/berlin/Kconfig:config PINCTRL_BERLIN_BG2
drivers/pinctrl/berlin/Kconfig: def_bool MACH_BERLIN_BG2
drivers/pinctrl/berlin/Makefile:obj-$(CONFIG_PINCTRL_BERLIN_BG2CD) += berlin-bg2cd.o
drivers/pinctrl/berlin/Kconfig:config PINCTRL_BERLIN_BG2CD
drivers/pinctrl/berlin/Kconfig: def_bool MACH_BERLIN_BG2CD
drivers/pinctrl/berlin/Makefile:obj-$(CONFIG_PINCTRL_BERLIN_BG2Q) += berlin-bg2q.o
drivers/pinctrl/berlin/Kconfig:config PINCTRL_BERLIN_BG2Q
drivers/pinctrl/berlin/Kconfig: def_bool MACH_BERLIN_BG2Q
drivers/pinctrl/berlin/Makefile:obj-$(CONFIG_PINCTRL_BERLIN_BG4CT) += berlin-bg4ct.o
drivers/pinctrl/berlin/Kconfig:config PINCTRL_BERLIN_BG4CT
drivers/pinctrl/berlin/Kconfig: bool "Marvell berlin4ct pin controller driver"
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_ARMADA_370) += pinctrl-armada-370.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_ARMADA_370
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_ARMADA_375) += pinctrl-armada-375.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_ARMADA_375
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_ARMADA_38X) += pinctrl-armada-38x.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_ARMADA_38X
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_ARMADA_39X) += pinctrl-armada-39x.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_ARMADA_39X
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_ARMADA_XP) += pinctrl-armada-xp.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_ARMADA_XP
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_DOVE) += pinctrl-dove.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_DOVE
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_KIRKWOOD) += pinctrl-kirkwood.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_KIRKWOOD
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/mvebu/Makefile:obj-y += pinctrl-mvebu.o
drivers/pinctrl/mvebu/Makefile:obj-$(CONFIG_PINCTRL_ORION) += pinctrl-orion.o
drivers/pinctrl/mvebu/Kconfig:config PINCTRL_ORION
drivers/pinctrl/mvebu/Kconfig: bool
drivers/pinctrl/spear/Makefile:obj-$(CONFIG_PINCTRL_SPEAR_PLGPIO) += pinctrl-plgpio.o
drivers/pinctrl/spear/Kconfig:config PINCTRL_SPEAR_PLGPIO
drivers/pinctrl/spear/Kconfig: bool "SPEAr SoC PLGPIO Controller"
drivers/pinctrl/spear/Makefile:obj-$(CONFIG_PINCTRL_SPEAR1310) += pinctrl-spear1310.o
drivers/pinctrl/spear/Kconfig:config PINCTRL_SPEAR1310
drivers/pinctrl/spear/Kconfig: bool "ST Microelectronics SPEAr1310 SoC pin controller driver"
drivers/pinctrl/spear/Makefile:obj-$(CONFIG_PINCTRL_SPEAR1340) += pinctrl-spear1340.o
drivers/pinctrl/spear/Kconfig:config PINCTRL_SPEAR1340
drivers/pinctrl/spear/Kconfig: bool "ST Microelectronics SPEAr1340 SoC pin controller driver"
drivers/pinctrl/spear/Makefile:obj-$(CONFIG_PINCTRL_SPEAR300) += pinctrl-spear300.o
drivers/pinctrl/spear/Kconfig:config PINCTRL_SPEAR300
drivers/pinctrl/spear/Kconfig: bool "ST Microelectronics SPEAr300 SoC pin controller driver"
drivers/pinctrl/spear/Makefile:obj-$(CONFIG_PINCTRL_SPEAR310) += pinctrl-spear310.o
drivers/pinctrl/spear/Kconfig:config PINCTRL_SPEAR310
drivers/pinctrl/spear/Kconfig: bool "ST Microelectronics SPEAr310 SoC pin controller driver"
drivers/pinctrl/spear/Makefile:obj-$(CONFIG_PINCTRL_SPEAR320) += pinctrl-spear320.o
drivers/pinctrl/spear/Kconfig:config PINCTRL_SPEAR320
drivers/pinctrl/spear/Kconfig: bool "ST Microelectronics SPEAr320 SoC pin controller driver"
---
Cc: "Antoine Ténart" <antoine.tenart@free-electrons.com>
Cc: Hongzhou Yang <hongzhou.yang@mediatek.com>
Cc: Jisheng Zhang <jszhang@marvell.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Cc: Thomas Hebb <tommyhebb@gmail.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Viresh Kumar <vireshk@kernel.org>
Cc: spear-devel@list.st.com
Cc: linux-gpio@vger.kernel.org
Paul Gortmaker (3):
pinctrl: mvebu: make bool drivers explicitly non-modular
pinctrl: spear: make bool drivers explicitly non-modular
pinctrl: berlin: make bool drivers explicitly non-modular
drivers/pinctrl/berlin/berlin-bg2.c | 9 ++-------
drivers/pinctrl/berlin/berlin-bg2cd.c | 9 ++-------
drivers/pinctrl/berlin/berlin-bg2q.c | 9 ++-------
drivers/pinctrl/berlin/berlin-bg4ct.c | 9 ++-------
drivers/pinctrl/mvebu/pinctrl-armada-370.c | 8 +-------
drivers/pinctrl/mvebu/pinctrl-armada-375.c | 8 +-------
drivers/pinctrl/mvebu/pinctrl-armada-38x.c | 8 +-------
drivers/pinctrl/mvebu/pinctrl-armada-39x.c | 8 +-------
drivers/pinctrl/mvebu/pinctrl-armada-xp.c | 8 +-------
drivers/pinctrl/mvebu/pinctrl-dove.c | 17 ++---------------
drivers/pinctrl/mvebu/pinctrl-kirkwood.c | 8 +-------
drivers/pinctrl/mvebu/pinctrl-mvebu.c | 1 -
drivers/pinctrl/mvebu/pinctrl-orion.c | 8 +-------
drivers/pinctrl/spear/pinctrl-plgpio.c | 7 +------
drivers/pinctrl/spear/pinctrl-spear1310.c | 12 ------------
drivers/pinctrl/spear/pinctrl-spear1340.c | 12 ------------
drivers/pinctrl/spear/pinctrl-spear300.c | 12 ------------
drivers/pinctrl/spear/pinctrl-spear310.c | 12 ------------
drivers/pinctrl/spear/pinctrl-spear320.c | 12 ------------
19 files changed, 18 insertions(+), 159 deletions(-)
--
2.11.0
next reply other threads:[~2017-02-06 8:03 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-06 8:03 Paul Gortmaker [this message]
2017-02-06 8:03 ` [PATCH 1/3] pinctrl: mvebu: make bool drivers explicitly non-modular Paul Gortmaker
2017-02-13 13:22 ` Linus Walleij
2017-02-06 8:03 ` [PATCH 2/3] pinctrl: spear: " Paul Gortmaker
2017-02-07 4:12 ` Viresh Kumar
2017-02-13 13:24 ` Linus Walleij
2017-02-06 8:03 ` [PATCH 3/3] pinctrl: berlin: " Paul Gortmaker
2017-02-07 11:19 ` Jisheng Zhang
2017-02-13 13:25 ` 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=20170206080352.17808-1-paul.gortmaker@windriver.com \
--to=paul.gortmaker@windriver.com \
--cc=antoine.tenart@free-electrons.com \
--cc=hongzhou.yang@mediatek.com \
--cc=jszhang@marvell.com \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sebastian.hesselbarth@gmail.com \
--cc=spear-devel@list.st.com \
--cc=thomas.petazzoni@free-electrons.com \
--cc=tommyhebb@gmail.com \
--cc=vireshk@kernel.org \
--cc=yamada.masahiro@socionext.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).