Re: [PATCH 10/10] ARM: dts: stm32: support display on stm32f746-disco board

2023-09-26 Thread Patrice CHOTARD



On 9/3/23 22:48, Dario Binacchi wrote:
> The patch applies the changes from Linux commit 10a970bc3ebfa ("ARM: dts:
> stm32: support display on stm32f746-disco board") and removes the same
> settings from stm32f746-disco-u-boot.dtsi.
> 
> Signed-off-by: Dario Binacchi 
> 
> ---
> 
>  arch/arm/dts/stm32f746-disco-u-boot.dtsi | 89 ++--
>  arch/arm/dts/stm32f746-disco.dts | 44 
>  2 files changed, 66 insertions(+), 67 deletions(-)
> 
> diff --git a/arch/arm/dts/stm32f746-disco-u-boot.dtsi 
> b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
> index 3c2b9fc59512..1b42d6cbbc19 100644
> --- a/arch/arm/dts/stm32f746-disco-u-boot.dtsi
> +++ b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
> @@ -23,12 +23,6 @@
>   spi0 = 
>   };
>  
> - backlight: backlight {
> - compatible = "gpio-backlight";
> - gpios = < 3 0>;
> - status = "okay";
> - };
> -
>   button1 {
>   compatible = "st,button1";
>   button-gpio = < 11 0>;
> @@ -38,37 +32,10 @@
>   compatible = "st,led1";
>   led-gpio = < 1 0>;
>   };
> -
> - panel-rgb@0 {
> - compatible = "simple-panel";
> - backlight = <>;
> - enable-gpios = < 12 0>;
> - status = "okay";
> -
> - display-timings {
> - timing@0 {
> - clock-frequency = <900>;
> - hactive = <480>;
> - vactive = <272>;
> - hfront-porch = <2>;
> - hback-porch = <2>;
> - hsync-len = <41>;
> - vfront-porch = <2>;
> - vback-porch = <2>;
> - vsync-len = <10>;
> - hsync-active = <0>;
> - vsync-active = <0>;
> - de-active = <1>;
> - pixelclk-active = <1>;
> - };
> - };
> - };
>  };
>  
>   {
>   clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> - pinctrl-0 = <_pins>;
> - status = "okay";
>   bootph-all;
>  };
>  
> @@ -96,6 +63,28 @@
>   };
>  };
>  
> +_rgb {
> + compatible = "simple-panel";
> +
> + display-timings {
> + timing@0 {
> + clock-frequency = <900>;
> + hactive = <480>;
> + vactive = <272>;
> + hfront-porch = <2>;
> + hback-porch = <2>;
> + hsync-len = <41>;
> + vfront-porch = <2>;
> + vback-porch = <2>;
> + vsync-len = <10>;
> + hsync-active = <0>;
> + vsync-active = <0>;
> + de-active = <1>;
> + pixelclk-active = <1>;
> + };
> + };
> +};
> +
>   {
>   ethernet_mii: mii@0 {
>   pins {
> @@ -160,40 +149,6 @@
>   };
>   };
>  
> - ltdc_pins: ltdc@0 {
> - pins {
> - pinmux = , /* B0 */
> -  ,  /* B4 */
> -  , /* VSYNC */
> -  , /* HSYNC */
> -  , /* CLK */
> -  , /* R0 */
> -  , /* R1 */
> -  , /* R2 */
> -  , /* R3 */
> -  , /* R4 */
> -  , /* R5 */
> -  , /* R6 */
> -  , /* R7 */
> -  , /* G0 */
> -  , /* G1 */
> -  , /* G2 */
> -  , /* G3 */
> -  , /* G4 */
> -  , /* B1 */
> -  , /* B2 */
> -  , /* B3 */
> -  , /* G5 */
> -  , /* G6 */
> -  , /* G7 */
> -  , /* B5 */
> -  , /* B6 */
> -  , /* B7 */
> -  ; /* DE */
> - slew-rate = <2>;
> - };
> - };
> -
>   qspi_pins: qspi@0 {
>   pins {
>   pinmux = , /* CLK */
> diff --git a/arch/arm/dts/stm32f746-disco.dts 
> b/arch/arm/dts/stm32f746-disco.dts
> index e1564d69f9f6..431275134033 100644
> --- a/arch/arm/dts/stm32f746-disco.dts
> +++ b/arch/arm/dts/stm32f746-disco.dts
> @@ -25,6 +25,19 @@
>   reg = <0xC000 0x80>;
>   };
>  
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + 

Re: [PATCH 10/10] ARM: dts: stm32: support display on stm32f746-disco board

2023-09-26 Thread Patrice CHOTARD



On 9/3/23 22:48, Dario Binacchi wrote:
> The patch applies the changes from Linux commit 10a970bc3ebfa ("ARM: dts:
> stm32: support display on stm32f746-disco board") and removes the same
> settings from stm32f746-disco-u-boot.dtsi.
> 
> Signed-off-by: Dario Binacchi 
> 
> ---
> 
>  arch/arm/dts/stm32f746-disco-u-boot.dtsi | 89 ++--
>  arch/arm/dts/stm32f746-disco.dts | 44 
>  2 files changed, 66 insertions(+), 67 deletions(-)
> 
> diff --git a/arch/arm/dts/stm32f746-disco-u-boot.dtsi 
> b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
> index 3c2b9fc59512..1b42d6cbbc19 100644
> --- a/arch/arm/dts/stm32f746-disco-u-boot.dtsi
> +++ b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
> @@ -23,12 +23,6 @@
>   spi0 = 
>   };
>  
> - backlight: backlight {
> - compatible = "gpio-backlight";
> - gpios = < 3 0>;
> - status = "okay";
> - };
> -
>   button1 {
>   compatible = "st,button1";
>   button-gpio = < 11 0>;
> @@ -38,37 +32,10 @@
>   compatible = "st,led1";
>   led-gpio = < 1 0>;
>   };
> -
> - panel-rgb@0 {
> - compatible = "simple-panel";
> - backlight = <>;
> - enable-gpios = < 12 0>;
> - status = "okay";
> -
> - display-timings {
> - timing@0 {
> - clock-frequency = <900>;
> - hactive = <480>;
> - vactive = <272>;
> - hfront-porch = <2>;
> - hback-porch = <2>;
> - hsync-len = <41>;
> - vfront-porch = <2>;
> - vback-porch = <2>;
> - vsync-len = <10>;
> - hsync-active = <0>;
> - vsync-active = <0>;
> - de-active = <1>;
> - pixelclk-active = <1>;
> - };
> - };
> - };
>  };
>  
>   {
>   clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
> - pinctrl-0 = <_pins>;
> - status = "okay";
>   bootph-all;
>  };
>  
> @@ -96,6 +63,28 @@
>   };
>  };
>  
> +_rgb {
> + compatible = "simple-panel";
> +
> + display-timings {
> + timing@0 {
> + clock-frequency = <900>;
> + hactive = <480>;
> + vactive = <272>;
> + hfront-porch = <2>;
> + hback-porch = <2>;
> + hsync-len = <41>;
> + vfront-porch = <2>;
> + vback-porch = <2>;
> + vsync-len = <10>;
> + hsync-active = <0>;
> + vsync-active = <0>;
> + de-active = <1>;
> + pixelclk-active = <1>;
> + };
> + };
> +};
> +
>   {
>   ethernet_mii: mii@0 {
>   pins {
> @@ -160,40 +149,6 @@
>   };
>   };
>  
> - ltdc_pins: ltdc@0 {
> - pins {
> - pinmux = , /* B0 */
> -  ,  /* B4 */
> -  , /* VSYNC */
> -  , /* HSYNC */
> -  , /* CLK */
> -  , /* R0 */
> -  , /* R1 */
> -  , /* R2 */
> -  , /* R3 */
> -  , /* R4 */
> -  , /* R5 */
> -  , /* R6 */
> -  , /* R7 */
> -  , /* G0 */
> -  , /* G1 */
> -  , /* G2 */
> -  , /* G3 */
> -  , /* G4 */
> -  , /* B1 */
> -  , /* B2 */
> -  , /* B3 */
> -  , /* G5 */
> -  , /* G6 */
> -  , /* G7 */
> -  , /* B5 */
> -  , /* B6 */
> -  , /* B7 */
> -  ; /* DE */
> - slew-rate = <2>;
> - };
> - };
> -
>   qspi_pins: qspi@0 {
>   pins {
>   pinmux = , /* CLK */
> diff --git a/arch/arm/dts/stm32f746-disco.dts 
> b/arch/arm/dts/stm32f746-disco.dts
> index e1564d69f9f6..431275134033 100644
> --- a/arch/arm/dts/stm32f746-disco.dts
> +++ b/arch/arm/dts/stm32f746-disco.dts
> @@ -25,6 +25,19 @@
>   reg = <0xC000 0x80>;
>   };
>  
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + 

[PATCH 10/10] ARM: dts: stm32: support display on stm32f746-disco board

2023-09-03 Thread Dario Binacchi
The patch applies the changes from Linux commit 10a970bc3ebfa ("ARM: dts:
stm32: support display on stm32f746-disco board") and removes the same
settings from stm32f746-disco-u-boot.dtsi.

Signed-off-by: Dario Binacchi 

---

 arch/arm/dts/stm32f746-disco-u-boot.dtsi | 89 ++--
 arch/arm/dts/stm32f746-disco.dts | 44 
 2 files changed, 66 insertions(+), 67 deletions(-)

diff --git a/arch/arm/dts/stm32f746-disco-u-boot.dtsi 
b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
index 3c2b9fc59512..1b42d6cbbc19 100644
--- a/arch/arm/dts/stm32f746-disco-u-boot.dtsi
+++ b/arch/arm/dts/stm32f746-disco-u-boot.dtsi
@@ -23,12 +23,6 @@
spi0 = 
};
 
-   backlight: backlight {
-   compatible = "gpio-backlight";
-   gpios = < 3 0>;
-   status = "okay";
-   };
-
button1 {
compatible = "st,button1";
button-gpio = < 11 0>;
@@ -38,37 +32,10 @@
compatible = "st,led1";
led-gpio = < 1 0>;
};
-
-   panel-rgb@0 {
-   compatible = "simple-panel";
-   backlight = <>;
-   enable-gpios = < 12 0>;
-   status = "okay";
-
-   display-timings {
-   timing@0 {
-   clock-frequency = <900>;
-   hactive = <480>;
-   vactive = <272>;
-   hfront-porch = <2>;
-   hback-porch = <2>;
-   hsync-len = <41>;
-   vfront-porch = <2>;
-   vback-porch = <2>;
-   vsync-len = <10>;
-   hsync-active = <0>;
-   vsync-active = <0>;
-   de-active = <1>;
-   pixelclk-active = <1>;
-   };
-   };
-   };
 };
 
  {
clocks = < 0 STM32F7_APB2_CLOCK(LTDC)>;
-   pinctrl-0 = <_pins>;
-   status = "okay";
bootph-all;
 };
 
@@ -96,6 +63,28 @@
};
 };
 
