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

2023-12-15 Thread 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 
> 
> ---
> 
>  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 = < STM32F7_APB2_RESET(LTDC)>;
> - clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> -
> - status = "okay";
> - bootph-all;
> + {
> + clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> + bootph-all;
>  
> - ports {
> - port@0 {
> - dp_out: endpoint {
> - remote-endpoint = <_in>;
> - };
> - };
> + ports {
> + port@0 {
> + dp_out: endpoint {
> + remote-endpoint = <_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";
>  };
>  
> + {
> + status = "okay";
> +};
> +
>   {
>   status = "okay";
>  };

Applied to u-boot-stm32/next

Thanks
Patrice

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

2023-11-13 Thread 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 
> 
> ---
> 
>  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 = < STM32F7_APB2_RESET(LTDC)>;
> - clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> -
> - status = "okay";
> - bootph-all;
> + {
> + clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> + bootph-all;
>  
> - ports {
> - port@0 {
> - dp_out: endpoint {
> - remote-endpoint = <_in>;
> - };
> - };
> + ports {
> + port@0 {
> + dp_out: endpoint {
> + remote-endpoint = <_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";
>  };
>  
> + {
> + status = "okay";
> +};
> +
>   {
>   status = "okay";
>  };

Reviewed-by: Patrice Chotard 

Thanks
Patrice

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

2023-11-13 Thread 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 
> 
> ---
> 
>  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 = < STM32F7_APB2_RESET(LTDC)>;
> - clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> -
> - status = "okay";
> - bootph-all;
> + {
> + clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> + bootph-all;
>  
> - ports {
> - port@0 {
> - dp_out: endpoint {
> - remote-endpoint = <_in>;
> - };
> - };
> + ports {
> + port@0 {
> + dp_out: endpoint {
> + remote-endpoint = <_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";
>  };
>  
> + {
> + status = "okay";
> +};
> +
>   {
>   status = "okay";
>  };
Reviewed-by: Patrice Chotard 

Thanks
Patrice

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

2023-11-11 Thread Dario Binacchi 
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 

---

 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 = < STM32F7_APB2_RESET(LTDC)>;
-   clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
-
-   status = "okay";
-   bootph-all;
+ {
+   clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
+   bootph-all;
 
-   ports {
-   port@0 {
-   dp_out: endpoint {
-   remote-endpoint = <_in>;
-   };
-   };
+   ports {
+   port@0 {
+   dp_out: endpoint {
+   remote-endpoint = <_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";
 };
 
+ {
+   status = "okay";
+};
+
  {
status = "okay";
 };
-- 
2.42.0