[PATCH] ARM: dts: stm32: Deduplicate DSI node

[Raphael Gallais-Pou]

Hi Marek,


On 6/1/23 01:10, Marek Vasut wrote:
> All boards using the DSI node duplicate the same pattern common pattern
> in board DTs, that pattern is ports with endpoint labels and the same
> in-SoC regulator connection. Move that common pattern into stm32mp157.dtsi
> instead.

I think that is a good idea, while it did not crossed my mind implementing it
this way on the first time.

I'll let my peers Philippe and Yannick review and ack it this time. I tested it
on DK2, it seems fine.


Thanks :)

> The two boards which do define panel at 0 directly in the DSI bridge node now
> have #address-cells/#size-cells in their board DT instead of it being in
> stm32mp157.dtsi and activated incorrectly for all boards, even the ones
> which use e.g. another DSI-to-something bridge.
>
> Signed-off-by: Marek Vasut <marex at denx.de>


Reviewed-by: Raphael Gallais-Pou <raphael.gallais-pou at foss.st.com>


> ---
> Cc: Alexandre Torgue <alexandre.torgue at foss.st.com>
> Cc: Conor Dooley <conor+dt at kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt at linaro.org>
> Cc: Maxime Coquelin <mcoquelin.stm32 at gmail.com>
> Cc: Raphael Gallais-Pou <raphael.gallais-pou at foss.st.com>
> Cc: Rob Herring <robh+dt at kernel.org>
> Cc: devicetree at vger.kernel.org
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-stm32 at st-md-mailman.stormreply.com
> ---
>  arch/arm/boot/dts/stm32mp157.dtsi             | 18 ++++++++++++
>  ...tm32mp157a-icore-stm32mp1-ctouch2-of10.dts | 24 ++++------------
>  .../stm32mp157a-icore-stm32mp1-edimm2.2.dts   | 28 ++++++-------------
>  arch/arm/boot/dts/stm32mp157c-dk2.dts         | 28 ++++++-------------
>  arch/arm/boot/dts/stm32mp157c-ev1.dts         | 28 ++++++-------------
>  5 files changed, 48 insertions(+), 78 deletions(-)