+_rgb {
+   compatible = "simple-panel";
+
+   display-timings {
+   timing@0 {
+   clock-frequency = <900>;
+   hactive = <480>;
+   vactive = <272>;
+   hfront-porch = <2>;
+   hback-porch = <2>;
+   hsync-len = <41>;
+   vfront-porch = <2>;
+   vback-porch = <2>;
+   vsync-len = <10>;
+   hsync-active = <0>;
+   vsync-active = <0>;
+   de-active = <1>;
+   pixelclk-active = <1>;
+   };
+   };
+};
+
  {
ethernet_mii: mii@0 {
pins {
@@ -160,40 +149,6 @@
};
};
 
-   ltdc_pins: ltdc@0 {
-   pins {
-   pinmux = , /* B0 */
-,  /* B4 */
-, /* VSYNC */
-, /* HSYNC */
-, /* CLK */
-, /* R0 */
-, /* R1 */
-, /* R2 */
-, /* R3 */
-, /* R4 */
-, /* R5 */
-, /* R6 */
-, /* R7 */
-, /* G0 */
-, /* G1 */
-, /* G2 */
-, /* G3 */
-, /* G4 */
-, /* B1 */
-, /* B2 */
-, /* B3 */
-, /* G5 */
-, /* G6 */
-, /* G7 */
-, /* B5 */
-, /* B6 */
-, /* B7 */
-; /* DE */
-   slew-rate = <2>;
-   };
-   };
-
qspi_pins: qspi@0 {
pins {
pinmux = , /* CLK */
diff --git a/arch/arm/dts/stm32f746-disco.dts b/arch/arm/dts/stm32f746-disco.dts
index e1564d69f9f6..431275134033 100644
--- a/arch/arm/dts/stm32f746-disco.dts
+++ b/arch/arm/dts/stm32f746-disco.dts
@@ -25,6 +25,19 @@
reg = <0xC000 0x80>;
};
 
+   reserved-memory {
+   #address-cells = <1>;
+   #size-cells = <1>;
+   ranges;
+
+   linux,cma {
+   compatible = "shared-dma-pool";
+