[PATCH 0/5] Add more device nodes for am43x-epos-evm

[PATCH 0/5] Add more device nodes for am43x-epos-evm

[Sourav Poddar]

The patch series adds support for enabling pwm backlight, i2c2, spi and
matrix gpio keys on am43x-gp-evm.

Done on top of 3.13-rc1 + tero clock series(1)

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

Tested on am43x-gp-evm.

There is a some bug while using regulators through backlight
driver on 3.13-rc1. So, tested pwm part with this temporary patch[2].

[2]: http://www.spinics.net/lists/arm-kernel/msg288215.html

Darren Etheridge (1):
  pinctrl: am43xx: dt-bindings: add MUX_MODE8

Sourav Poddar (4):
  arm: dts: am4372: Add pwm-cellsproperty for ecap device.
  arm: dts: am43x-epos-evm: Add I2C data.
  arm: dts: am43x-epos-evm: Add SPI data.
  ARM: dts: am43x-epos-evm: Add pwm backlight support.

 arch/arm/boot/dts/am4372.dtsi        |    9 +++++
 arch/arm/boot/dts/am43x-epos-evm.dts |   67 ++++++++++++++++++++++++++++++++++
 include/dt-bindings/pinctrl/am43xx.h |    1 +
 3 files changed, 77 insertions(+), 0 deletions(-)

[PATCH 1/5] arm: dts: am4372: Add pwm-cells property for ecap device.

[Sourav Poddar]

Add "pwm-cells" node for all ecap, ehrpwm in am43x dtsi files.

Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
---
 arch/arm/boot/dts/am4372.dtsi |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
