linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 0/2] Add PWM and backlight support for Colibri VF61 modules
@ 2014-09-22 10:08 Bhuvanchandra DV
  2014-09-22 10:08 ` [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support Bhuvanchandra DV
  2014-09-22 10:08 ` [PATCH v3 2/2] ARM: dts: vf610-colibri: Add backlight support Bhuvanchandra DV
  0 siblings, 2 replies; 7+ messages in thread
From: Bhuvanchandra DV @ 2014-09-22 10:08 UTC (permalink / raw)
  To: linux-arm-kernel

Changes since v2:
[1] Droped the changes done in the root node.
[2] Fix typo in backlight node.

Bhuvanchandra DV (2):
  ARM: dts: vf610-colibri: Add PWM support
  ARM: dts: vf610-colibri: Add backlight support

 arch/arm/boot/dts/vf610-colibri-eval-v3.dts |   14 +++++++++++++
 arch/arm/boot/dts/vf610-colibri.dtsi        |   29 +++++++++++++++++++++++++++
 2 files changed, 43 insertions(+)

-- 
1.7.9.5

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support
  2014-09-22 10:08 [PATCH v3 0/2] Add PWM and backlight support for Colibri VF61 modules Bhuvanchandra DV
@ 2014-09-22 10:08 ` Bhuvanchandra DV
  2014-09-26  2:22   ` Shawn Guo
  2014-09-22 10:08 ` [PATCH v3 2/2] ARM: dts: vf610-colibri: Add backlight support Bhuvanchandra DV
  1 sibling, 1 reply; 7+ messages in thread
From: Bhuvanchandra DV @ 2014-09-22 10:08 UTC (permalink / raw)
  To: linux-arm-kernel

The Colibri standard defines four pins as PWM outputs, two of them (PWM
A and C) are routed to FTM instance 0 and the other two (PWM B and D)
are routed to FTM instance 1. Hence enable both FTM instances for the
Colibri module and mux the four pins accordingly.

Signed-off-by: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
---
 arch/arm/boot/dts/vf610-colibri-eval-v3.dts |    8 ++++++++
 arch/arm/boot/dts/vf610-colibri.dtsi        |   24 ++++++++++++++++++++++++
 2 files changed, 32 insertions(+)

diff --git a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
index 7fb3066..00f63d9 100644
--- a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
+++ b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
@@ -33,6 +33,14 @@
 	status = "okay";
 };
 
+&pwm0 {
+	status = "okay";
+};
+
+&pwm1 {
+	status = "okay";
+};
+
 &uart0 {
 	status = "okay";
 };
diff --git a/arch/arm/boot/dts/vf610-colibri.dtsi b/arch/arm/boot/dts/vf610-colibri.dtsi
index 0cd8343..1f73c1c 100644
--- a/arch/arm/boot/dts/vf610-colibri.dtsi
+++ b/arch/arm/boot/dts/vf610-colibri.dtsi
@@ -44,6 +44,16 @@
 	arm,tag-latency = <3 2 3>;
 };
 
+&pwm0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm0>;
+};
+
+&pwm1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_pwm1>;
+};
+
 &uart0 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_uart0>;
@@ -97,6 +107,20 @@
 			>;
 		};
 
+		pinctrl_pwm0: pwm0grp {
+			fsl,pins = <
+				VF610_PAD_PTB0__FTM0_CH0		0x1182
+				VF610_PAD_PTB1__FTM0_CH1		0x1182
+			>;
+		};
+
+		pinctrl_pwm1: pwm1grp {
+			fsl,pins = <
+				VF610_PAD_PTB8__FTM1_CH0		0x1182
+				VF610_PAD_PTB9__FTM1_CH1		0x1182
+			>;
+		};
+
 		pinctrl_uart0: uart0grp {
 			fsl,pins = <
 				VF610_PAD_PTB10__UART0_TX		0x21a2
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH v3 2/2] ARM: dts: vf610-colibri: Add backlight support
  2014-09-22 10:08 [PATCH v3 0/2] Add PWM and backlight support for Colibri VF61 modules Bhuvanchandra DV
  2014-09-22 10:08 ` [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support Bhuvanchandra DV
@ 2014-09-22 10:08 ` Bhuvanchandra DV
  1 sibling, 0 replies; 7+ messages in thread
From: Bhuvanchandra DV @ 2014-09-22 10:08 UTC (permalink / raw)
  To: linux-arm-kernel


Signed-off-by: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
---
 arch/arm/boot/dts/vf610-colibri-eval-v3.dts |    6 ++++++
 arch/arm/boot/dts/vf610-colibri.dtsi        |    5 +++++
 2 files changed, 11 insertions(+)

diff --git a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
index 00f63d9..711cf82 100644
--- a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
+++ b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
@@ -19,6 +19,12 @@
 	};
 };
 
+&bl {
+	brightness-levels = <0 4 8 16 32 64 128 255>;
+	default-brightness-level = <6>;
+	status  = "okay";
+};
+
 &esdhc1 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_esdhc1>;
diff --git a/arch/arm/boot/dts/vf610-colibri.dtsi b/arch/arm/boot/dts/vf610-colibri.dtsi
index 1f73c1c..807953b 100644
--- a/arch/arm/boot/dts/vf610-colibri.dtsi
+++ b/arch/arm/boot/dts/vf610-colibri.dtsi
@@ -25,6 +25,11 @@
 		};
 	};
 
+	bl: backlight {
+		compatible = "pwm-backlight";
+		pwms = <&pwm0 0 5000000 0>;
+		status = "disabled";
+	};
 };
 
 &esdhc1 {
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support
  2014-09-22 10:08 ` [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support Bhuvanchandra DV
@ 2014-09-26  2:22   ` Shawn Guo
  2014-09-28 16:38     ` BhuvanChandra.DV
  0 siblings, 1 reply; 7+ messages in thread
From: Shawn Guo @ 2014-09-26  2:22 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Sep 22, 2014 at 03:38:11PM +0530, Bhuvanchandra DV wrote:
> The Colibri standard defines four pins as PWM outputs, two of them (PWM
> A and C) are routed to FTM instance 0 and the other two (PWM B and D)
> are routed to FTM instance 1. Hence enable both FTM instances for the
> Colibri module and mux the four pins accordingly.
> 
> Signed-off-by: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>

You should add Stefan's ACK tag.

> ---
>  arch/arm/boot/dts/vf610-colibri-eval-v3.dts |    8 ++++++++
>  arch/arm/boot/dts/vf610-colibri.dtsi        |   24 ++++++++++++++++++++++++
>  2 files changed, 32 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
> index 7fb3066..00f63d9 100644
> --- a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
> +++ b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
> @@ -33,6 +33,14 @@
>  	status = "okay";
>  };
>  
> +&pwm0 {
> +	status = "okay";
> +};
> +
> +&pwm1 {
> +	status = "okay";
> +};
> +

I was trying to apply the patches, but found pwm1 is not available in
vf610.dtsi at all.  What tree is your patch generated against?  Or did I
miss any prerequisite patches?

Shawn

>  &uart0 {
>  	status = "okay";
>  };
> diff --git a/arch/arm/boot/dts/vf610-colibri.dtsi b/arch/arm/boot/dts/vf610-colibri.dtsi
> index 0cd8343..1f73c1c 100644
> --- a/arch/arm/boot/dts/vf610-colibri.dtsi
> +++ b/arch/arm/boot/dts/vf610-colibri.dtsi
> @@ -44,6 +44,16 @@
>  	arm,tag-latency = <3 2 3>;
>  };
>  
> +&pwm0 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&pinctrl_pwm0>;
> +};
> +
> +&pwm1 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&pinctrl_pwm1>;
> +};
> +
>  &uart0 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_uart0>;
> @@ -97,6 +107,20 @@
>  			>;
>  		};
>  
> +		pinctrl_pwm0: pwm0grp {
> +			fsl,pins = <
> +				VF610_PAD_PTB0__FTM0_CH0		0x1182
> +				VF610_PAD_PTB1__FTM0_CH1		0x1182
> +			>;
> +		};
> +
> +		pinctrl_pwm1: pwm1grp {
> +			fsl,pins = <
> +				VF610_PAD_PTB8__FTM1_CH0		0x1182
> +				VF610_PAD_PTB9__FTM1_CH1		0x1182
> +			>;
> +		};
> +
>  		pinctrl_uart0: uart0grp {
>  			fsl,pins = <
>  				VF610_PAD_PTB10__UART0_TX		0x21a2
> -- 
> 1.7.9.5
> 

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support
  2014-09-28 16:38     ` BhuvanChandra.DV
@ 2014-09-28  1:24       ` Shawn Guo
  2014-09-29 16:45         ` BhuvanChandra.DV
  0 siblings, 1 reply; 7+ messages in thread
From: Shawn Guo @ 2014-09-28  1:24 UTC (permalink / raw)
  To: linux-arm-kernel

On Sun, Sep 28, 2014 at 09:38:43AM -0700, BhuvanChandra.DV wrote:
> One prerequisite patch(PATCH 1/3) was missed. Available over here:
> https://lkml.org/lkml/2014/9/17/86

For any prerequisite patches that haven't been applied, you should
either include it in your series or mention the dependency in the cover
letter.

For this thime, I just picked the prerequisite and applied these two
with Stefan's ACK.

Shawn

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support
  2014-09-26  2:22   ` Shawn Guo
@ 2014-09-28 16:38     ` BhuvanChandra.DV
  2014-09-28  1:24       ` Shawn Guo
  0 siblings, 1 reply; 7+ messages in thread
From: BhuvanChandra.DV @ 2014-09-28 16:38 UTC (permalink / raw)
  To: linux-arm-kernel


On 9/25/2014 7:22 PM, Shawn Guo wrote:

> On Mon, Sep 22, 2014 at 03:38:11PM +0530, Bhuvanchandra DV wrote:
>> The Colibri standard defines four pins as PWM outputs, two of them (PWM
>> A and C) are routed to FTM instance 0 and the other two (PWM B and D)
>> are routed to FTM instance 1. Hence enable both FTM instances for the
>> Colibri module and mux the four pins accordingly.
>>
>> Signed-off-by: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
> You should add Stefan's ACK tag.

Ok

>> ---
>>   arch/arm/boot/dts/vf610-colibri-eval-v3.dts |    8 ++++++++
>>   arch/arm/boot/dts/vf610-colibri.dtsi        |   24 ++++++++++++++++++++++++
>>   2 files changed, 32 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
>> index 7fb3066..00f63d9 100644
>> --- a/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
>> +++ b/arch/arm/boot/dts/vf610-colibri-eval-v3.dts
>> @@ -33,6 +33,14 @@
>>   	status = "okay";
>>   };
>>   
>> +&pwm0 {
>> +	status = "okay";
>> +};
>> +
>> +&pwm1 {
>> +	status = "okay";
>> +};
>> +
> I was trying to apply the patches, but found pwm1 is not available in
> vf610.dtsi at all.  What tree is your patch generated against?  Or did I
> miss any prerequisite patches?
>
> Shawn

