diff mbox

ARM: s3c64xx: Select GPIO_GENERIC_PLATFORM for Cragganmore

Message ID 1453400718-26720-1-git-send-email-broonie@kernel.org (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Mark Brown Jan. 21, 2016, 6:25 p.m. UTC
The Cragganmore machine driver provides platform data for the generic GPIO
platform driver which uses structures only defined if the driver is
enabled. Since there is little reason to use the platform without the GPIOs
add a select to avoid build failures.

Signed-off-by: Mark Brown <broonie@kernel.org>
---
 arch/arm/mach-s3c64xx/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski Jan. 22, 2016, 12:21 a.m. UTC | #1
On 22.01.2016 03:25, Mark Brown wrote:
> The Cragganmore machine driver provides platform data for the generic GPIO
> platform driver which uses structures only defined if the driver is
> enabled. Since there is little reason to use the platform without the GPIOs
> add a select to avoid build failures.
> 
> Signed-off-by: Mark Brown <broonie@kernel.org>
> ---
>  arch/arm/mach-s3c64xx/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/mach-s3c64xx/Kconfig b/arch/arm/mach-s3c64xx/Kconfig
> index 7c0c420c3016..98d2aa9393f1 100644
> --- a/arch/arm/mach-s3c64xx/Kconfig
> +++ b/arch/arm/mach-s3c64xx/Kconfig
> @@ -310,6 +310,7 @@ config MACH_WLF_CRAGG_6410
>  	depends on I2C=y
>  	select CPU_S3C6410
>  	select LEDS_GPIO_REGISTER
> +	select GPIO_GENERIC_PLATFORM
>  	select S3C64XX_DEV_SPI0
>  	select S3C64XX_SETUP_FB_24BPP
>  	select S3C64XX_SETUP_I2C1
> 


Thanks for the patch!
This was already fixed by Arnd:
http://marc.info/?l=linux-gpio&m=145237423205247&w=2

Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Charles Keepax Jan. 26, 2016, 3:10 p.m. UTC | #2
On Thu, Jan 21, 2016 at 06:25:18PM +0000, Mark Brown wrote:
> The Cragganmore machine driver provides platform data for the generic GPIO
> platform driver which uses structures only defined if the driver is
> enabled. Since there is little reason to use the platform without the GPIOs
> add a select to avoid build failures.
> 
> Signed-off-by: Mark Brown <broonie@kernel.org>
> ---

All looks good to me and doesn't cause any issues on my Speyside.

Tested-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>

Thanks,
Charles
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/mach-s3c64xx/Kconfig b/arch/arm/mach-s3c64xx/Kconfig
index 7c0c420c3016..98d2aa9393f1 100644
--- a/arch/arm/mach-s3c64xx/Kconfig
+++ b/arch/arm/mach-s3c64xx/Kconfig
@@ -310,6 +310,7 @@  config MACH_WLF_CRAGG_6410
 	depends on I2C=y
 	select CPU_S3C6410
 	select LEDS_GPIO_REGISTER
+	select GPIO_GENERIC_PLATFORM
 	select S3C64XX_DEV_SPI0
 	select S3C64XX_SETUP_FB_24BPP
 	select S3C64XX_SETUP_I2C1