diff mbox

ARM: do away with final ARCH_REQUIRE_GPIOLIB

Message ID 1470123873-8073-1-git-send-email-linus.walleij@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Linus Walleij Aug. 2, 2016, 7:44 a.m. UTC
A new user of the Kconfig selection of ARCH_REQUIRE_GPIOLIB
has appeared. Replace with just selecting GPIOLIB.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
ARM SoC folks: I plan to send this directly in this merge
window to get rid of the old Kconfig symbols, please ACK
it.
---
 arch/arm/mach-clps711x/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Arnd Bergmann Aug. 2, 2016, 7:47 a.m. UTC | #1
On Tuesday, August 2, 2016 9:44:33 AM CEST Linus Walleij wrote:
> A new user of the Kconfig selection of ARCH_REQUIRE_GPIOLIB
> has appeared. Replace with just selecting GPIOLIB.
> 
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> ARM SoC folks: I plan to send this directly in this merge
> window to get rid of the old Kconfig symbols, please ACK
> it.

Acked-by: Arnd Bergmann <arnd@arndb.de>

Note that sending this one along with your patches requires
you to base on top of our branches (which are now merged).

Alternatively we could queue this up as a bugfix: there is no
harm in selecting a symbol that has been removed, so we don't
have a strict dependency that requries it this one to go
in before the removal.

	Arnd
Olof Johansson Aug. 2, 2016, 6:55 p.m. UTC | #2
On Tue, Aug 02, 2016 at 09:47:50AM +0200, Arnd Bergmann wrote:
> On Tuesday, August 2, 2016 9:44:33 AM CEST Linus Walleij wrote:
> > A new user of the Kconfig selection of ARCH_REQUIRE_GPIOLIB
> > has appeared. Replace with just selecting GPIOLIB.
> > 
> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> > ---
> > ARM SoC folks: I plan to send this directly in this merge
> > window to get rid of the old Kconfig symbols, please ACK
> > it.
> 
> Acked-by: Arnd Bergmann <arnd@arndb.de>
> 
> Note that sending this one along with your patches requires
> you to base on top of our branches (which are now merged).
> 
> Alternatively we could queue this up as a bugfix: there is no
> harm in selecting a symbol that has been removed, so we don't
> have a strict dependency that requries it this one to go
> in before the removal.

We'll need to start building a fixes branch shortly so we can include
this with it.


-Olof
Olof Johansson Aug. 2, 2016, 8:13 p.m. UTC | #3
On Tue, Aug 2, 2016 at 11:55 AM, Olof Johansson <olof@lixom.net> wrote:
> On Tue, Aug 02, 2016 at 09:47:50AM +0200, Arnd Bergmann wrote:
>> On Tuesday, August 2, 2016 9:44:33 AM CEST Linus Walleij wrote:
>> > A new user of the Kconfig selection of ARCH_REQUIRE_GPIOLIB
>> > has appeared. Replace with just selecting GPIOLIB.
>> >
>> > Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
>> > ---
>> > ARM SoC folks: I plan to send this directly in this merge
>> > window to get rid of the old Kconfig symbols, please ACK
>> > it.
>>
>> Acked-by: Arnd Bergmann <arnd@arndb.de>
>>
>> Note that sending this one along with your patches requires
>> you to base on top of our branches (which are now merged).
>>
>> Alternatively we could queue this up as a bugfix: there is no
>> harm in selecting a symbol that has been removed, so we don't
>> have a strict dependency that requries it this one to go
>> in before the removal.
>
> We'll need to start building a fixes branch shortly so we can include
> this with it.

Applied to fixes.

-Olof
diff mbox

Patch

diff --git a/arch/arm/mach-clps711x/Kconfig b/arch/arm/mach-clps711x/Kconfig
index dc7c6edeab39..61284b9389cf 100644
--- a/arch/arm/mach-clps711x/Kconfig
+++ b/arch/arm/mach-clps711x/Kconfig
@@ -1,13 +1,13 @@ 
 menuconfig ARCH_CLPS711X
 	bool "Cirrus Logic EP721x/EP731x-based"
 	depends on ARCH_MULTI_V4T
-	select ARCH_REQUIRE_GPIOLIB
 	select AUTO_ZRELADDR
 	select CLKSRC_OF
 	select CLPS711X_TIMER
 	select COMMON_CLK
 	select CPU_ARM720T
 	select GENERIC_CLOCKEVENTS
+	select GPIOLIB
 	select MFD_SYSCON
 	select OF_IRQ
 	select USE_OF