One prerequisite patch(PATCH 1/3) was missed. Available over here:
https://lkml.org/lkml/2014/9/17/86


>>   &uart0 {
>>   	status = "okay";
>>   };
>> diff --git a/arch/arm/boot/dts/vf610-colibri.dtsi b/arch/arm/boot/dts/vf610-colibri.dtsi
>> index 0cd8343..1f73c1c 100644
>> --- a/arch/arm/boot/dts/vf610-colibri.dtsi
>> +++ b/arch/arm/boot/dts/vf610-colibri.dtsi
>> @@ -44,6 +44,16 @@
>>   	arm,tag-latency = <3 2 3>;
>>   };
>>   
>> +&pwm0 {
>> +	pinctrl-names = "default";
>> +	pinctrl-0 = <&pinctrl_pwm0>;
>> +};
>> +
>> +&pwm1 {
>> +	pinctrl-names = "default";
>> +	pinctrl-0 = <&pinctrl_pwm1>;
>> +};
>> +
>>   &uart0 {
>>   	pinctrl-names = "default";
>>   	pinctrl-0 = <&pinctrl_uart0>;
>> @@ -97,6 +107,20 @@
>>   			>;
>>   		};
>>   
>> +		pinctrl_pwm0: pwm0grp {
>> +			fsl,pins = <
>> +				VF610_PAD_PTB0__FTM0_CH0		0x1182
>> +				VF610_PAD_PTB1__FTM0_CH1		0x1182
>> +			>;
>> +		};
>> +
>> +		pinctrl_pwm1: pwm1grp {
>> +			fsl,pins = <
>> +				VF610_PAD_PTB8__FTM1_CH0		0x1182
>> +				VF610_PAD_PTB9__FTM1_CH1		0x1182
>> +			>;
>> +		};
>> +
>>   		pinctrl_uart0: uart0grp {
>>   			fsl,pins = <
>>   				VF610_PAD_PTB10__UART0_TX		0x21a2
>> -- 
>> 1.7.9.5
>>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support
  2014-09-28  1:24       ` Shawn Guo
@ 2014-09-29 16:45         ` BhuvanChandra.DV
  0 siblings, 0 replies; 7+ messages in thread
From: BhuvanChandra.DV @ 2014-09-29 16:45 UTC (permalink / raw)
  To: linux-arm-kernel


On Sunday 28 September 2014 06:54 AM, Shawn Guo wrote:

> On Sun, Sep 28, 2014 at 09:38:43AM -0700, BhuvanChandra.DV wrote:
>> One prerequisite patch(PATCH 1/3) was missed. Available over here:
>> https://lkml.org/lkml/2014/9/17/86
> For any prerequisite patches that haven't been applied, you should
> either include it in your series or mention the dependency in the cover
> letter.

OK

> For this thime, I just picked the prerequisite and applied these two
> with Stefan's ACK.

Thanks Shawn

>
> Shawn

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2014-09-29 16:45 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-22 10:08 [PATCH v3 0/2] Add PWM and backlight support for Colibri VF61 modules Bhuvanchandra DV
2014-09-22 10:08 ` [PATCH v3 1/2] ARM: dts: vf610-colibri: Add PWM support Bhuvanchandra DV
2014-09-26  2:22   ` Shawn Guo
2014-09-28 16:38     ` BhuvanChandra.DV
2014-09-28  1:24       ` Shawn Guo
2014-09-29 16:45         ` BhuvanChandra.DV
2014-09-22 10:08 ` [PATCH v3 2/2] ARM: dts: vf610-colibri: Add backlight support Bhuvanchandra DV

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).