From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: linux-kernel@vger.kernel.org, Philipp Zabel <p.zabel@pengutronix.de>
Cc: linux-mips@linux-mips.org, kernel@stlinux.com,
David Airlie <airlied@linux.ie>,
Catalin Marinas <catalin.marinas@arm.com>,
Linus Walleij <linus.walleij@linaro.org>,
Will Deacon <will.deacon@arm.com>,
dri-devel@lists.freedesktop.org,
Russell King <linux@arm.linux.org.uk>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Thierry Reding <thierry.reding@gmail.com>,
Benjamin Gaignard <benjamin.gaignard@linaro.org>,
Heiko Stuebner <heiko@sntech.de>,
Alexandre Courbot <gnurou@gmail.com>,
Michael Turquette <mturquette@linaro.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
linux-rockchip@lists.infradead.org, Chen-Yu Tsai <wens@csie.org>,
Maxime Coquelin <maxime.coquelin@st.com>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
Arnd Bergmann <arnd@arndb.de>,
Stephen Warren <swarren@wwwdotorg.org>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Hans de Goede <hdegoede@redhat.>
Subject: [RFC PATCH 0/7] reset: make RESET_CONTROLLER a select'ed option
Date: Thu, 5 Nov 2015 20:15:21 +0900 [thread overview]
Message-ID: <1446722128-11961-1-git-send-email-yamada.masahiro@socionext.com> (raw)
When I was implementing a new reset controller for my SoCs,
I struggled to make my sub-menu shown under the reset
controller menu.
I noticed the Kconfig in reset sub-system are screwed up due to two
config options (ARCH_HAS_RESET_CONTROLLER and RESET_CONTROLLER).
I think only the former should be select'ed by relevant SoCs,
but in fact the latter is also select'ed here and there.
Mixing "select" to a user-configurable option is a mess.
Finally, I started to wonder whether it could be more simpler?
The first patch drops ARCH_HAS_RESET_CONTROLLER.
RESET_CONTROLLER should be directly selected by SoCs.
The rest of this series are minor clean ups in other
sub-systems.
I can postpone them if changes over cross sub-systems
are not preferred.
Masahiro Yamada (7):
reset: drop ARCH_HAS_RESET_CONTROLLER
spi: sunxi: remove redundant "depends on RESET_CONTROLLER"
spi: tegra: remove redundant "depends on RESET_CONTROLLER"
pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER"
drm/sti: replace "select RESET_CONTROLLER" with "depends on ..."
drm/rockchip: remove redundant "depends on RESET_CONTROLLER"
drm/tegra: tegra: remove redundant "depends on RESET_CONTROLLER"
arch/arm/Kconfig | 3 +--
arch/arm/mach-berlin/Kconfig | 2 +-
arch/arm/mach-imx/Kconfig | 2 +-
arch/arm/mach-mmp/Kconfig | 4 ++--
arch/arm/mach-prima2/Kconfig | 2 +-
arch/arm/mach-rockchip/Kconfig | 2 +-
arch/arm/mach-sti/Kconfig | 1 -
arch/arm/mach-sunxi/Kconfig | 1 -
arch/arm/mach-tegra/Kconfig | 1 -
arch/arm64/Kconfig.platforms | 3 +--
arch/mips/Kconfig | 4 +---
drivers/gpu/drm/rockchip/Kconfig | 1 -
drivers/gpu/drm/sti/Kconfig | 4 ++--
drivers/gpu/drm/tegra/Kconfig | 1 -
drivers/pinctrl/sunxi/Kconfig | 2 --
drivers/reset/Kconfig | 12 +++++++-----
drivers/reset/sti/Kconfig | 1 -
drivers/spi/Kconfig | 6 ++----
18 files changed, 20 insertions(+), 32 deletions(-)
--
1.9.1
WARNING: multiple messages have this Message-ID (diff)
From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: linux-kernel@vger.kernel.org, Philipp Zabel <p.zabel@pengutronix.de>
Cc: "Masahiro Yamada" <yamada.masahiro@socionext.com>,
linux-mips@linux-mips.org,
"Patrice Chotard" <patrice.chotard@st.com>,
"Sascha Hauer" <kernel@pengutronix.de>,
kernel@stlinux.com, "Haojian Zhuang" <haojian.zhuang@gmail.com>,
"Eric Miao" <eric.y.miao@gmail.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Michael Turquette" <mturquette@linaro.org>,
"Vishnu Patekar" <vishnupatekar0510@gmail.com>,
"Arnd Bergmann" <arnd@arndb.de>,
"David Airlie" <airlied@linux.ie>,
"Barry Song" <baohua@kernel.org>,
"Catalin Marinas" <catalin.marinas@arm.com>,
"Terje Bergström" <tbergstrom@nvidia.com>,
"Heiko Stuebner" <heiko@sntech.de>,
"Tuomas Tynkkynen" <ttynkkynen@nvidia.com>,
"Hans de Goede" <hdegoede@redhat.com>,
linux-rockchip@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
"Will Deacon" <will.deacon@arm.com>,
"Ralf Baechle" <ralf@linux-mips.org>,
"Alexandre Courbot" <gnurou@gmail.com>,
"Daniel Lezcano" <daniel.lezcano@linaro.org>,
"Benjamin Gaignard" <benjamin.gaignard@linaro.org>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Mark Yao" <mark.yao@rock-chips.com>,
"Vincent Abriou" <vincent.abriou@st.com>,
"Sebastian Hesselbarth" <sebastian.hesselbarth@gmail.com>,
"Jens Kuske" <jenskuske@gmail.com>,
"Russell King" <linux@arm.linux.org.uk>,
linux-gpio@vger.kernel.org, dri-devel@lists.freedesktop.org,
linux-tegra@vger.kernel.org,
"Maxime Coquelin" <maxime.coquelin@st.com>,
"Srinivas Kandagatla" <srinivas.kandagatla@gmail.com>,
"Shawn Guo" <shawnguo@kernel.org>,
"Mark Brown" <broonie@kernel.org>,
"Stephen Warren" <swarren@wwwdotorg.org>,
linux-spi@vger.kernel.org, "Chen-Yu Tsai" <wens@csie.org>,
"Maxime Ripard" <maxime.ripard@free-electrons.com>
Subject: [RFC PATCH 0/7] reset: make RESET_CONTROLLER a select'ed option
Date: Thu, 5 Nov 2015 20:15:21 +0900 [thread overview]
Message-ID: <1446722128-11961-1-git-send-email-yamada.masahiro@socionext.com> (raw)
When I was implementing a new reset controller for my SoCs,
I struggled to make my sub-menu shown under the reset
controller menu.
I noticed the Kconfig in reset sub-system are screwed up due to two
config options (ARCH_HAS_RESET_CONTROLLER and RESET_CONTROLLER).
I think only the former should be select'ed by relevant SoCs,
but in fact the latter is also select'ed here and there.
Mixing "select" to a user-configurable option is a mess.
Finally, I started to wonder whether it could be more simpler?
The first patch drops ARCH_HAS_RESET_CONTROLLER.
RESET_CONTROLLER should be directly selected by SoCs.
The rest of this series are minor clean ups in other
sub-systems.
I can postpone them if changes over cross sub-systems
are not preferred.
Masahiro Yamada (7):
reset: drop ARCH_HAS_RESET_CONTROLLER
spi: sunxi: remove redundant "depends on RESET_CONTROLLER"
spi: tegra: remove redundant "depends on RESET_CONTROLLER"
pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER"
drm/sti: replace "select RESET_CONTROLLER" with "depends on ..."
drm/rockchip: remove redundant "depends on RESET_CONTROLLER"
drm/tegra: tegra: remove redundant "depends on RESET_CONTROLLER"
arch/arm/Kconfig | 3 +--
arch/arm/mach-berlin/Kconfig | 2 +-
arch/arm/mach-imx/Kconfig | 2 +-
arch/arm/mach-mmp/Kconfig | 4 ++--
arch/arm/mach-prima2/Kconfig | 2 +-
arch/arm/mach-rockchip/Kconfig | 2 +-
arch/arm/mach-sti/Kconfig | 1 -
arch/arm/mach-sunxi/Kconfig | 1 -
arch/arm/mach-tegra/Kconfig | 1 -
arch/arm64/Kconfig.platforms | 3 +--
arch/mips/Kconfig | 4 +---
drivers/gpu/drm/rockchip/Kconfig | 1 -
drivers/gpu/drm/sti/Kconfig | 4 ++--
drivers/gpu/drm/tegra/Kconfig | 1 -
drivers/pinctrl/sunxi/Kconfig | 2 --
drivers/reset/Kconfig | 12 +++++++-----
drivers/reset/sti/Kconfig | 1 -
drivers/spi/Kconfig | 6 ++----
18 files changed, 20 insertions(+), 32 deletions(-)
--
1.9.1
WARNING: multiple messages have this Message-ID (diff)
From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: linux-kernel@vger.kernel.org, Philipp Zabel <p.zabel@pengutronix.de>
Cc: linux-mips@linux-mips.org, kernel@stlinux.com,
David Airlie <airlied@linux.ie>,
Catalin Marinas <catalin.marinas@arm.com>,
Linus Walleij <linus.walleij@linaro.org>,
Will Deacon <will.deacon@arm.com>,
dri-devel@lists.freedesktop.org,
Russell King <linux@arm.linux.org.uk>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Thierry Reding <thierry.reding@gmail.com>,
Benjamin Gaignard <benjamin.gaignard@linaro.org>,
Heiko Stuebner <heiko@sntech.de>,
Alexandre Courbot <gnurou@gmail.com>,
Michael Turquette <mturquette@linaro.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
linux-rockchip@lists.infradead.org, Chen-Yu Tsai <wens@csie.org>,
Maxime Coquelin <maxime.coquelin@st.com>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
Arnd Bergmann <arnd@arndb.de>,
Stephen Warren <swarren@wwwdotorg.org>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Hans de Goede <hdegoede@redhat.
Subject: [RFC PATCH 0/7] reset: make RESET_CONTROLLER a select'ed option
Date: Thu, 5 Nov 2015 20:15:21 +0900 [thread overview]
Message-ID: <1446722128-11961-1-git-send-email-yamada.masahiro@socionext.com> (raw)
When I was implementing a new reset controller for my SoCs,
I struggled to make my sub-menu shown under the reset
controller menu.
I noticed the Kconfig in reset sub-system are screwed up due to two
config options (ARCH_HAS_RESET_CONTROLLER and RESET_CONTROLLER).
I think only the former should be select'ed by relevant SoCs,
but in fact the latter is also select'ed here and there.
Mixing "select" to a user-configurable option is a mess.
Finally, I started to wonder whether it could be more simpler?
The first patch drops ARCH_HAS_RESET_CONTROLLER.
RESET_CONTROLLER should be directly selected by SoCs.
The rest of this series are minor clean ups in other
sub-systems.
I can postpone them if changes over cross sub-systems
are not preferred.
Masahiro Yamada (7):
reset: drop ARCH_HAS_RESET_CONTROLLER
spi: sunxi: remove redundant "depends on RESET_CONTROLLER"
spi: tegra: remove redundant "depends on RESET_CONTROLLER"
pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER"
drm/sti: replace "select RESET_CONTROLLER" with "depends on ..."
drm/rockchip: remove redundant "depends on RESET_CONTROLLER"
drm/tegra: tegra: remove redundant "depends on RESET_CONTROLLER"
arch/arm/Kconfig | 3 +--
arch/arm/mach-berlin/Kconfig | 2 +-
arch/arm/mach-imx/Kconfig | 2 +-
arch/arm/mach-mmp/Kconfig | 4 ++--
arch/arm/mach-prima2/Kconfig | 2 +-
arch/arm/mach-rockchip/Kconfig | 2 +-
arch/arm/mach-sti/Kconfig | 1 -
arch/arm/mach-sunxi/Kconfig | 1 -
arch/arm/mach-tegra/Kconfig | 1 -
arch/arm64/Kconfig.platforms | 3 +--
arch/mips/Kconfig | 4 +---
drivers/gpu/drm/rockchip/Kconfig | 1 -
drivers/gpu/drm/sti/Kconfig | 4 ++--
drivers/gpu/drm/tegra/Kconfig | 1 -
drivers/pinctrl/sunxi/Kconfig | 2 --
drivers/reset/Kconfig | 12 +++++++-----
drivers/reset/sti/Kconfig | 1 -
drivers/spi/Kconfig | 6 ++----
18 files changed, 20 insertions(+), 32 deletions(-)
--
1.9.1
WARNING: multiple messages have this Message-ID (diff)
From: yamada.masahiro@socionext.com (Masahiro Yamada)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 0/7] reset: make RESET_CONTROLLER a select'ed option
Date: Thu, 5 Nov 2015 20:15:21 +0900 [thread overview]
Message-ID: <1446722128-11961-1-git-send-email-yamada.masahiro@socionext.com> (raw)
When I was implementing a new reset controller for my SoCs,
I struggled to make my sub-menu shown under the reset
controller menu.
I noticed the Kconfig in reset sub-system are screwed up due to two
config options (ARCH_HAS_RESET_CONTROLLER and RESET_CONTROLLER).
I think only the former should be select'ed by relevant SoCs,
but in fact the latter is also select'ed here and there.
Mixing "select" to a user-configurable option is a mess.
Finally, I started to wonder whether it could be more simpler?
The first patch drops ARCH_HAS_RESET_CONTROLLER.
RESET_CONTROLLER should be directly selected by SoCs.
The rest of this series are minor clean ups in other
sub-systems.
I can postpone them if changes over cross sub-systems
are not preferred.
Masahiro Yamada (7):
reset: drop ARCH_HAS_RESET_CONTROLLER
spi: sunxi: remove redundant "depends on RESET_CONTROLLER"
spi: tegra: remove redundant "depends on RESET_CONTROLLER"
pinctrl: sunxi: remove redundant "depends on RESET_CONTROLLER"
drm/sti: replace "select RESET_CONTROLLER" with "depends on ..."
drm/rockchip: remove redundant "depends on RESET_CONTROLLER"
drm/tegra: tegra: remove redundant "depends on RESET_CONTROLLER"
arch/arm/Kconfig | 3 +--
arch/arm/mach-berlin/Kconfig | 2 +-
arch/arm/mach-imx/Kconfig | 2 +-
arch/arm/mach-mmp/Kconfig | 4 ++--
arch/arm/mach-prima2/Kconfig | 2 +-
arch/arm/mach-rockchip/Kconfig | 2 +-
arch/arm/mach-sti/Kconfig | 1 -
arch/arm/mach-sunxi/Kconfig | 1 -
arch/arm/mach-tegra/Kconfig | 1 -
arch/arm64/Kconfig.platforms | 3 +--
arch/mips/Kconfig | 4 +---
drivers/gpu/drm/rockchip/Kconfig | 1 -
drivers/gpu/drm/sti/Kconfig | 4 ++--
drivers/gpu/drm/tegra/Kconfig | 1 -
drivers/pinctrl/sunxi/Kconfig | 2 --
drivers/reset/Kconfig | 12 +++++++-----
drivers/reset/sti/Kconfig | 1 -
drivers/spi/Kconfig | 6 ++----
18 files changed, 20 insertions(+), 32 deletions(-)
--
1.9.1
next reply other threads:[~2015-11-05 11:15 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-05 11:15 Masahiro Yamada [this message]
2015-11-05 11:15 ` [RFC PATCH 0/7] reset: make RESET_CONTROLLER a select'ed option Masahiro Yamada
2015-11-05 11:15 ` Masahiro Yamada
2015-11-05 11:15 ` Masahiro Yamada
[not found] ` <1446722128-11961-1-git-send-email-yamada.masahiro-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
2015-11-05 11:15 ` [RFC PATCH 1/7] reset: drop ARCH_HAS_RESET_CONTROLLER Masahiro Yamada
2015-11-05 11:15 ` Masahiro Yamada
2015-11-05 11:15 ` Masahiro Yamada
2015-11-05 14:49 ` [RFC PATCH 0/7] reset: make RESET_CONTROLLER a select'ed option Arnd Bergmann
2015-11-05 14:49 ` Arnd Bergmann
2015-11-05 14:49 ` Arnd Bergmann
2015-11-05 14:49 ` Arnd Bergmann
2015-11-06 5:58 ` Masahiro Yamada
2015-11-06 5:58 ` Masahiro Yamada
2015-11-06 5:58 ` Masahiro Yamada
2015-11-06 9:29 ` Arnd Bergmann
2015-11-06 9:29 ` Arnd Bergmann
2015-11-06 9:29 ` Arnd Bergmann
2015-11-05 11:15 ` [RFC PATCH 2/7] spi: sunxi: remove redundant "depends on RESET_CONTROLLER" Masahiro Yamada
2015-11-05 11:28 ` Mark Brown
2015-11-05 11:28 ` Mark Brown
2015-11-05 12:11 ` Masahiro Yamada
2015-11-05 12:11 ` Masahiro Yamada
2015-11-05 12:20 ` Mark Brown
2015-11-05 12:20 ` Mark Brown
2015-11-05 12:32 ` Masahiro Yamada
2015-11-05 12:32 ` Masahiro Yamada
2015-11-05 15:05 ` Mark Brown
2015-11-05 15:05 ` Mark Brown
2015-11-06 6:00 ` Masahiro Yamada
2015-11-06 6:00 ` Masahiro Yamada
2015-11-06 10:21 ` Mark Brown
2015-11-06 10:21 ` Mark Brown
2015-11-05 11:15 ` [RFC PATCH 3/7] spi: tegra: " Masahiro Yamada
2015-11-05 11:50 ` Mark Brown
2015-11-06 6:02 ` Masahiro Yamada
2015-11-06 6:02 ` Masahiro Yamada
2015-11-06 10:22 ` Mark Brown
2015-11-05 11:15 ` [RFC PATCH 4/7] pinctrl: sunxi: " Masahiro Yamada
2015-11-05 11:15 ` Masahiro Yamada
2015-11-05 11:15 ` Masahiro Yamada
2015-11-05 13:40 ` Linus Walleij
2015-11-05 13:40 ` Linus Walleij
2015-11-05 13:40 ` Linus Walleij
2015-11-05 14:06 ` Linus Walleij
2015-11-05 14:06 ` Linus Walleij
2015-11-05 14:06 ` Linus Walleij
2015-11-16 3:40 ` Masahiro Yamada
2015-11-16 3:40 ` Masahiro Yamada
2015-11-16 3:40 ` Masahiro Yamada
2015-11-17 14:03 ` Linus Walleij
2015-11-17 14:03 ` Linus Walleij
2015-11-17 14:03 ` Linus Walleij
2015-11-05 11:15 ` [RFC PATCH 5/7] drm/sti: replace "select RESET_CONTROLLER" with "depends on ..." Masahiro Yamada
2015-11-05 11:15 ` [RFC PATCH 6/7] drm/rockchip: remove redundant "depends on RESET_CONTROLLER" Masahiro Yamada
2015-11-05 11:15 ` Masahiro Yamada
2015-11-05 11:15 ` [RFC PATCH 7/7] drm/tegra: tegra: " Masahiro Yamada
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=1446722128-11961-1-git-send-email-yamada.masahiro@socionext.com \
--to=yamada.masahiro@socionext.com \
--cc=airlied@linux.ie \
--cc=arnd@arndb.de \
--cc=benjamin.gaignard@linaro.org \
--cc=catalin.marinas@arm.com \
--cc=daniel.lezcano@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=gnurou@gmail.com \
--cc=haojian.zhuang@gmail.com \
--cc=hdegoede@redhat. \
--cc=heiko@sntech.de \
--cc=kernel@stlinux.com \
--cc=linus.walleij@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux@arm.linux.org.uk \
--cc=maxime.coquelin@st.com \
--cc=mturquette@linaro.org \
--cc=p.zabel@pengutronix.de \
--cc=sebastian.hesselbarth@gmail.com \
--cc=swarren@wwwdotorg.org \
--cc=thierry.reding@gmail.com \
--cc=wens@csie.org \
--cc=will.deacon@arm.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.