index 6ccf5c2..ea55a4e 100644
--- a/arch/arm/boot/dts/am4372.dtsi
+++ b/arch/arm/boot/dts/am4372.dtsi
@@ -495,6 +495,7 @@
 
 			ecap0: ecap@48300100 {
 				compatible = "ti,am4372-ecap","ti,am33xx-ecap";
+				#pwm-cells = <3>;
 				reg = <0x48300100 0x80>;
 				ti,hwmods = "ecap0";
 				status = "disabled";
@@ -502,6 +503,7 @@
 
 			ehrpwm0: ehrpwm@48300200 {
 				compatible = "ti,am4372-ehrpwm","ti,am33xx-ehrpwm";
+				#pwm-cells = <3>;
 				reg = <0x48300200 0x80>;
 				ti,hwmods = "ehrpwm0";
 				status = "disabled";
@@ -519,6 +521,7 @@
 
 			ecap1: ecap@48302100 {
 				compatible = "ti,am4372-ecap","ti,am33xx-ecap";
+				#pwm-cells = <3>;
 				reg = <0x48302100 0x80>;
 				ti,hwmods = "ecap1";
 				status = "disabled";
@@ -526,6 +529,7 @@
 
 			ehrpwm1: ehrpwm@48302200 {
 				compatible = "ti,am4372-ehrpwm","ti,am33xx-ehrpwm";
+				#pwm-cells = <3>;
 				reg = <0x48302200 0x80>;
 				ti,hwmods = "ehrpwm1";
 				status = "disabled";
@@ -543,6 +547,7 @@
 
 			ecap2: ecap@48304100 {
 				compatible = "ti,am4372-ecap","ti,am33xx-ecap";
+				#pwm-cells = <3>;
 				reg = <0x48304100 0x80>;
 				ti,hwmods = "ecap2";
 				status = "disabled";
@@ -550,6 +555,7 @@
 
 			ehrpwm2: ehrpwm@48304200 {
 				compatible = "ti,am4372-ehrpwm","ti,am33xx-ehrpwm";
+				#pwm-cells = <3>;
 				reg = <0x48304200 0x80>;
 				ti,hwmods = "ehrpwm2";
 				status = "disabled";
@@ -567,6 +573,7 @@
 
 			ehrpwm3: ehrpwm@48306200 {
 				compatible = "ti,am4372-ehrpwm","ti,am33xx-ehrpwm";
+				#pwm-cells = <3>;
 				reg = <0x48306200 0x80>;
 				ti,hwmods = "ehrpwm3";
 				status = "disabled";
@@ -584,6 +591,7 @@
 
 			ehrpwm4: ehrpwm@48308200 {
 				compatible = "ti,am4372-ehrpwm","ti,am33xx-ehrpwm";
+				#pwm-cells = <3>;
 				reg = <0x48308200 0x80>;
 				ti,hwmods = "ehrpwm4";
 				status = "disabled";
@@ -601,6 +609,7 @@
 
 			ehrpwm5: ehrpwm@4830a200 {
 				compatible = "ti,am4372-ehrpwm","ti,am33xx-ehrpwm";
+				#pwm-cells = <3>;
 				reg = <0x4830a200 0x80>;
 				ti,hwmods = "ehrpwm5";
 				status = "disabled";
-- 
1.7.1

[PATCH 2/5] ARM: dts: am43x-epos-evm: Add pwm backlight support.

[Sourav Poddar]

From: Sourav Poddar Date: Sat, 5 Oct 2013 05:19:46 +0530 <sourav.poddar@ti.com>

Add pwm backlight support for am43x epos evm.

Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
---
 arch/arm/boot/dts/am43x-epos-evm.dts |   24 ++++++++++++++++++++++++
 1 files changed, 24 insertions(+), 0 deletions(-)

diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
index a037a92..5521999 100644
--- a/arch/arm/boot/dts/am43x-epos-evm.dts
+++ b/arch/arm/boot/dts/am43x-epos-evm.dts
@@ -13,6 +13,7 @@
 #include "am4372.dtsi"
 #include <dt-bindings/pinctrl/am43xx.h>
 #include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/pwm/pwm.h>
 
 / {
 	model = "TI AM43x EPOS EVM";
@@ -90,6 +91,12 @@
 				0x9c (PIN_INPUT_PULLUP | MUX_MODE3)
 			>;
 		};
+
+		ecap0_pins: backlight_pins {
+			pinctrl-single,pins = <
+				0x164 MUX_MODE0         /* eCAP0_in_PWM0_out.eCAP0_in_PWM0_out MODE0 */
+			>;
+		};
 	};
 
 	matrix_keypad: matrix_keypad@0 {
@@ -124,6 +131,13 @@
 				0x0203006c	/* DOWN */
 				0x03030069>;	/* LEFT */
 		};
+
+	backlight {
+		compatible = "pwm-backlight";
+		pwms = <&ecap0 0 50000 PWM_POLARITY_INVERTED>;
+		brightness-levels = <0 51 53 56 62 75 101 152 255>;
+		default-brightness-level = <8>;
+	};
 };
 
 &mmc1 {
@@ -212,3 +226,13 @@
 		spi-rx-bus-width = <4>;
 	};
 };
+
+&epwmss0 {
+	status = "okay";
+};
+
+&ecap0 {
+		status = "okay";
+		pinctrl-names = "default";
+		pinctrl-0 = <&ecap0_pins>;
+};
-- 
1.7.1

[PATCH 3/5] pinctrl: am43xx: dt-bindings: add MUX_MODE8

[Sourav Poddar]

From: Darren Etheridge <detheridge@ti.com>

AM43xx devices have an extra MUX_MODE for certain pins.
Updating dt include to have MUX_MODE8 which maps to 0x8.

Signed-off-by: Darren Etheridge <detheridge@ti.com>
Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
---
 include/dt-bindings/pinctrl/am43xx.h |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/dt-bindings/pinctrl/am43xx.h b/include/dt-bindings/pinctrl/am43xx.h
index eb6c366..9c2e4f8 100644
--- a/include/dt-bindings/pinctrl/am43xx.h
+++ b/include/dt-bindings/pinctrl/am43xx.h
@@ -13,6 +13,7 @@
 #define MUX_MODE5	5
 #define MUX_MODE6	6
 #define MUX_MODE7	7
+#define MUX_MODE8	8
 
 #define PULL_DISABLE		(1 << 16)
 #define PULL_UP			(1 << 17)
-- 
1.7.1

[PATCH 4/5] arm: dts: am43x-epos-evm: Add I2C2 data.

[Sourav Poddar]

Add I2C2 dts data.

Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
---
 arch/arm/boot/dts/am43x-epos-evm.dts |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
index 5521999..7a504d1 100644
--- a/arch/arm/boot/dts/am43x-epos-evm.dts
+++ b/arch/arm/boot/dts/am43x-epos-evm.dts
@@ -97,6 +97,13 @@
 				0x164 MUX_MODE0         /* eCAP0_in_PWM0_out.eCAP0_in_PWM0_out MODE0 */
 			>;
 		};
+
+		i2c2_pins: pinmux_i2c2_pins {
+			pinctrl-single,pins = <
+				0x1c0 (PIN_INPUT_PULLUP | SLEWCTRL_FAST | MUX_MODE8)    /* i2c2_sda.i2c2_sda */
+				0x1c4 (PIN_INPUT_PULLUP | SLEWCTRL_FAST | MUX_MODE8)    /* i2c2_scl.i2c2_scl */
+			>;
+		};
 	};
 
 	matrix_keypad: matrix_keypad@0 {
@@ -194,6 +201,12 @@
 	};
 };
 
+&i2c2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&i2c2_pins>;
+	status = "okay";
+};
+
 &gpio0 {
 	status = "okay";
 };
-- 
1.7.1

[PATCH 5/5] arm: dts: am43x-epos-evm: Add SPI data.

[Sourav Poddar]

Add SPI dts data.

Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
---
 arch/arm/boot/dts/am43x-epos-evm.dts |   30 ++++++++++++++++++++++++++++++
 1 files changed, 30 insertions(+), 0 deletions(-)

diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
index 7a504d1..2ebcde6 100644
--- a/arch/arm/boot/dts/am43x-epos-evm.dts
+++ b/arch/arm/boot/dts/am43x-epos-evm.dts
@@ -104,6 +104,24 @@
 				0x1c4 (PIN_INPUT_PULLUP | SLEWCTRL_FAST | MUX_MODE8)    /* i2c2_scl.i2c2_scl */
 			>;
 		};
+
+		spi0_pins: pinmux_spi0_pins {
+			pinctrl-single,pins = <
+				0x150 (PIN_INPUT | MUX_MODE0)           /* spi0_clk.spi0_clk */
+				0x154 (PIN_OUTPUT | MUX_MODE0)           /* spi0_d0.spi0_d0 */
+				0x158 (PIN_INPUT | MUX_MODE0)           /* spi0_d1.spi0_d1 */
+				0x15c (PIN_OUTPUT | MUX_MODE0)          /* spi0_cs0.spi0_cs0 */
+			>;
+		};
+
+		spi1_pins: pinmux_spi1_pins {
+			pinctrl-single,pins = <
+				0x190 (PIN_INPUT | MUX_MODE3)           /* mcasp0_aclkx.spi1_clk */
+				0x194 (PIN_OUTPUT | MUX_MODE3)           /* mcasp0_fsx.spi1_d0 */
+				0x198 (PIN_INPUT | MUX_MODE3)           /* mcasp0_axr0.spi1_d1 */
+				0x19c (PIN_OUTPUT | MUX_MODE3)          /* mcasp0_ahclkr.spi1_cs0 */
+			>;
+		};
 	};
 
 	matrix_keypad: matrix_keypad@0 {
@@ -249,3 +267,15 @@
 		pinctrl-names = "default";
 		pinctrl-0 = <&ecap0_pins>;
 };
