From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: RE: GENERIC_GPIO considered deprecated Date: Mon, 08 Apr 2013 16:31:28 +0900 Message-ID: <2c9501ce342b$1535d290$3fa177b0$%kim@samsung.com> References: <51576214.8010307@openwide.fr> <26e401ce30cc$602e45f0$208ad1d0$%kim@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Return-path: Received: from mailout3.samsung.com ([203.254.224.33]:60538 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932739Ab3DHHba convert rfc822-to-8bit (ORCPT ); Mon, 8 Apr 2013 03:31:30 -0400 In-reply-to: Content-language: ko Sender: linux-next-owner@vger.kernel.org List-ID: To: 'Alexandre Courbot' Cc: 'Romain Naour' , 'linux-next' , 'Grant Likely' , 'Linus Walleij' , 'Tomasz Figa' , =?utf-8?Q?'Heiko_St=C3=BCbner'?= , linux-samsung-soc@vger.kernel.org Alexandre Courbot wrote: > > On Wed, Apr 3, 2013 at 5:35 PM, Kukjin Kim 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 > > 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 > > Cc: Romain Naour > > Signed-off-by: Kukjin Kim > > --- > > 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. I'm not sure but I think, arm-soc tree should be merged into mainline before others... > Can you put it into your tree for 3.10? > I did, so it should be fine. Thanks. - Kukjin