Re: [PATCH] ARM: dts: stm32f769-disco: rework ltdc node

[Patrice CHOTARD]

On 11/11/23 11:44, Dario Binacchi wrote:
> With commit f479f5dbb7ac ("ARM: dts: stm32: add ltdc support on
> stm32f746 MCU"), which adds the 'ltdc' node in stm32f746.dtsi, we can
> simplify stm32f769-disco-uboot.dtsi and align stm32f769-disco.dtsi with
> the kernel version.
> 
> Signed-off-by: Dario Binacchi <dario.binac...@amarulasolutions.com>
> 
> ---
> 
>  arch/arm/dts/stm32f769-disco-u-boot.dtsi | 23 +++++++++--------------
>  arch/arm/dts/stm32f769-disco.dts         |  4 ++++
>  2 files changed, 13 insertions(+), 14 deletions(-)
> 
> diff --git a/arch/arm/dts/stm32f769-disco-u-boot.dtsi 
> b/arch/arm/dts/stm32f769-disco-u-boot.dtsi
> index 2c823cce98b4..add55c96e21f 100644
> --- a/arch/arm/dts/stm32f769-disco-u-boot.dtsi
> +++ b/arch/arm/dts/stm32f769-disco-u-boot.dtsi
> @@ -70,22 +70,17 @@
>                               };
>                       };
>               };
> +     };
> +};
>  
> -             ltdc: display-controller@40016800 {
> -                     compatible = "st,stm32-ltdc";
> -                     reg = <0x40016800 0x200>;
> -                     resets = <&rcc STM32F7_APB2_RESET(LTDC)>;
> -                     clocks = <&rcc 0 STM32F7_APB2_CLOCK(LTDC)>;
> -
> -                     status = "okay";
> -                     bootph-all;
> +&ltdc {
> +     clocks = <&rcc 0 STM32F7_APB2_CLOCK(LTDC)>;
> +     bootph-all;
>  
> -                     ports {
> -                             port@0 {
> -                                     dp_out: endpoint {
> -                                             remote-endpoint = <&dsi_in>;
> -                                     };
> -                             };
> +     ports {
> +             port@0 {
> +                     dp_out: endpoint {
> +                             remote-endpoint = <&dsi_in>;
>                       };
>               };
>       };
> diff --git a/arch/arm/dts/stm32f769-disco.dts 
> b/arch/arm/dts/stm32f769-disco.dts
> index 6f93fc7bcfcd..d63cd2ba7eb4 100644
> --- a/arch/arm/dts/stm32f769-disco.dts
> +++ b/arch/arm/dts/stm32f769-disco.dts
> @@ -86,6 +86,10 @@
>       status = "okay";
>  };
>  
> +&ltdc {
> +     status = "okay";
> +};
> +
>  &rtc {
>       status = "okay";
>  };
Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com>

Thanks
Patrice