mbox series

[0/3] STM32 warning cleanup

Message ID 20230517132214.254757-1-raphael.gallais-pou@foss.st.com (mailing list archive)
Headers show
Series STM32 warning cleanup | expand

Message

Raphael Gallais-Pou May 17, 2023, 1:22 p.m. UTC
This serie aims to reduce the number of device-tree warnings of
following boards :

  - STM32F429-DISCO
  - STM32MP15*

Those warnings were appearing either during build or when checking
dt-bindings and concern mostly LTDC and DSI IPs and were due to the
following cases:

  - panel-dsi@0 nodes that needed
  - unnecessary #address-cells and #size-cells properties
  - residual 'reg' field on single endpoints

Raphael Gallais-Pou (3):
  ARM: dts: stm32: fix warnings on stm32f469-disco board
  dt-bindings: display: st,stm32-dsi: Remove unnecessary fields
  ARM: dts: stm32: fix several DT warnings on stm32mp15

 .../devicetree/bindings/display/st,stm32-dsi.yaml      |  2 --
 arch/arm/boot/dts/stm32f469-disco.dts                  |  4 ++--
 arch/arm/boot/dts/stm32mp151.dtsi                      |  5 -----
 arch/arm/boot/dts/stm32mp157.dtsi                      |  7 -------
 .../dts/stm32mp157a-icore-stm32mp1-ctouch2-of10.dts    |  6 ++++--
 .../boot/dts/stm32mp157a-icore-stm32mp1-edimm2.2.dts   |  6 ++++--
 .../stm32mp157a-microgea-stm32mp1-microdev2.0-of7.dts  |  3 +--
 arch/arm/boot/dts/stm32mp157c-dk2.dts                  |  8 ++++++++
 arch/arm/boot/dts/stm32mp157c-ev1.dts                  | 10 +++++++---
 arch/arm/boot/dts/stm32mp157c-lxa-mc1.dts              |  3 +--
 arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi     |  6 +-----
 arch/arm/boot/dts/stm32mp15xx-dkx.dtsi                 |  3 +--
 12 files changed, 29 insertions(+), 34 deletions(-)

Comments

Marek Vasut May 17, 2023, 2:06 p.m. UTC | #1
On 5/17/23 15:22, Raphael Gallais-Pou wrote:
> This serie aims to reduce the number of device-tree warnings of
> following boards :
> 
>    - STM32F429-DISCO
>    - STM32MP15*
> 
> Those warnings were appearing either during build or when checking
> dt-bindings and concern mostly LTDC and DSI IPs and were due to the
> following cases:
> 
>    - panel-dsi@0 nodes that needed
>    - unnecessary #address-cells and #size-cells properties
>    - residual 'reg' field on single endpoints
> 
> Raphael Gallais-Pou (3):
>    ARM: dts: stm32: fix warnings on stm32f469-disco board
>    dt-bindings: display: st,stm32-dsi: Remove unnecessary fields
>    ARM: dts: stm32: fix several DT warnings on stm32mp15
> 
>   .../devicetree/bindings/display/st,stm32-dsi.yaml      |  2 --
>   arch/arm/boot/dts/stm32f469-disco.dts                  |  4 ++--
>   arch/arm/boot/dts/stm32mp151.dtsi                      |  5 -----
>   arch/arm/boot/dts/stm32mp157.dtsi                      |  7 -------
>   .../dts/stm32mp157a-icore-stm32mp1-ctouch2-of10.dts    |  6 ++++--
>   .../boot/dts/stm32mp157a-icore-stm32mp1-edimm2.2.dts   |  6 ++++--
>   .../stm32mp157a-microgea-stm32mp1-microdev2.0-of7.dts  |  3 +--
>   arch/arm/boot/dts/stm32mp157c-dk2.dts                  |  8 ++++++++
>   arch/arm/boot/dts/stm32mp157c-ev1.dts                  | 10 +++++++---
>   arch/arm/boot/dts/stm32mp157c-lxa-mc1.dts              |  3 +--
>   arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi     |  6 +-----
>   arch/arm/boot/dts/stm32mp15xx-dkx.dtsi                 |  3 +--
>   12 files changed, 29 insertions(+), 34 deletions(-)
> 

Is this a V2 series ?

If so, please do include changelog .

