* [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB
@ 2016-06-02 12:14 Linus Walleij
2016-06-02 18:05 ` Will Deacon
0 siblings, 1 reply; 6+ messages in thread
From: Linus Walleij @ 2016-06-02 12:14 UTC (permalink / raw)
To: linux-arm-kernel
This replaces:
- "select ARCH_REQUIRE_GPIOLIB" with "select GPIOLIB" as this can
now be selected directly.
- "select ARCH_WANT_OPTIONAL_GPIOLIB" with no dependency: GPIOLIB
is now selectable by everyone, so we need not declare our
intent to select it.
Cc: Michael B?sch <m@bues.ch>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
ARM64/AArch64 maintainers:
Please apply this patch directly for cleanups.
---
arch/arm64/Kconfig | 1 -
arch/arm64/Kconfig.platforms | 10 +++++-----
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
index 76747d92bc72..2cfbf83b2498 100644
--- a/arch/arm64/Kconfig
+++ b/arch/arm64/Kconfig
@@ -12,7 +12,6 @@ config ARM64
select ARCH_USE_CMPXCHG_LOCKREF
select ARCH_SUPPORTS_ATOMIC_RMW
select ARCH_SUPPORTS_NUMA_BALANCING
- select ARCH_WANT_OPTIONAL_GPIOLIB
select ARCH_WANT_COMPAT_IPC_PARSE_VERSION
select ARCH_WANT_FRAME_POINTERS
select ARCH_HAS_UBSAN_SANITIZE_ALL
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 7ef1d05859ae..fe66708e9a20 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -16,15 +16,15 @@ config ARCH_ALPINE
config ARCH_BCM_IPROC
bool "Broadcom iProc SoC Family"
select COMMON_CLK_IPROC
+ select GPIOLIB
select PINCTRL
- select ARCH_REQUIRE_GPIOLIB
help
This enables support for Broadcom iProc based SoCs
config ARCH_BERLIN
bool "Marvell Berlin SoC Family"
- select ARCH_REQUIRE_GPIOLIB
select DW_APB_ICTL
+ select GPIOLIB
select PINCTRL
help
This enables support for Marvell Berlin SoC Family
@@ -89,7 +89,7 @@ config ARCH_QCOM
config ARCH_ROCKCHIP
bool "Rockchip Platforms"
select ARCH_HAS_RESET_CONTROLLER
- select ARCH_REQUIRE_GPIOLIB
+ select GPIOLIB
select PINCTRL
select PINCTRL_ROCKCHIP
select ROCKCHIP_TIMER
@@ -129,11 +129,11 @@ config ARCH_STRATIX10
config ARCH_TEGRA
bool "NVIDIA Tegra SoC Family"
select ARCH_HAS_RESET_CONTROLLER
- select ARCH_REQUIRE_GPIOLIB
select CLKDEV_LOOKUP
select CLKSRC_MMIO
select CLKSRC_OF
select GENERIC_CLOCKEVENTS
+ select GPIOLIB
select HAVE_CLK
select PINCTRL
select RESET_CONTROLLER
@@ -158,8 +158,8 @@ config ARCH_UNIPHIER
config ARCH_VEXPRESS
bool "ARMv8 software model (Versatile Express)"
- select ARCH_REQUIRE_GPIOLIB
select COMMON_CLK_VERSATILE
+ select GPIOLIB
select POWER_RESET_VEXPRESS
select VEXPRESS_CONFIG
help
--
2.4.11
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB
2016-06-02 12:14 [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB Linus Walleij
@ 2016-06-02 18:05 ` Will Deacon
2016-06-02 22:28 ` Linus Walleij
0 siblings, 1 reply; 6+ messages in thread
From: Will Deacon @ 2016-06-02 18:05 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jun 02, 2016 at 02:14:30PM +0200, Linus Walleij wrote:
> This replaces:
>
> - "select ARCH_REQUIRE_GPIOLIB" with "select GPIOLIB" as this can
> now be selected directly.
>
> - "select ARCH_WANT_OPTIONAL_GPIOLIB" with no dependency: GPIOLIB
> is now selectable by everyone, so we need not declare our
> intent to select it.
>
> Cc: Michael B?sch <m@bues.ch>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> ARM64/AArch64 maintainers:
> Please apply this patch directly for cleanups.
Just checking -- you're aiming to have this in 4.7-rc ?
Will
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB
2016-06-02 18:05 ` Will Deacon
@ 2016-06-02 22:28 ` Linus Walleij
2016-06-03 9:16 ` Will Deacon
0 siblings, 1 reply; 6+ messages in thread
From: Linus Walleij @ 2016-06-02 22:28 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Jun 2, 2016 at 8:05 PM, Will Deacon <will.deacon@arm.com> wrote:
> On Thu, Jun 02, 2016 at 02:14:30PM +0200, Linus Walleij wrote:
>> This replaces:
>>
>> - "select ARCH_REQUIRE_GPIOLIB" with "select GPIOLIB" as this can
>> now be selected directly.
>>
>> - "select ARCH_WANT_OPTIONAL_GPIOLIB" with no dependency: GPIOLIB
>> is now selectable by everyone, so we need not declare our
>> intent to select it.
>>
>> Cc: Michael B?sch <m@bues.ch>
>> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
>> ---
>> ARM64/AArch64 maintainers:
>> Please apply this patch directly for cleanups.
>
> Just checking -- you're aiming to have this in 4.7-rc ?
No cleanups for v4.8, not fixes for v4.7.
There is no hurry, I have all patience in the world.
Just letting these trickle out in the beginning of the
development cycle to be safe.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB
2016-06-02 22:28 ` Linus Walleij
@ 2016-06-03 9:16 ` Will Deacon
0 siblings, 0 replies; 6+ messages in thread
From: Will Deacon @ 2016-06-03 9:16 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 03, 2016 at 12:28:15AM +0200, Linus Walleij wrote:
> On Thu, Jun 2, 2016 at 8:05 PM, Will Deacon <will.deacon@arm.com> wrote:
> > On Thu, Jun 02, 2016 at 02:14:30PM +0200, Linus Walleij wrote:
> >> This replaces:
> >>
> >> - "select ARCH_REQUIRE_GPIOLIB" with "select GPIOLIB" as this can
> >> now be selected directly.
> >>
> >> - "select ARCH_WANT_OPTIONAL_GPIOLIB" with no dependency: GPIOLIB
> >> is now selectable by everyone, so we need not declare our
> >> intent to select it.
> >>
> >> Cc: Michael B?sch <m@bues.ch>
> >> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> >> ---
> >> ARM64/AArch64 maintainers:
> >> Please apply this patch directly for cleanups.
> >
> > Just checking -- you're aiming to have this in 4.7-rc ?
>
> No cleanups for v4.8, not fixes for v4.7.
> There is no hurry, I have all patience in the world.
Okey doke! I thought this might be blocking some bigger work for the merge
window, but I'll leave it to Catalin given that's not the case.
Thanks for the clarification,
Will
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB
@ 2016-08-02 7:46 Linus Walleij
2016-08-02 9:48 ` Will Deacon
0 siblings, 1 reply; 6+ messages in thread
From: Linus Walleij @ 2016-08-02 7:46 UTC (permalink / raw)
To: linux-arm-kernel
This replaces:
- "select ARCH_REQUIRE_GPIOLIB" with "select GPIOLIB" as this can
now be selected directly.
- "select ARCH_WANT_OPTIONAL_GPIOLIB" with no dependency: GPIOLIB
is now selectable by everyone, so we need not declare our
intent to select it.
Cc: Michael B?sch <m@bues.ch>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
ARM64/AArch64 maintainers:
I see you must have missed this patch. Please ACK this rebased
version so I can send it upstream in this (v4.8) merge window.
---
arch/arm64/Kconfig | 1 -
arch/arm64/Kconfig.platforms | 12 ++++++------
2 files changed, 6 insertions(+), 7 deletions(-)
diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
index 9f8b99e20557..227a6171fce4 100644
--- a/arch/arm64/Kconfig
+++ b/arch/arm64/Kconfig
@@ -14,7 +14,6 @@ config ARM64
select ARCH_USE_CMPXCHG_LOCKREF
select ARCH_SUPPORTS_ATOMIC_RMW
select ARCH_SUPPORTS_NUMA_BALANCING
- select ARCH_WANT_OPTIONAL_GPIOLIB
select ARCH_WANT_COMPAT_IPC_PARSE_VERSION
select ARCH_WANT_FRAME_POINTERS
select ARCH_HAS_UBSAN_SANITIZE_ALL
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index bb2616b16157..4fb0bef0f094 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -15,8 +15,8 @@ config ARCH_ALPINE
config ARCH_BCM2835
bool "Broadcom BCM2835 family"
- select ARCH_REQUIRE_GPIOLIB
select CLKSRC_OF
+ select GPIOLIB
select PINCTRL
select PINCTRL_BCM2835
select ARM_AMBA
@@ -29,15 +29,15 @@ config ARCH_BCM2835
config ARCH_BCM_IPROC
bool "Broadcom iProc SoC Family"
select COMMON_CLK_IPROC
+ select GPIOLIB
select PINCTRL
- select ARCH_REQUIRE_GPIOLIB
help
This enables support for Broadcom iProc based SoCs
config ARCH_BERLIN
bool "Marvell Berlin SoC Family"
- select ARCH_REQUIRE_GPIOLIB
select DW_APB_ICTL
+ select GPIOLIB
select PINCTRL
help
This enables support for Marvell Berlin SoC Family
@@ -108,7 +108,7 @@ config ARCH_QCOM
config ARCH_ROCKCHIP
bool "Rockchip Platforms"
select ARCH_HAS_RESET_CONTROLLER
- select ARCH_REQUIRE_GPIOLIB
+ select GPIOLIB
select PINCTRL
select PINCTRL_ROCKCHIP
select ROCKCHIP_TIMER
@@ -154,11 +154,11 @@ config ARCH_STRATIX10
config ARCH_TEGRA
bool "NVIDIA Tegra SoC Family"
select ARCH_HAS_RESET_CONTROLLER
- select ARCH_REQUIRE_GPIOLIB
select CLKDEV_LOOKUP
select CLKSRC_MMIO
select CLKSRC_OF
select GENERIC_CLOCKEVENTS
+ select GPIOLIB
select HAVE_CLK
select PINCTRL
select RESET_CONTROLLER
@@ -183,8 +183,8 @@ config ARCH_UNIPHIER
config ARCH_VEXPRESS
bool "ARMv8 software model (Versatile Express)"
- select ARCH_REQUIRE_GPIOLIB
select COMMON_CLK_VERSATILE
+ select GPIOLIB
select PM
select PM_GENERIC_DOMAINS
select POWER_RESET_VEXPRESS
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB
2016-08-02 7:46 Linus Walleij
@ 2016-08-02 9:48 ` Will Deacon
0 siblings, 0 replies; 6+ messages in thread
From: Will Deacon @ 2016-08-02 9:48 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 02, 2016 at 09:46:21AM +0200, Linus Walleij wrote:
> This replaces:
>
> - "select ARCH_REQUIRE_GPIOLIB" with "select GPIOLIB" as this can
> now be selected directly.
>
> - "select ARCH_WANT_OPTIONAL_GPIOLIB" with no dependency: GPIOLIB
> is now selectable by everyone, so we need not declare our
> intent to select it.
>
> Cc: Michael B?sch <m@bues.ch>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> ARM64/AArch64 maintainers:
> I see you must have missed this patch. Please ACK this rebased
> version so I can send it upstream in this (v4.8) merge window.
> ---
Sorry about that:
Acked-by: Will Deacon <will.deacon@arm.com>
Will
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-08-02 9:48 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-02 12:14 [PATCH] arm64: do away with ARCH_[WANT_OPTIONAL|REQUIRE]_GPIOLIB Linus Walleij
2016-06-02 18:05 ` Will Deacon
2016-06-02 22:28 ` Linus Walleij
2016-06-03 9:16 ` Will Deacon
-- strict thread matches above, loose matches on Subject: below --
2016-08-02 7:46 Linus Walleij
2016-08-02 9:48 ` Will Deacon
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).