Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1405324 > unrolled thread

[PATCH 0/3] ARM: keystone-k2l: Enable pinctrl support

Started byKeerthy <j-keerthy@ti.com>
First post2016-05-23 14:40 +0200
Last post2016-05-24 05:50 +0200
Articles 9 — 4 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/3] ARM: keystone-k2l: Enable pinctrl support Keerthy <j-keerthy@ti.com> - 2016-05-23 14:40 +0200
    [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config Keerthy <j-keerthy@ti.com> - 2016-05-23 14:40 +0200
      Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config Lokesh Vutla <lokeshvutla@ti.com> - 2016-05-24 05:50 +0200
        Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config Keerthy <a0393675@ti.com> - 2016-05-24 06:00 +0200
          Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config Santosh Shilimkar <santosh.shilimkar@oracle.com> - 2016-05-24 18:40 +0200
            Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config Keerthy <a0393675@ti.com> - 2016-05-25 05:30 +0200
    [PATCH 1/3] ARM: dts: keystone-k2l: Add pinctrl node Keerthy <j-keerthy@ti.com> - 2016-05-23 14:40 +0200
    [PATCH 2/3] arm: mach-keystone: Enable PINCTRL config Keerthy <j-keerthy@ti.com> - 2016-05-23 14:40 +0200
      Re: [PATCH 2/3] arm: mach-keystone: Enable PINCTRL config Lokesh Vutla <lokeshvutla@ti.com> - 2016-05-24 05:50 +0200

#1405324 — [PATCH 0/3] ARM: keystone-k2l: Enable pinctrl support

FromKeerthy <j-keerthy@ti.com>
Date2016-05-23 14:40 +0200
Subject[PATCH 0/3] ARM: keystone-k2l: Enable pinctrl support
Message-ID<rBXjk-5NQ-7@gated-at.bofh.it>
The series enables pinctrl support for k2l devices.
Tested on keystone-k2l-evm board.

Keerthy (3):
  ARM: dts: keystone-k2l: Add pinctrl node
  arm: mach-keystone: Enable PINCTRL config
  ARM: configs: keystone: Enable PINCTRL_SINGLE Config

 arch/arm/boot/dts/keystone-k2l.dtsi | 149 ++++++++++++++++++++++++++++++++++++
 arch/arm/configs/keystone_defconfig |   1 +
 arch/arm/mach-keystone/Kconfig      |   1 +
 3 files changed, 151 insertions(+)

-- 
1.9.1

[toc] | [next] | [standalone]

#1405327 — [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config

FromKeerthy <j-keerthy@ti.com>
Date2016-05-23 14:40 +0200
Subject[PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config
Message-ID<rBXjk-5NQ-15@gated-at.bofh.it>
In reply to#1405324
keystone-k2l devices use pinmux and are compliant with PINCTRL_SINGLE.
Hence enable the config option.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/configs/keystone_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/keystone_defconfig b/arch/arm/configs/keystone_defconfig
index faba04d..71b42e6 100644
--- a/arch/arm/configs/keystone_defconfig
+++ b/arch/arm/configs/keystone_defconfig
@@ -144,6 +144,7 @@ CONFIG_I2C_DAVINCI=y
 CONFIG_SPI=y
 CONFIG_SPI_DAVINCI=y
 CONFIG_SPI_SPIDEV=y
+CONFIG_PINCTRL_SINGLE=y
 CONFIG_GPIOLIB=y
 CONFIG_GPIO_SYSFS=y
 CONFIG_GPIO_DAVINCI=y
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1405813 — Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config

FromLokesh Vutla <lokeshvutla@ti.com>
Date2016-05-24 05:50 +0200
SubjectRe: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config
Message-ID<rCbvX-6io-1@gated-at.bofh.it>
In reply to#1405327

On Monday 23 May 2016 05:59 PM, Keerthy wrote:
> keystone-k2l devices use pinmux and are compliant with PINCTRL_SINGLE.
> Hence enable the config option.
> 
> Signed-off-by: Keerthy <j-keerthy@ti.com>

A similar patch[1] is already posted.

[1]https://patchwork.kernel.org/patch/8958091/

Thanks and regards,
Lokesh

> ---
>  arch/arm/configs/keystone_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/configs/keystone_defconfig b/arch/arm/configs/keystone_defconfig
> index faba04d..71b42e6 100644
> --- a/arch/arm/configs/keystone_defconfig
> +++ b/arch/arm/configs/keystone_defconfig
> @@ -144,6 +144,7 @@ CONFIG_I2C_DAVINCI=y
>  CONFIG_SPI=y
>  CONFIG_SPI_DAVINCI=y
>  CONFIG_SPI_SPIDEV=y
> +CONFIG_PINCTRL_SINGLE=y
>  CONFIG_GPIOLIB=y
>  CONFIG_GPIO_SYSFS=y
>  CONFIG_GPIO_DAVINCI=y
>

[toc] | [prev] | [next] | [standalone]

#1405818 — Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config

FromKeerthy <a0393675@ti.com>
Date2016-05-24 06:00 +0200
SubjectRe: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config
Message-ID<rCbFD-6ln-5@gated-at.bofh.it>
In reply to#1405813

On Tuesday 24 May 2016 09:07 AM, Lokesh Vutla wrote:
>
>
> On Monday 23 May 2016 05:59 PM, Keerthy wrote:
>> keystone-k2l devices use pinmux and are compliant with PINCTRL_SINGLE.
>> Hence enable the config option.
>>
>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>
> A similar patch[1] is already posted.
>
> [1]https://patchwork.kernel.org/patch/8958091/

Ah I had not seen them. If they are already reviewed and closer to be 
merged then Patch 2 and Patch 3 of this series can be dropped.

Regards,
Keerthy

>
> Thanks and regards,
> Lokesh
>
>> ---
>>   arch/arm/configs/keystone_defconfig | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/configs/keystone_defconfig b/arch/arm/configs/keystone_defconfig
>> index faba04d..71b42e6 100644
>> --- a/arch/arm/configs/keystone_defconfig
>> +++ b/arch/arm/configs/keystone_defconfig
>> @@ -144,6 +144,7 @@ CONFIG_I2C_DAVINCI=y
>>   CONFIG_SPI=y
>>   CONFIG_SPI_DAVINCI=y
>>   CONFIG_SPI_SPIDEV=y
>> +CONFIG_PINCTRL_SINGLE=y
>>   CONFIG_GPIOLIB=y
>>   CONFIG_GPIO_SYSFS=y
>>   CONFIG_GPIO_DAVINCI=y
>>

[toc] | [prev] | [next] | [standalone]

#1406279 — Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config

FromSantosh Shilimkar <santosh.shilimkar@oracle.com>
Date2016-05-24 18:40 +0200
SubjectRe: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config
Message-ID<rCnx8-5GI-23@gated-at.bofh.it>
In reply to#1405818
Hi Keerthy,

On 5/23/2016 8:56 PM, Keerthy wrote:
>
>
> On Tuesday 24 May 2016 09:07 AM, Lokesh Vutla wrote:
>>
>>
>> On Monday 23 May 2016 05:59 PM, Keerthy wrote:
>>> keystone-k2l devices use pinmux and are compliant with PINCTRL_SINGLE.
>>> Hence enable the config option.
>>>
>>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>>
>> A similar patch[1] is already posted.
>>
>> [1]https://patchwork.kernel.org/patch/8958091/
>
> Ah I had not seen them. If they are already reviewed and closer to be
> merged then Patch 2 and Patch 3 of this series can be dropped.
>
Once the 4.7-rc2 is out, please rebase these floating patches
against it and post a consolidated patches. I will line them
up for 4.8

Regards,
Santosh

[toc] | [prev] | [next] | [standalone]

#1406571 — Re: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config

FromKeerthy <a0393675@ti.com>
Date2016-05-25 05:30 +0200
SubjectRe: [PATCH 3/3] ARM: configs: keystone: Enable PINCTRL_SINGLE Config
Message-ID<rCxG9-4ar-1@gated-at.bofh.it>
In reply to#1406279
Hi Santosh,

On Tuesday 24 May 2016 09:59 PM, Santosh Shilimkar wrote:
> Hi Keerthy,
>
> On 5/23/2016 8:56 PM, Keerthy wrote:
>>
>>
>> On Tuesday 24 May 2016 09:07 AM, Lokesh Vutla wrote:
>>>
>>>
>>> On Monday 23 May 2016 05:59 PM, Keerthy wrote:
>>>> keystone-k2l devices use pinmux and are compliant with PINCTRL_SINGLE.
>>>> Hence enable the config option.
>>>>
>>>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>>>
>>> A similar patch[1] is already posted.
>>>
>>> [1]https://patchwork.kernel.org/patch/8958091/
>>
>> Ah I had not seen them. If they are already reviewed and closer to be
>> merged then Patch 2 and Patch 3 of this series can be dropped.
>>
> Once the 4.7-rc2 is out, please rebase these floating patches
> against it and post a consolidated patches. I will line them
> up for 4.8

Thanks for the quick response. I will wait for 4.7-rc2 and send the 
re-based series.

Regards,
Keerthy

>
> Regards,
> Santosh

[toc] | [prev] | [next] | [standalone]

#1405328 — [PATCH 1/3] ARM: dts: keystone-k2l: Add pinctrl node

FromKeerthy <j-keerthy@ti.com>
Date2016-05-23 14:40 +0200
Subject[PATCH 1/3] ARM: dts: keystone-k2l: Add pinctrl node
Message-ID<rBXjk-5NQ-25@gated-at.bofh.it>
In reply to#1405324
Add pinctrl node and populate the pinctrl registers with the default
values.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/boot/dts/keystone-k2l.dtsi | 149 ++++++++++++++++++++++++++++++++++++
 1 file changed, 149 insertions(+)

diff --git a/arch/arm/boot/dts/keystone-k2l.dtsi b/arch/arm/boot/dts/keystone-k2l.dtsi
index ff22ffc..2ee3d0a 100644
--- a/arch/arm/boot/dts/keystone-k2l.dtsi
+++ b/arch/arm/boot/dts/keystone-k2l.dtsi
@@ -54,6 +54,155 @@
 			interrupts = <GIC_SPI 435 IRQ_TYPE_EDGE_RISING>;
 		};
 
+		k2l_pmx: pinmux@02620690 {
+			compatible = "pinctrl-single";
+			reg = <0x02620690 0xc>;
+			#address-cells = <1>;
+			#size-cells = <0>;
+			pinctrl-single,bit-per-mux;
+			pinctrl-single,register-width = <32>;
+			pinctrl-single,function-mask = <0x1>;
+			status = "disabled";
+
+			uart3_emifa_pins: pinmux_uart3_emifa_pins {
+				pinctrl-single,bits = <
+					/* UART3_EMIFA_SEL */
+					0x0 0x0  0xc0
+				>;
+			};
+
+			uart2_emifa_pins: pinmux_uart2_emifa_pins {
+			pinctrl-single,bits = <
+					/* UART2_EMIFA_SEL */
+					0x0 0x0  0x30
+				>;
+			};
+
+			uart01_spi2_pins: pinmux_uart01_spi2_pins {
+				pinctrl-single,bits = <
+					/* UART01_SPI2_SEL */
+					0x0 0x0 0x4
+				>;
+			};
+
+			dfesync_rp1_pins: pinmux_dfesync_rp1_pins{
+				pinctrl-single,bits = <
+					/* DFESYNC_RP1_SEL */
+					0x0 0x0 0x2
+				>;
+			};
+
+			avsif_pins: pinmux_avsif_pins {
+				pinctrl-single,bits = <
+					/* AVSIF_SEL */
+					0x0 0x0 0x1
+				>;
+			};
+
+			gpio_emu_pins: pinmux_gpio_emu_pins {
+				pinctrl-single,bits = <
+				/*
+				 * GPIO_EMU_SEL[31]: 0-GPIO31, 1-EMU33
+				 * GPIO_EMU_SEL[30]: 0-GPIO30, 1-EMU32
+				 * GPIO_EMU_SEL[29]: 0-GPIO29, 1-EMU31
+				 * GPIO_EMU_SEL[28]: 0-GPIO28, 1-EMU30
+				 * GPIO_EMU_SEL[27]: 0-GPIO27, 1-EMU29
+				 * GPIO_EMU_SEL[26]: 0-GPIO26, 1-EMU28
+				 * GPIO_EMU_SEL[25]: 0-GPIO25, 1-EMU27
+				 * GPIO_EMU_SEL[24]: 0-GPIO24, 1-EMU26
+				 * GPIO_EMU_SEL[23]: 0-GPIO23, 1-EMU25
+				 * GPIO_EMU_SEL[22]: 0-GPIO22, 1-EMU24
+				 * GPIO_EMU_SEL[21]: 0-GPIO21, 1-EMU23
+				 * GPIO_EMU_SEL[20]: 0-GPIO20, 1-EMU22
+				 * GPIO_EMU_SEL[19]: 0-GPIO19, 1-EMU21
+				 * GPIO_EMU_SEL[18]: 0-GPIO18, 1-EMU20
+				 * GPIO_EMU_SEL[17]: 0-GPIO17, 1-EMU19
+				 */
+					0x4 0x0000 0xFFFE0000
+				>;
+			};
+
+			gpio_timio_pins: pinmux_gpio_timio_pins {
+				pinctrl-single,bits = <
+				/*
+				 * GPIO_TIMIO_SEL[15]: 0-GPIO15, 1-TIMO7
+				 * GPIO_TIMIO_SEL[14]: 0-GPIO14, 1-TIMO6
+				 * GPIO_TIMIO_SEL[13]: 0-GPIO13, 1-TIMO5
+				 * GPIO_TIMIO_SEL[12]: 0-GPIO12, 1-TIMO4
+				 * GPIO_TIMIO_SEL[11]: 0-GPIO11, 1-TIMO3
+				 * GPIO_TIMIO_SEL[10]: 0-GPIO10, 1-TIMO2
+				 * GPIO_TIMIO_SEL[9]: 0-GPIO9, 1-TIMI7
+				 * GPIO_TIMIO_SEL[8]: 0-GPIO8, 1-TIMI6
+				 * GPIO_TIMIO_SEL[7]: 0-GPIO7, 1-TIMI5
+				 * GPIO_TIMIO_SEL[6]: 0-GPIO6, 1-TIMI4
+				 * GPIO_TIMIO_SEL[5]: 0-GPIO5, 1-TIMI3
+				 * GPIO_TIMIO_SEL[4]: 0-GPIO4, 1-TIMI2
+				 */
+					0x4 0x0 0xFFF0
+				>;
+			};
+
+			gpio_spi2cs_pins: pinmux_gpio_spi2cs_pins {
+				pinctrl-single,bits = <
+				/*
+				 * GPIO_SPI2CS_SEL[3]: 0-GPIO3, 1-SPI2CS4
+				 * GPIO_SPI2CS_SEL[2]: 0-GPIO2, 1-SPI2CS3
+				 * GPIO_SPI2CS_SEL[1]: 0-GPIO1, 1-SPI2CS2
+				 * GPIO_SPI2CS_SEL[0]: 0-GPIO0, 1-SPI2CS1
+				 */
+					0x4 0x0 0xF
+				>;
+			};
+
+			gpio_dfeio_pins: pinmux_gpio_dfeio_pins {
+				pinctrl-single,bits = <
+				/*
+				 * GPIO_DFEIO_SEL[31]: 0-DFEIO17, 1-GPIO63
+				 * GPIO_DFEIO_SEL[30]: 0-DFEIO16, 1-GPIO62
+				 * GPIO_DFEIO_SEL[29]: 0-DFEIO15, 1-GPIO61
+				 * GPIO_DFEIO_SEL[28]: 0-DFEIO14, 1-GPIO60
+				 * GPIO_DFEIO_SEL[27]: 0-DFEIO13, 1-GPIO59
+				 * GPIO_DFEIO_SEL[26]: 0-DFEIO12, 1-GPIO58
+				 * GPIO_DFEIO_SEL[25]: 0-DFEIO11, 1-GPIO57
+				 * GPIO_DFEIO_SEL[24]: 0-DFEIO10, 1-GPIO56
+				 * GPIO_DFEIO_SEL[23]: 0-DFEIO9, 1-GPIO55
+				 * GPIO_DFEIO_SEL[22]: 0-DFEIO8, 1-GPIO54
+				 * GPIO_DFEIO_SEL[21]: 0-DFEIO7, 1-GPIO53
+				 * GPIO_DFEIO_SEL[20]: 0-DFEIO6, 1-GPIO52
+				 * GPIO_DFEIO_SEL[19]: 0-DFEIO5, 1-GPIO51
+				 * GPIO_DFEIO_SEL[18]: 0-DFEIO4, 1-GPIO50
+				 * GPIO_DFEIO_SEL[17]: 0-DFEIO3, 1-GPIO49
+				 * GPIO_DFEIO_SEL[16]: 0-DFEIO2, 1-GPIO48
+				 */
+					0x8 0x0 0xFFFF0000
+				>;
+			};
+
+			gpio_emifa_pins: pinmux_gpio_emifa_pins {
+				pinctrl-single,bits = <
+				/*
+				 * GPIO_EMIFA_SEL[15]: 0-EMIFA17, 1-GPIO47
+				 * GPIO_EMIFA_SEL[14]: 0-EMIFA16, 1-GPIO46
+				 * GPIO_EMIFA_SEL[13]: 0-EMIFA15, 1-GPIO45
+				 * GPIO_EMIFA_SEL[12]: 0-EMIFA14, 1-GPIO44
+				 * GPIO_EMIFA_SEL[11]: 0-EMIFA13, 1-GPIO43
+				 * GPIO_EMIFA_SEL[10]: 0-EMIFA10, 1-GPIO42
+				 * GPIO_EMIFA_SEL[9]: 0-EMIFA9, 1-GPIO41
+				 * GPIO_EMIFA_SEL[8]: 0-EMIFA8, 1-GPIO40
+				 * GPIO_EMIFA_SEL[7]: 0-EMIFA7, 1-GPIO39
+				 * GPIO_EMIFA_SEL[6]: 0-EMIFA6, 1-GPIO38
+				 * GPIO_EMIFA_SEL[5]: 0-EMIFA5, 1-GPIO37
+				 * GPIO_EMIFA_SEL[4]: 0-EMIFA4, 1-GPIO36
+				 * GPIO_EMIFA_SEL[3]: 0-EMIFA3, 1-GPIO35
+				 * GPIO_EMIFA_SEL[2]: 0-EMIFA2, 1-GPIO34
+				 * GPIO_EMIFA_SEL[1]: 0-EMIFA1, 1-GPIO33
+				 * GPIO_EMIFA_SEL[0]: 0-EMIFA0, 1-GPIO32
+				 */
+					0x8 0x0 0xFFFF
+				>;
+			};
+		};
+
 		dspgpio0: keystone_dsp_gpio@02620240 {
 			compatible = "ti,keystone-dsp-gpio";
 			gpio-controller;
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1405330 — [PATCH 2/3] arm: mach-keystone: Enable PINCTRL config

FromKeerthy <j-keerthy@ti.com>
Date2016-05-23 14:40 +0200
Subject[PATCH 2/3] arm: mach-keystone: Enable PINCTRL config
Message-ID<rBXjk-5NQ-19@gated-at.bofh.it>
In reply to#1405324
keystone-k2l uses pinmux and is compliant with PINCTRL_SINGLE
which depends on PINCTRL. Hence enable PINCTRL.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 arch/arm/mach-keystone/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-keystone/Kconfig b/arch/arm/mach-keystone/Kconfig
index ea955f6db..7a432ee 100644
--- a/arch/arm/mach-keystone/Kconfig
+++ b/arch/arm/mach-keystone/Kconfig
@@ -11,6 +11,7 @@ config ARCH_KEYSTONE
 	select ZONE_DMA if ARM_LPAE
 	select MIGHT_HAVE_PCI
 	select PCI_DOMAINS if PCI
+	select PINCTRL
 	help
 	  Support for boards based on the Texas Instruments Keystone family of
 	  SoCs.
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1405814 — Re: [PATCH 2/3] arm: mach-keystone: Enable PINCTRL config

FromLokesh Vutla <lokeshvutla@ti.com>
Date2016-05-24 05:50 +0200
SubjectRe: [PATCH 2/3] arm: mach-keystone: Enable PINCTRL config
Message-ID<rCbvX-6io-3@gated-at.bofh.it>
In reply to#1405330

On Monday 23 May 2016 05:59 PM, Keerthy wrote:
> keystone-k2l uses pinmux and is compliant with PINCTRL_SINGLE
> which depends on PINCTRL. Hence enable PINCTRL.
> 
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> ---
>  arch/arm/mach-keystone/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/mach-keystone/Kconfig b/arch/arm/mach-keystone/Kconfig
> index ea955f6db..7a432ee 100644
> --- a/arch/arm/mach-keystone/Kconfig
> +++ b/arch/arm/mach-keystone/Kconfig
> @@ -11,6 +11,7 @@ config ARCH_KEYSTONE
>  	select ZONE_DMA if ARM_LPAE
>  	select MIGHT_HAVE_PCI
>  	select PCI_DOMAINS if PCI
> +	select PINCTRL

I guess a similar patch[1] is already posted

[1] https://patchwork.kernel.org/patch/8958081/

Thanks and regards,
Lokesh

>  	help
>  	  Support for boards based on the Texas Instruments Keystone family of
>  	  SoCs.
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web