On Wed, Apr 3, 2013 at 5:35 PM, Kukjin Kim <kgene.kim@xxxxxxxxxxx> wrote: >> could you amend the patches that adds them such as they get changed >> into "select ARCH_REQUIRE_GPIOLIB" instead? You can grep for "select > > I can do it for my tree but the branch already included in arm-soc tree so I think, it should be fixed with another patch. And > >> GENERIC_GPIO" in arch/arm to find the offending lines. We are removing >> GENERIC_GPIO and this work cannot be merged until you do this since it >> would break ARM builds. Thanks! >> > So how about following? If you are OK, let me take into samsung tree. > > --------8<----------------8<-------- > From: Kukjin Kim <kgene.kim@xxxxxxxxxxx> > Subject: [PATCH] ARM: SAMSUNG: change GENERIC_GPIO to ARCH_REQUIRE_GPIOLIB > > When I applied regarding samsung-time patches, the "select GENERIC_GPIO" > has been added wrong, so this patch fixes that. > And since the GENERIC_GPIO in arch/arm/ will be gone away, this adds > ARCH_REQUIRE_GPIOLIB for S3C24XX and S5PC100 instead. > > Reported-by: Alexandre Courbot <gnurou@xxxxxxxxx> > Cc: Romain Naour <romain.naour@xxxxxxxxxxx> > Signed-off-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx> > --- > arch/arm/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 46fcfa8..a239c7e 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -770,10 +770,10 @@ config ARCH_SA1100 > config ARCH_S3C24XX > bool "Samsung S3C24XX SoCs" > select ARCH_HAS_CPUFREQ > + select ARCH_REQUIRE_GPIOLIB > select CLKDEV_LOOKUP > select CLKSRC_MMIO > select GENERIC_CLOCKEVENTS > - select GENERIC_GPIO > select HAVE_CLK > select HAVE_S3C2410_I2C if I2C > select HAVE_S3C2410_WATCHDOG if WATCHDOG > @@ -828,11 +828,11 @@ config ARCH_S5P64X0 > > config ARCH_S5PC100 > bool "Samsung S5PC100" > + select ARCH_REQUIRE_GPIOLIB > select CLKDEV_LOOKUP > select CLKSRC_MMIO > select CPU_V7 > select GENERIC_CLOCKEVENTS > - select GENERIC_GPIO > select HAVE_CLK > select HAVE_S3C2410_I2C if I2C > select HAVE_S3C2410_WATCHDOG if WATCHDOG > -- > 1.7.10.4 Should do the trick, if we can make sure that your tree is merged prior to my patches. Can you put it into your tree for 3.10? Thanks! Alex. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html