+
+&spi0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&spi0_pins>;
+	status = "okay";
+};
+
+&spi1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&spi1_pins>;
+	status = "okay";
+};
-- 
1.7.1

Re: [PATCH 0/5] Add more device nodes for am43x-epos-evm

[Sathya Prakash]

On Wednesday 27 November 2013 01:00 PM, Sourav Poddar wrote:
> The patch series adds support for enabling pwm backlight, i2c2, spi and
> matrix gpio keys on am43x-gp-evm.
Seems a typo - series is for epos evm.
>
> Done on top of 3.13-rc1 + tero clock series(1)
>
> [1]: https://patchwork.kernel.org/patch/3009541/
>
> Tested on am43x-gp-evm.
>
> There is a some bug while using regulators through backlight
> driver on 3.13-rc1. So, tested pwm part with this temporary patch[2].
>
> [2]: http://www.spinics.net/lists/arm-kernel/msg288215.html
>
> Darren Etheridge (1):
>    pinctrl: am43xx: dt-bindings: add MUX_MODE8
>
> Sourav Poddar (4):
>    arm: dts: am4372: Add pwm-cellsproperty for ecap device.
>    arm: dts: am43x-epos-evm: Add I2C data.
>    arm: dts: am43x-epos-evm: Add SPI data.
>    ARM: dts: am43x-epos-evm: Add pwm backlight support.
>
>   arch/arm/boot/dts/am4372.dtsi        |    9 +++++
>   arch/arm/boot/dts/am43x-epos-evm.dts |   67 ++++++++++++++++++++++++++++++++++
>   include/dt-bindings/pinctrl/am43xx.h |    1 +
>   3 files changed, 77 insertions(+), 0 deletions(-)
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
~Sathya

