From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751989AbbLAAe0 (ORCPT ); Mon, 30 Nov 2015 19:34:26 -0500 Received: from mailout3.w1.samsung.com ([210.118.77.13]:12157 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751710AbbLAAeY (ORCPT ); Mon, 30 Nov 2015 19:34:24 -0500 X-AuditID: cbfec7f4-f79026d00000418a-43-565ceb0da1d6 Subject: Re: [PATCH 03/10] gpio: samsung: move gpio-samsung driver back to platform code To: Arnd Bergmann , linux-arm-kernel@lists.infradead.org References: <1448467615-447097-1-git-send-email-arnd@arndb.de> <1448467615-447097-4-git-send-email-arnd@arndb.de> Cc: linux-kernel@vger.kernel.org, Kukjin Kim , Mark Brown , Vasily Khoruzhick , Charles Keepax , Tomasz Figa , Ben Dooks , linux-samsung-soc@vger.kernel.org From: Krzysztof Kozlowski X-Enigmail-Draft-Status: N1110 Message-id: <565CEB0A.4090107@samsung.com> Date: Tue, 01 Dec 2015 09:34:18 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-version: 1.0 In-reply-to: <1448467615-447097-4-git-send-email-arnd@arndb.de> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFuplkeLIzCtJLcpLzFFi42I5/e/4FV3e1zFhBq9bdCxajz1jtPg76Ri7 xYNbrUwWUx8+YbP4N+UGu8XrF4YW/Y9fM1tsenyN1eLyrjlsFjPO72Oy+Nz6j82B2+P3r0mM HrMbLrJ4PHs5mclj56y77B6bVnWyeWxeUu/xcuJvNo++LasYPT5vkgvgjOKySUnNySxLLdK3 S+DKOLF0MnvBNcGK3olfGBsYb/N2MXJySAiYSFw4t4cVwhaTuHBvPVsXIxeHkMBSRonHey9C OU8ZJZb0L2cHqRIWiJI4tucEI4gtIuAusW76PLBuIYFiiasrr7CDNDALbGKS2LrxPxtIgk3A WGLz8iVsECvkJHq7J7GA2LwCWhL/9u0AG8QioCrx79puZhBbVCBCYuKEBlaIGkGJH5PvgdVz CthLXN1/DqieA2iBnsT9i1ogYWYBeYnNa94yT2AUnIWkYxZC1SwkVQsYmVcxiqaWJhcUJ6Xn GuoVJ+YWl+al6yXn525ihETSlx2Mi49ZHWIU4GBU4uGVXBsTJsSaWFZcmXuIUYKDWUmEV+UJ UIg3JbGyKrUoP76oNCe1+BCjNAeLkjjv3F3vQ4QE0hNLUrNTUwtSi2CyTBycUg2MBV7iLKkr u6J8Xv59+ajAY6vsNcvWHW8ux775nchmrGB/Q7Ltt+3pTcf/9cu+UHxnd/DUft4Tm42SW1ZL 3Vmd8pq/mOFb6Gyj2bInjv0szJl+10Dz+67DPksfn8n512sW1cCbmbSGtdPxUniG9ZsFwl7d s4OtsmcI+RWeuxi7TkzY/fIGfdt/SizFGYmGWsxFxYkANFt4cKACAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 26.11.2015 01:06, Arnd Bergmann wrote: > The gpio-samsung driver is special in the sense that it > interacts directly in multiple ways with the legacy platform > code for the s3c24xx and s3c64xx platforms. In contrast, > all devicetree based machines for Samsung, including the > ones on those two SoC families use a different driver. > > The header files that define the interface between the platform > code and the gpio driver are not visible when building a > kernel for ARCH_MULTIPLATFORM, which prevents us from > turning on this option for s3c64xx. > > To work around this, we now move the driver back into platform > code, from where it was originally moved to as part of commit > 1b39d5f2cc5c28 ("gpio/samsung: gpio-samsung.c to support > Samsung GPIOs"). > > The long-term plan for this driver would be to remove it > entirely, after all Samsung machines have been converted > over to boot from DT, but there is currently no timeline > for when that might happen. > > Signed-off-by: Arnd Bergmann > --- > arch/arm/plat-samsung/Kconfig | 3 +++ > arch/arm/plat-samsung/Makefile | 2 ++ > {drivers/gpio => arch/arm/plat-samsung}/gpio-samsung.c | 0 > drivers/gpio/Kconfig | 7 ------- > drivers/gpio/Makefile | 1 - > 5 files changed, 5 insertions(+), 8 deletions(-) > rename {drivers/gpio => arch/arm/plat-samsung}/gpio-samsung.c (100%) > > diff --git a/arch/arm/plat-samsung/Kconfig b/arch/arm/plat-samsung/Kconfig > index 57729b915003..83697414bfa0 100644 > --- a/arch/arm/plat-samsung/Kconfig > +++ b/arch/arm/plat-samsung/Kconfig > @@ -294,6 +294,9 @@ config SAMSUNG_WDT_RESET > Compile support for system restart by triggering watchdog reset. > Used on SoCs that do not provide dedicated reset control. > > +config GPIO_SAMSUNG > + def_bool SAMSUNG_ATAGS > + I think this should be put inside "if SAMSUNG_ATAGS" section, just before SAMSUNG_PM_GPIO (with default=y?). It makes more consistent with rest of the options depending on ATAGS. Rest seems fine: Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof