diff mbox

[2/2] ARM: EXYNOS: Enable PINCTRL config option for Exynos4 device tree enabled platform

Message ID 1345061442-32378-3-git-send-email-thomas.abraham@linaro.org
State New
Headers show

Commit Message

thomas.abraham@linaro.org Aug. 15, 2012, 8:10 p.m. UTC
This enables support for Samsung and Exynos4 pinctrl driver for device
tree enabled Exynos4 platforms.

Cc: Kukjin Kim <kgene.kim@samsung.com>
Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org>
---
 arch/arm/mach-exynos/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

Linus Walleij Aug. 21, 2012, 12:31 p.m. UTC | #1
On Wed, Aug 15, 2012 at 10:10 PM, Thomas Abraham
<thomas.abraham@linaro.org> wrote:

> This enables support for Samsung and Exynos4 pinctrl driver for device
> tree enabled Exynos4 platforms.
>
> Cc: Kukjin Kim <kgene.kim@samsung.com>
> Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org>
> ---
>  arch/arm/mach-exynos/Kconfig |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
> index b5b4c8c..ad96c59 100644
> --- a/arch/arm/mach-exynos/Kconfig
> +++ b/arch/arm/mach-exynos/Kconfig
> @@ -405,6 +405,7 @@ config MACH_EXYNOS4_DT
>         select USE_OF
>         select ARM_AMBA
>         select HAVE_SAMSUNG_KEYPAD if INPUT_KEYBOARD
> +       select PINCTRL

I don't think this works, since PINCTRL_SAMSUNG is not default "y"
(nor should it be).

For the platforms I converted I had to select both the subsystem and
the indivudual driver, as is already done for some GPIO controllers
for example:

select PINCTRL
select PINCTRL_SAMSUNG

Yours,
Linus Walleij
thomas.abraham@linaro.org Aug. 22, 2012, 4:58 a.m. UTC | #2
On 21 August 2012 18:01, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Wed, Aug 15, 2012 at 10:10 PM, Thomas Abraham
> <thomas.abraham@linaro.org> wrote:
>
>> This enables support for Samsung and Exynos4 pinctrl driver for device
>> tree enabled Exynos4 platforms.
>>
>> Cc: Kukjin Kim <kgene.kim@samsung.com>
>> Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org>
>> ---
>>  arch/arm/mach-exynos/Kconfig |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
>> index b5b4c8c..ad96c59 100644
>> --- a/arch/arm/mach-exynos/Kconfig
>> +++ b/arch/arm/mach-exynos/Kconfig
>> @@ -405,6 +405,7 @@ config MACH_EXYNOS4_DT
>>         select USE_OF
>>         select ARM_AMBA
>>         select HAVE_SAMSUNG_KEYPAD if INPUT_KEYBOARD
>> +       select PINCTRL
>
> I don't think this works, since PINCTRL_SAMSUNG is not default "y"
> (nor should it be).

I think I got this wrong. PINCTRL_SAMSUNG depends of OF, which is not
correct. The correct way would be select PINCTRL_SAMSUNG from
PINCTRL_EXYNOS4. I will fix this.

>
> For the platforms I converted I had to select both the subsystem and
> the indivudual driver, as is already done for some GPIO controllers
> for example:
>
> select PINCTRL
> select PINCTRL_SAMSUNG
>
> Yours,
> Linus Walleij

Thanks,
Thomas.
diff mbox

Patch

diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
index b5b4c8c..ad96c59 100644
--- a/arch/arm/mach-exynos/Kconfig
+++ b/arch/arm/mach-exynos/Kconfig
@@ -405,6 +405,7 @@  config MACH_EXYNOS4_DT
 	select USE_OF
 	select ARM_AMBA
 	select HAVE_SAMSUNG_KEYPAD if INPUT_KEYBOARD
+	select PINCTRL
 	help
 	  Machine support for Samsung Exynos4 machine with device tree enabled.
 	  Select this if a fdt blob is available for the Exynos4 SoC based board.