Re: [PATCH 0/5] Add more device nodes for am43x-epos-evm

[Sourav Poddar]

On Wednesday 27 November 2013 01:38 PM, Sathya Prakash wrote:
> On Wednesday 27 November 2013 01:00 PM, Sourav Poddar wrote:
>> The patch series adds support for enabling pwm backlight, i2c2, spi and
>> matrix gpio keys on am43x-gp-evm.
> Seems a typo - series is for epos evm.
Yes, true. The series if for epos.
>
>>
>> Done on top of 3.13-rc1 + tero clock series(1)
>>
>> [1]: https://patchwork.kernel.org/patch/3009541/
>>
>> Tested on am43x-gp-evm.
>>
>> There is a some bug while using regulators through backlight
>> driver on 3.13-rc1. So, tested pwm part with this temporary patch[2].
>>
>> [2]: http://www.spinics.net/lists/arm-kernel/msg288215.html
>>
>> Darren Etheridge (1):
>>    pinctrl: am43xx: dt-bindings: add MUX_MODE8
>>
>> Sourav Poddar (4):
>>    arm: dts: am4372: Add pwm-cellsproperty for ecap device.
>>    arm: dts: am43x-epos-evm: Add I2C data.
>>    arm: dts: am43x-epos-evm: Add SPI data.
>>    ARM: dts: am43x-epos-evm: Add pwm backlight support.
>>
>>   arch/arm/boot/dts/am4372.dtsi        |    9 +++++
>>   arch/arm/boot/dts/am43x-epos-evm.dts |   67 
>> ++++++++++++++++++++++++++++++++++
>>   include/dt-bindings/pinctrl/am43xx.h |    1 +
>>   3 files changed, 77 insertions(+), 0 deletions(-)
>>
>> -- 
>> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> ~Sathya

Re: [PATCH 0/5] Add more device nodes for am43x-epos-evm

[Sourav Poddar]

On Wednesday 27 November 2013 01:00 PM, Sourav Poddar wrote:
> The patch series adds support for enabling pwm backlight, i2c2, spi and
> matrix gpio keys on am43x-gp-evm.
>
> Done on top of 3.13-rc1 + tero clock series(1)
>
> [1]: https://patchwork.kernel.org/patch/3009541/
>
> Tested on am43x-gp-evm.
>
> There is a some bug while using regulators through backlight
> driver on 3.13-rc1. So, tested pwm part with this temporary patch[2].
>
> [2]: http://www.spinics.net/lists/arm-kernel/msg288215.html
>
> Darren Etheridge (1):
>    pinctrl: am43xx: dt-bindings: add MUX_MODE8
>
> Sourav Poddar (4):
>    arm: dts: am4372: Add pwm-cellsproperty for ecap device.
>    arm: dts: am43x-epos-evm: Add I2C data.
>    arm: dts: am43x-epos-evm: Add SPI data.
>    ARM: dts: am43x-epos-evm: Add pwm backlight support.
>
>   arch/arm/boot/dts/am4372.dtsi        |    9 +++++
>   arch/arm/boot/dts/am43x-epos-evm.dts |   67 ++++++++++++++++++++++++++++++++++
>   include/dt-bindings/pinctrl/am43xx.h |    1 +
>   3 files changed, 77 insertions(+), 0 deletions(-)
>
gentle ping on this..

Re: [PATCH 0/5] Add more device nodes for am43x-epos-evm

[Sourav Poddar]

+ Andrew Morton.

Ping on this.

