mbox series

[v4,0/4] STM32 warning cleanup

Message ID 20230529091359.71987-1-raphael.gallais-pou@foss.st.com
Headers
Series STM32 warning cleanup |

Message

Raphael Gallais-Pou May 29, 2023, 9:13 a.m. UTC 
  This serie aims to reduce the number of device-tree warnings of
following boards :

  - STM32F469-DISCO
  - STM32MP15*

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

  - 'panel-dsi@0' instead of 'panel@0' according to the YAML
  - unnecessary #address-cells and #size-cells properties
  - residual 'reg' field on single endpoints

First patch fixes stm32f469-disco device-tree.
Second patch fixes st,stm32-dsi.yaml dt-bindings.
Third patch fixes DSI warnings on stm32mp15* device-trees.
Last patch fixes LTDC warnings on stm32mp15* device-trees.

Changes since v3:
	* Added Conor's acked-by
	* Added Marek's reviewed-by
	* Split last patch into two separate for clearer review

Changes since v2:
	* Added changelog
	* Enhanced commit descriptions

Changes since v1:
	* Added DSI subnode name change
	* Included stm32f469-disco DT in the cleanup
	* Included YAML fix to prevent regression

Raphael Gallais-Pou (4):
  ARM: dts: stm32: fix warnings on stm32f469-disco board
  dt-bindings: display: st,stm32-dsi: Remove unnecessary fields
  ARM: dts: stm32: fix dsi warnings on stm32mp15 boards
  ARM: dts: stm32: fix ltdc warnings in stm32mp15 boards

 .../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 -------
 arch/arm/boot/dts/stm32mp157a-dk1.dts         | 20 +++++++++++++++++++
 ...tm32mp157a-icore-stm32mp1-ctouch2-of10.dts |  6 ++++--
 .../stm32mp157a-icore-stm32mp1-edimm2.2.dts   |  6 ++++--
 ...157a-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 +--
 .../boot/dts/stm32mp15xx-dhcor-avenger96.dtsi |  6 +-----
 arch/arm/boot/dts/stm32mp15xx-dkx.dtsi        | 18 +----------------
 13 files changed, 49 insertions(+), 49 deletions(-)

Comments

Alexandre TORGUE May 30, 2023, 8:19 a.m. UTC | #1 
Hi Raphael

On 5/29/23 11:13, Raphael Gallais-Pou wrote:
> This serie aims to reduce the number of device-tree warnings of
> following boards :
> 
>    - STM32F469-DISCO
>    - STM32MP15*
> 
> Those warnings were appearing either during build or when checking
> dt-bindings and concern mostly LTDC and DSI IPs. They were due to the
> following cases:
> 
>    - 'panel-dsi@0' instead of 'panel@0' according to the YAML
>    - unnecessary #address-cells and #size-cells properties
>    - residual 'reg' field on single endpoints
> 
> First patch fixes stm32f469-disco device-tree.
> Second patch fixes st,stm32-dsi.yaml dt-bindings.
> Third patch fixes DSI warnings on stm32mp15* device-trees.
> Last patch fixes LTDC warnings on stm32mp15* device-trees.
> 
> Changes since v3:
> 	* Added Conor's acked-by
> 	* Added Marek's reviewed-by
> 	* Split last patch into two separate for clearer review
> 
> Changes since v2:
> 	* Added changelog
> 	* Enhanced commit descriptions
> 
> Changes since v1:
> 	* Added DSI subnode name change
> 	* Included stm32f469-disco DT in the cleanup
> 	* Included YAML fix to prevent regression
> 
> Raphael Gallais-Pou (4):
>    ARM: dts: stm32: fix warnings on stm32f469-disco board
>    dt-bindings: display: st,stm32-dsi: Remove unnecessary fields
>    ARM: dts: stm32: fix dsi warnings on stm32mp15 boards
>    ARM: dts: stm32: fix ltdc warnings in stm32mp15 boards
> 
>   .../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 -------
>   arch/arm/boot/dts/stm32mp157a-dk1.dts         | 20 +++++++++++++++++++
>   ...tm32mp157a-icore-stm32mp1-ctouch2-of10.dts |  6 ++++--
>   .../stm32mp157a-icore-stm32mp1-edimm2.2.dts   |  6 ++++--
>   ...157a-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 +--
>   .../boot/dts/stm32mp15xx-dhcor-avenger96.dtsi |  6 +-----
>   arch/arm/boot/dts/stm32mp15xx-dkx.dtsi        | 18 +----------------
>   13 files changed, 49 insertions(+), 49 deletions(-)
> 

Thanks for this series! No more W=1 Warnings and no more YAML issue 
regarding DSI/LTDC stuff. Thanks Marek for reviews and feedback.

Series applied on stm32-next.

Cheers
Alx