All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
@ 2020-08-22 20:32 Marek Vasut
  2020-08-28 12:20 ` Alexandre Torgue
  2020-09-23 14:32 ` Alexandre Torgue
  0 siblings, 2 replies; 9+ messages in thread
From: Marek Vasut @ 2020-08-22 20:32 UTC (permalink / raw
  To: linux-arm-kernel
  Cc: Marek Vasut, Alexandre Torgue, Patrice Chotard, Patrick Delaunay,
	Maxime Coquelin, linux-stm32

The display PWM channel is number 3 (PWM2 CH4), make it so.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Alexandre Torgue <alexandre.torgue@st.com>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Cc: Patrice Chotard <patrice.chotard@st.com>
Cc: Patrick Delaunay <patrick.delaunay@st.com>
Cc: linux-stm32@st-md-mailman.stormreply.com
To: linux-arm-kernel@lists.infradead.org
---
 arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
index 7c4bd615b311..075e2a95713f 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
@@ -26,7 +26,7 @@ clk_ext_audio_codec: clock-codec {
 
 	display_bl: display-bl {
 		compatible = "pwm-backlight";
-		pwms = <&pwm2 0 500000 PWM_POLARITY_INVERTED>;
+		pwms = <&pwm2 3 500000 PWM_POLARITY_INVERTED>;
 		brightness-levels = <0 16 22 30 40 55 75 102 138 188 255>;
 		default-brightness-level = <8>;
 		enable-gpios = <&gpioi 0 GPIO_ACTIVE_HIGH>;
-- 
2.28.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
  2020-08-22 20:32 [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel Marek Vasut
@ 2020-08-28 12:20 ` Alexandre Torgue
  2020-08-28 12:26   ` Marek Vasut
  2020-09-23 14:32 ` Alexandre Torgue
  1 sibling, 1 reply; 9+ messages in thread
From: Alexandre Torgue @ 2020-08-28 12:20 UTC (permalink / raw
  To: Marek Vasut, linux-arm-kernel
  Cc: Patrick Delaunay, Patrice Chotard, Maxime Coquelin, linux-stm32

Hi Marek

On 8/22/20 10:32 PM, Marek Vasut wrote:
> The display PWM channel is number 3 (PWM2 CH4), make it so.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>

No fixes tag ?

regards
alex

> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
> Cc: Patrice Chotard <patrice.chotard@st.com>
> Cc: Patrick Delaunay <patrick.delaunay@st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> To: linux-arm-kernel@lists.infradead.org
> ---
>   arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
> index 7c4bd615b311..075e2a95713f 100644
> --- a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
> @@ -26,7 +26,7 @@ clk_ext_audio_codec: clock-codec {
>   
>   	display_bl: display-bl {
>   		compatible = "pwm-backlight";
> -		pwms = <&pwm2 0 500000 PWM_POLARITY_INVERTED>;
> +		pwms = <&pwm2 3 500000 PWM_POLARITY_INVERTED>;
>   		brightness-levels = <0 16 22 30 40 55 75 102 138 188 255>;
>   		default-brightness-level = <8>;
>   		enable-gpios = <&gpioi 0 GPIO_ACTIVE_HIGH>;
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
  2020-08-28 12:20 ` Alexandre Torgue
@ 2020-08-28 12:26   ` Marek Vasut
  0 siblings, 0 replies; 9+ messages in thread
From: Marek Vasut @ 2020-08-28 12:26 UTC (permalink / raw
  To: Alexandre Torgue, linux-arm-kernel
  Cc: Patrick Delaunay, Patrice Chotard, Maxime Coquelin, linux-stm32

On 8/28/20 2:20 PM, Alexandre Torgue wrote:
> Hi Marek

Hi,

> On 8/22/20 10:32 PM, Marek Vasut wrote:
>> The display PWM channel is number 3 (PWM2 CH4), make it so.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
> 
> No fixes tag ?

It should be
Fixes: 34e0c7847dcf ("ARM: dts: stm32: Add DH Electronics DHCOM STM32MP1
SoM and PDK2 board")

except backporting it across
604536dc5837 ("ARM: dts: stm32: Split SoC-independent parts of DHCOM SOM
and PDK2")
would generate a conflict.

If you are fine with that Fixes above, then please add it.

Thanks!

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
@ 2020-08-31 17:09 Marek Vasut
  2020-09-17  0:41 ` Marek Vasut
  0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2020-08-31 17:09 UTC (permalink / raw
  To: linux-arm-kernel
  Cc: Marek Vasut, Alexandre Torgue, Patrice Chotard, Patrick Delaunay,
	Maxime Coquelin, linux-stm32

The display PWM channel is number 3 (PWM2 CH4), make it so.

Fixes: 34e0c7847dcf ("ARM: dts: stm32: Add DH Electronics DHCOM STM32MP1 SoM and PDK2 board")
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Alexandre Torgue <alexandre.torgue@st.com>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
Cc: Patrice Chotard <patrice.chotard@st.com>
Cc: Patrick Delaunay <patrick.delaunay@st.com>
Cc: linux-stm32@st-md-mailman.stormreply.com
To: linux-arm-kernel@lists.infradead.org
---
 arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
index f8ac065d4597..5dff24e39af8 100644
--- a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
+++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
@@ -25,7 +25,7 @@
 
 	display_bl: display-bl {
 		compatible = "pwm-backlight";
-		pwms = <&pwm2 0 500000 PWM_POLARITY_INVERTED>;
+		pwms = <&pwm2 3 500000 PWM_POLARITY_INVERTED>;
 		brightness-levels = <0 16 22 30 40 55 75 102 138 188 255>;
 		default-brightness-level = <8>;
 		enable-gpios = <&gpioi 0 GPIO_ACTIVE_HIGH>;
-- 
2.28.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
  2020-08-31 17:09 Marek Vasut
@ 2020-09-17  0:41 ` Marek Vasut
  0 siblings, 0 replies; 9+ messages in thread
From: Marek Vasut @ 2020-09-17  0:41 UTC (permalink / raw
  To: Alexandre Torgue
  Cc: Patrick Delaunay, linux-stm32, Maxime Coquelin, linux-arm-kernel,
	Patrice Chotard

On 8/31/20 7:09 PM, Marek Vasut wrote:
> The display PWM channel is number 3 (PWM2 CH4), make it so.
> 
> Fixes: 34e0c7847dcf ("ARM: dts: stm32: Add DH Electronics DHCOM STM32MP1 SoM and PDK2 board")

This one seems to be missing from upstream.

btw. is there any chance the stm32 linux upstreaming queue can be merged
into linux-next automatically? It makes development much easier if one
can rebase on next and see how the amount of patches in local tree is
decreasing, and the patches get tested in next a bit more by various CI
runners.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
  2020-08-22 20:32 [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel Marek Vasut
  2020-08-28 12:20 ` Alexandre Torgue
@ 2020-09-23 14:32 ` Alexandre Torgue
  2020-09-23 16:04   ` Marek Vasut
  1 sibling, 1 reply; 9+ messages in thread
From: Alexandre Torgue @ 2020-09-23 14:32 UTC (permalink / raw
  To: Marek Vasut, linux-arm-kernel
  Cc: Patrick Delaunay, Patrice Chotard, Maxime Coquelin, linux-stm32

Hi Marek

On 8/22/20 10:32 PM, Marek Vasut wrote:
> The display PWM channel is number 3 (PWM2 CH4), make it so.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
> Cc: Patrice Chotard <patrice.chotard@st.com>
> Cc: Patrick Delaunay <patrick.delaunay@st.com>
> Cc: linux-stm32@st-md-mailman.stormreply.com
> To: linux-arm-kernel@lists.infradead.org
> ---
>   arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
> index 7c4bd615b311..075e2a95713f 100644
> --- a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
> @@ -26,7 +26,7 @@ clk_ext_audio_codec: clock-codec {
>   
>   	display_bl: display-bl {
>   		compatible = "pwm-backlight";
> -		pwms = <&pwm2 0 500000 PWM_POLARITY_INVERTED>;
> +		pwms = <&pwm2 3 500000 PWM_POLARITY_INVERTED>;
>   		brightness-levels = <0 16 22 30 40 55 75 102 138 188 255>;
>   		default-brightness-level = <8>;
>   		enable-gpios = <&gpioi 0 GPIO_ACTIVE_HIGH>;
> 


Applied on stm32-next.

Thanks.
Alex

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
  2020-09-23 14:32 ` Alexandre Torgue
@ 2020-09-23 16:04   ` Marek Vasut
  2020-09-23 16:23     ` Alexandre Torgue
  0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2020-09-23 16:04 UTC (permalink / raw
  To: Alexandre Torgue, linux-arm-kernel
  Cc: Patrick Delaunay, Patrice Chotard, Maxime Coquelin, linux-stm32

On 9/23/20 4:32 PM, Alexandre Torgue wrote:
> Hi Marek

Hi,

> On 8/22/20 10:32 PM, Marek Vasut wrote:
>> The display PWM channel is number 3 (PWM2 CH4), make it so.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
>> Cc: Alexandre Torgue <alexandre.torgue@st.com>
>> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
>> Cc: Patrice Chotard <patrice.chotard@st.com>
>> Cc: Patrick Delaunay <patrick.delaunay@st.com>
>> Cc: linux-stm32@st-md-mailman.stormreply.com
>> To: linux-arm-kernel@lists.infradead.org
>> ---
>>   arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>> b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>> index 7c4bd615b311..075e2a95713f 100644
>> --- a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>> @@ -26,7 +26,7 @@ clk_ext_audio_codec: clock-codec {
>>         display_bl: display-bl {
>>           compatible = "pwm-backlight";
>> -        pwms = <&pwm2 0 500000 PWM_POLARITY_INVERTED>;
>> +        pwms = <&pwm2 3 500000 PWM_POLARITY_INVERTED>;
>>           brightness-levels = <0 16 22 30 40 55 75 102 138 188 255>;
>>           default-brightness-level = <8>;
>>           enable-gpios = <&gpioi 0 GPIO_ACTIVE_HIGH>;
>>
> 
> 
> Applied on stm32-next.

Don't you want to pick the one with the Fixes: tag ? :)

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
  2020-09-23 16:04   ` Marek Vasut
@ 2020-09-23 16:23     ` Alexandre Torgue
  2020-09-23 16:26       ` Marek Vasut
  0 siblings, 1 reply; 9+ messages in thread
From: Alexandre Torgue @ 2020-09-23 16:23 UTC (permalink / raw
  To: Marek Vasut, linux-arm-kernel
  Cc: Patrick Delaunay, Patrice Chotard, Maxime Coquelin, linux-stm32



On 9/23/20 6:04 PM, Marek Vasut wrote:
> On 9/23/20 4:32 PM, Alexandre Torgue wrote:
>> Hi Marek
> 
> Hi,
> 
>> On 8/22/20 10:32 PM, Marek Vasut wrote:
>>> The display PWM channel is number 3 (PWM2 CH4), make it so.
>>>
>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>> Cc: Alexandre Torgue <alexandre.torgue@st.com>
>>> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
>>> Cc: Patrice Chotard <patrice.chotard@st.com>
>>> Cc: Patrick Delaunay <patrick.delaunay@st.com>
>>> Cc: linux-stm32@st-md-mailman.stormreply.com
>>> To: linux-arm-kernel@lists.infradead.org
>>> ---
>>>    arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi | 2 +-
>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>> b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>> index 7c4bd615b311..075e2a95713f 100644
>>> --- a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>> @@ -26,7 +26,7 @@ clk_ext_audio_codec: clock-codec {
>>>          display_bl: display-bl {
>>>            compatible = "pwm-backlight";
>>> -        pwms = <&pwm2 0 500000 PWM_POLARITY_INVERTED>;
>>> +        pwms = <&pwm2 3 500000 PWM_POLARITY_INVERTED>;
>>>            brightness-levels = <0 16 22 30 40 55 75 102 138 188 255>;
>>>            default-brightness-level = <8>;
>>>            enable-gpios = <&gpioi 0 GPIO_ACTIVE_HIGH>;
>>>
>>
>>
>> Applied on stm32-next.
> 
> Don't you want to pick the one with the Fixes: tag ? :)
> 

Yes I did. I just responded on the bad one ;)

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel
  2020-09-23 16:23     ` Alexandre Torgue
@ 2020-09-23 16:26       ` Marek Vasut
  0 siblings, 0 replies; 9+ messages in thread
From: Marek Vasut @ 2020-09-23 16:26 UTC (permalink / raw
  To: Alexandre Torgue, linux-arm-kernel
  Cc: Patrick Delaunay, Patrice Chotard, Maxime Coquelin, linux-stm32

On 9/23/20 6:23 PM, Alexandre Torgue wrote:
> 
> 
> On 9/23/20 6:04 PM, Marek Vasut wrote:
>> On 9/23/20 4:32 PM, Alexandre Torgue wrote:
>>> Hi Marek
>>
>> Hi,
>>
>>> On 8/22/20 10:32 PM, Marek Vasut wrote:
>>>> The display PWM channel is number 3 (PWM2 CH4), make it so.
>>>>
>>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>>> Cc: Alexandre Torgue <alexandre.torgue@st.com>
>>>> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
>>>> Cc: Patrice Chotard <patrice.chotard@st.com>
>>>> Cc: Patrick Delaunay <patrick.delaunay@st.com>
>>>> Cc: linux-stm32@st-md-mailman.stormreply.com
>>>> To: linux-arm-kernel@lists.infradead.org
>>>> ---
>>>>    arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi | 2 +-
>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>>> b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>>> index 7c4bd615b311..075e2a95713f 100644
>>>> --- a/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>>> +++ b/arch/arm/boot/dts/stm32mp15xx-dhcom-pdk2.dtsi
>>>> @@ -26,7 +26,7 @@ clk_ext_audio_codec: clock-codec {
>>>>          display_bl: display-bl {
>>>>            compatible = "pwm-backlight";
>>>> -        pwms = <&pwm2 0 500000 PWM_POLARITY_INVERTED>;
>>>> +        pwms = <&pwm2 3 500000 PWM_POLARITY_INVERTED>;
>>>>            brightness-levels = <0 16 22 30 40 55 75 102 138 188 255>;
>>>>            default-brightness-level = <8>;
>>>>            enable-gpios = <&gpioi 0 GPIO_ACTIVE_HIGH>;
>>>>
>>>
>>>
>>> Applied on stm32-next.
>>
>> Don't you want to pick the one with the Fixes: tag ? :)
>>
> 
> Yes I did. I just responded on the bad one ;)

Ah, right, OK :)

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, other threads:[~2020-09-23 16:27 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-08-22 20:32 [PATCH] ARM: dts: stm32: Fix DH PDK2 display PWM channel Marek Vasut
2020-08-28 12:20 ` Alexandre Torgue
2020-08-28 12:26   ` Marek Vasut
2020-09-23 14:32 ` Alexandre Torgue
2020-09-23 16:04   ` Marek Vasut
2020-09-23 16:23     ` Alexandre Torgue
2020-09-23 16:26       ` Marek Vasut
  -- strict thread matches above, loose matches on Subject: below --
2020-08-31 17:09 Marek Vasut
2020-09-17  0:41 ` Marek Vasut

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.