On Wednesday 27 November 2013 01:00 PM, Sourav Poddar wrote:
> The patch series adds support for enabling pwm backlight, i2c2, spi and
> matrix gpio keys on am43x-gp-evm.
>
> Done on top of 3.13-rc1 + tero clock series(1)
>
> [1]: https://patchwork.kernel.org/patch/3009541/
>
> Tested on am43x-gp-evm.
>
> There is a some bug while using regulators through backlight
> driver on 3.13-rc1. So, tested pwm part with this temporary patch[2].
>
> [2]: http://www.spinics.net/lists/arm-kernel/msg288215.html
>
> Darren Etheridge (1):
>    pinctrl: am43xx: dt-bindings: add MUX_MODE8
>
> Sourav Poddar (4):
>    arm: dts: am4372: Add pwm-cellsproperty for ecap device.
>    arm: dts: am43x-epos-evm: Add I2C data.
>    arm: dts: am43x-epos-evm: Add SPI data.
>    ARM: dts: am43x-epos-evm: Add pwm backlight support.
>
>   arch/arm/boot/dts/am4372.dtsi        |    9 +++++
>   arch/arm/boot/dts/am43x-epos-evm.dts |   67 ++++++++++++++++++++++++++++++++++
>   include/dt-bindings/pinctrl/am43xx.h |    1 +
>   3 files changed, 77 insertions(+), 0 deletions(-)
>

Re: [PATCH 0/5] Add more device nodes for am43x-epos-evm

[Sourav Poddar]

Benoit,

On Wednesday 27 November 2013 01:00 PM, Sourav Poddar wrote:
> The patch series adds support for enabling pwm backlight, i2c2, spi and
> matrix gpio keys on am43x-gp-evm.
>
> Done on top of 3.13-rc1 + tero clock series(1)
>
> [1]: https://patchwork.kernel.org/patch/3009541/
>
> Tested on am43x-gp-evm.
>
> There is a some bug while using regulators through backlight
> driver on 3.13-rc1. So, tested pwm part with this temporary patch[2].
>
> [2]: http://www.spinics.net/lists/arm-kernel/msg288215.html
>
> Darren Etheridge (1):
>    pinctrl: am43xx: dt-bindings: add MUX_MODE8
>
> Sourav Poddar (4):
>    arm: dts: am4372: Add pwm-cellsproperty for ecap device.
>    arm: dts: am43x-epos-evm: Add I2C data.
>    arm: dts: am43x-epos-evm: Add SPI data.
>    ARM: dts: am43x-epos-evm: Add pwm backlight support.
>
>   arch/arm/boot/dts/am4372.dtsi        |    9 +++++
>   arch/arm/boot/dts/am43x-epos-evm.dts |   67 ++++++++++++++++++++++++++++++++++
>   include/dt-bindings/pinctrl/am43xx.h |    1 +
>   3 files changed, 77 insertions(+), 0 deletions(-)
>
Ping on this?

Re: [PATCH 0/5] Add more device nodes for am43x-epos-evm

[Tony Lindgren]

* Sourav Poddar <sourav.poddar@ti.com> [131205 01:50]:
> On Wednesday 27 November 2013 01:00 PM, Sourav Poddar wrote:
> >The patch series adds support for enabling pwm backlight, i2c2, spi and
> >matrix gpio keys on am43x-gp-evm.
> >
> >Done on top of 3.13-rc1 + tero clock series(1)
> >
> >[1]: https://patchwork.kernel.org/patch/3009541/
> >
> >Tested on am43x-gp-evm.
> >
> >There is a some bug while using regulators through backlight
> >driver on 3.13-rc1. So, tested pwm part with this temporary patch[2].
> >
> >[2]: http://www.spinics.net/lists/arm-kernel/msg288215.html
> >
> >Darren Etheridge (1):
> >   pinctrl: am43xx: dt-bindings: add MUX_MODE8
> >
> >Sourav Poddar (4):
> >   arm: dts: am4372: Add pwm-cellsproperty for ecap device.
> >   arm: dts: am43x-epos-evm: Add I2C data.
> >   arm: dts: am43x-epos-evm: Add SPI data.
> >   ARM: dts: am43x-epos-evm: Add pwm backlight support.
> >
> >  arch/arm/boot/dts/am4372.dtsi        |    9 +++++
> >  arch/arm/boot/dts/am43x-epos-evm.dts |   67 ++++++++++++++++++++++++++++++++++
> >  include/dt-bindings/pinctrl/am43xx.h |    1 +
> >  3 files changed, 77 insertions(+), 0 deletions(-)
> >
> gentle ping on this..

FYI at least I don't even have these patches in my inbox any longer,
so hopefully Benoit queues them.

Regards,

Tony