Also, use "git send-email -v2" to mark those patches as v2 automatically 
when sending .
Raphael Gallais-Pou May 17, 2023, 2:38 p.m. UTC | #2
On 5/17/23 16:06, Marek Vasut wrote:
> On 5/17/23 15:22, Raphael Gallais-Pou wrote:
>> This serie aims to reduce the number of device-tree warnings of
>> following boards :
>>
>>    - STM32F429-DISCO
>>    - STM32MP15*
>>
>> Those warnings were appearing either during build or when checking
>> dt-bindings and concern mostly LTDC and DSI IPs and were due to the
>> following cases:
>>
>>    - panel-dsi@0 nodes that needed
>>    - unnecessary #address-cells and #size-cells properties
>>    - residual 'reg' field on single endpoints
>>
>> Raphael Gallais-Pou (3):
>>    ARM: dts: stm32: fix warnings on stm32f469-disco board
>>    dt-bindings: display: st,stm32-dsi: Remove unnecessary fields
>>    ARM: dts: stm32: fix several DT warnings on stm32mp15
>>
>>   .../devicetree/bindings/display/st,stm32-dsi.yaml      |  2 --
>>   arch/arm/boot/dts/stm32f469-disco.dts                  |  4 ++--
>>   arch/arm/boot/dts/stm32mp151.dtsi                      |  5 -----
>>   arch/arm/boot/dts/stm32mp157.dtsi                      |  7 -------
>>   .../dts/stm32mp157a-icore-stm32mp1-ctouch2-of10.dts    |  6 ++++--
>>   .../boot/dts/stm32mp157a-icore-stm32mp1-edimm2.2.dts   |  6 ++++--
>>   .../stm32mp157a-microgea-stm32mp1-microdev2.0-of7.dts  |  3 +--
>>   arch/arm/boot/dts/stm32mp157c-dk2.dts                  |  8 ++++++++
>>   arch/arm/boot/dts/stm32mp157c-ev1.dts                  | 10 +++++++---
>>   arch/arm/boot/dts/stm32mp157c-lxa-mc1.dts              |  3 +--
>>   arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi     |  6 +-----
>>   arch/arm/boot/dts/stm32mp15xx-dkx.dtsi                 |  3 +--
>>   12 files changed, 29 insertions(+), 34 deletions(-)
>>
>
> Is this a V2 series ?

Hi Marek


yes it was,

Thanks for highlighting it to me.

Raphaël

>
> If so, please do include changelog .
>
> Also, use "git send-email -v2" to mark those patches as v2 automatically when
> sending .
Marek Vasut May 17, 2023, 3:38 p.m. UTC | #3
On 5/17/23 16:38, Raphael Gallais-Pou wrote:
> 
> On 5/17/23 16:06, Marek Vasut wrote:
>> On 5/17/23 15:22, Raphael Gallais-Pou wrote:
>>> This serie aims to reduce the number of device-tree warnings of
>>> following boards :
>>>
>>>     - STM32F429-DISCO
>>>     - STM32MP15*
>>>
>>> Those warnings were appearing either during build or when checking
>>> dt-bindings and concern mostly LTDC and DSI IPs and were due to the
>>> following cases:
>>>
>>>     - panel-dsi@0 nodes that needed
>>>     - unnecessary #address-cells and #size-cells properties
>>>     - residual 'reg' field on single endpoints
>>>
>>> Raphael Gallais-Pou (3):
>>>     ARM: dts: stm32: fix warnings on stm32f469-disco board
>>>     dt-bindings: display: st,stm32-dsi: Remove unnecessary fields
>>>     ARM: dts: stm32: fix several DT warnings on stm32mp15
>>>
>>>    .../devicetree/bindings/display/st,stm32-dsi.yaml      |  2 --
>>>    arch/arm/boot/dts/stm32f469-disco.dts                  |  4 ++--
>>>    arch/arm/boot/dts/stm32mp151.dtsi                      |  5 -----
>>>    arch/arm/boot/dts/stm32mp157.dtsi                      |  7 -------
>>>    .../dts/stm32mp157a-icore-stm32mp1-ctouch2-of10.dts    |  6 ++++--
>>>    .../boot/dts/stm32mp157a-icore-stm32mp1-edimm2.2.dts   |  6 ++++--
>>>    .../stm32mp157a-microgea-stm32mp1-microdev2.0-of7.dts  |  3 +--
>>>    arch/arm/boot/dts/stm32mp157c-dk2.dts                  |  8 ++++++++
>>>    arch/arm/boot/dts/stm32mp157c-ev1.dts                  | 10 +++++++---
>>>    arch/arm/boot/dts/stm32mp157c-lxa-mc1.dts              |  3 +--
>>>    arch/arm/boot/dts/stm32mp15xx-dhcor-avenger96.dtsi     |  6 +-----
>>>    arch/arm/boot/dts/stm32mp15xx-dkx.dtsi                 |  3 +--
>>>    12 files changed, 29 insertions(+), 34 deletions(-)
>>>
>>
>> Is this a V2 series ?
> 
> Hi Marek
> 
> 
> yes it was,
> 
> Thanks for highlighting it to me.

You're welcome, I see a V3 now, thanks !