2013-03-05 18:24:33

by Thomas Abraham

[permalink] [raw]
Subject: Re: [PATCH 1/3] pinctrl: exynos: add exynos5250 SoC specific data

On 27 December 2012 22:28, Kukjin Kim <[email protected]> wrote:
> Thomas Abraham wrote:
>>
>> Hi Linus,
>>
>> On 14 December 2012 21:21, Linus Walleij <[email protected]> wrote:
>> > On Thu, Dec 13, 2012 at 12:54 PM, Thomas Abraham
>> > <[email protected]> wrote:
>> >
>> >> Add Samsung Exynos5250 SoC specific data to enable pinctrl support for
>> >> all platforms based on Exynos5250.
>> >>
>> >> Signed-off-by: Thomas Abraham <[email protected]>
>> >
>> > Acked-by: Linus Walleij <[email protected]>
>> >
>> > For this series, I guess you'll fix it through the Samsung tree?
>>
>> Thanks for your ack. Since there are patches to be prepared for
>> migrating the rest of Exynos5250 related code to the pinctrl
>> framework, it would be easier if this series goes through the Samsung
>> tree.
>>
> I applied this series, thanks.

This series has not made it into 3.9-rc1. So I will rebase this series
to 3.9-rc1 and post again.

Thanks,
Thomas.

>
> And I think, would be clear if the config could be changed like following.
>
> 8<----------------------------------------------------------------------
> From: Kukjin Kim <[email protected]>
> Subject: [PATCH] pinctrl: exynos: change PINCTRL_EXYNOS option
>
> Since pinctrl-exynos can support exynos4 and exynos5 so changed
> the option name to PINCTRL_EXYNOS for more clarity.
>
> Cc: Thomas Abraham <[email protected]>
> Cc: Linus Walleij <[email protected]>
> Cc: Grant Likely <[email protected]>
> Signed-off-by: Kukjin Kim <[email protected]>
> ---
> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> index 91d5b6f..e1f63e7 100644
> --- a/arch/arm/mach-exynos/Kconfig
> +++ b/arch/arm/mach-exynos/Kconfig
> @@ -412,7 +412,7 @@ config MACH_EXYNOS4_DT
> select CPU_EXYNOS4210
> select HAVE_SAMSUNG_KEYPAD if INPUT_KEYBOARD
> select PINCTRL
> - select PINCTRL_EXYNOS4
> + select PINCTRL_EXYNOS
> select USE_OF
> help
> Machine support for Samsung Exynos4 machine with device tree
> enabled.
> diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig
> index c31aeb0..f59feab 100644
> --- a/drivers/pinctrl/Kconfig
> +++ b/drivers/pinctrl/Kconfig
> @@ -185,8 +185,8 @@ config PINCTRL_SAMSUNG
> select PINMUX
> select PINCONF
>
> -config PINCTRL_EXYNOS4
> - bool "Pinctrl driver data for Exynos4 SoC"
> +config PINCTRL_EXYNOS
> + bool "Pinctrl driver data for Samsung EXYNOS SoCs"
> depends on OF && GPIOLIB
> select PINCTRL_SAMSUNG
>
> diff --git a/drivers/pinctrl/Makefile b/drivers/pinctrl/Makefile
> index fc4606f..6e87e52 100644
> --- a/drivers/pinctrl/Makefile
> +++ b/drivers/pinctrl/Makefile
> @@ -36,7 +36,7 @@ obj-$(CONFIG_PINCTRL_TEGRA30) += pinctrl-tegra30.o
> obj-$(CONFIG_PINCTRL_U300) += pinctrl-u300.o
> obj-$(CONFIG_PINCTRL_COH901) += pinctrl-coh901.o
> obj-$(CONFIG_PINCTRL_SAMSUNG) += pinctrl-samsung.o
> -obj-$(CONFIG_PINCTRL_EXYNOS4) += pinctrl-exynos.o
> +obj-$(CONFIG_PINCTRL_EXYNOS) += pinctrl-exynos.o
> obj-$(CONFIG_PINCTRL_EXYNOS5440) += pinctrl-exynos5440.o
> obj-$(CONFIG_PINCTRL_XWAY) += pinctrl-xway.o
> obj-$(CONFIG_PINCTRL_LANTIQ) += pinctrl-lantiq.o
> 8<----------------------------------------------------------------------
>
> - Kukjin
>