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

[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 <dario.binacchi at amarulasolutions.com>
> 
> ---
> 
>  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 = &qspi;
>  	};
>  
> -	backlight: backlight {
> -		compatible = "gpio-backlight";
> -		gpios = <&gpiok 3 0>;
> -		status = "okay";
> -	};
> -
>  	button1 {
>  		compatible = "st,button1";
>  		button-gpio = <&gpioi 11 0>;
> @@ -38,37 +32,10 @@
>  		compatible = "st,led1";
>  		led-gpio = <&gpioi 1 0>;
>  	};
> -
> -	panel-rgb at 0 {
> -		compatible = "simple-panel";
> -		backlight = <&backlight>;
> -		enable-gpios = <&gpioi 12 0>;
> -		status = "okay";
> -
> -		display-timings {
> -			timing at 0 {
> -				clock-frequency = <9000000>;
> -				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>;
> -			};
> -		};
> -	};
>  };
>  
>  &ltdc {
>  	clocks = <&rcc 0 STM32F7_APB2_CLOCK(LTDC)>;
> -	pinctrl-0 = <&ltdc_pins>;
> -	status = "okay";
>  	bootph-all;
>  };
>  
> @@ -96,6 +63,28 @@
>  	};
>  };
>  
> +&panel_rgb {
> +	compatible = "simple-panel";
> +
> +	display-timings {
> +		timing at 0 {
> +			clock-frequency = <9000000>;
> +			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>;
> +		};
> +	};
> +};
> +
>  &pinctrl {
>  	ethernet_mii: mii at 0 {
>  		pins {
> @@ -160,40 +149,6 @@
>  		};
>  	};
>  
> -	ltdc_pins: ltdc at 0 {
> -		pins {
> -			pinmux = <STM32_PINMUX('E', 4, AF14)>, /* B0 */
> -				 <STM32_PINMUX('G',12, AF9)>,  /* B4 */
> -				 <STM32_PINMUX('I', 9, AF14)>, /* VSYNC */
> -				 <STM32_PINMUX('I',10, AF14)>, /* HSYNC */
> -				 <STM32_PINMUX('I',14, AF14)>, /* CLK */
> -				 <STM32_PINMUX('I',15, AF14)>, /* R0 */
> -				 <STM32_PINMUX('J', 0, AF14)>, /* R1 */
> -				 <STM32_PINMUX('J', 1, AF14)>, /* R2 */
> -				 <STM32_PINMUX('J', 2, AF14)>, /* R3 */
> -				 <STM32_PINMUX('J', 3, AF14)>, /* R4 */
> -				 <STM32_PINMUX('J', 4, AF14)>, /* R5 */
> -				 <STM32_PINMUX('J', 5, AF14)>, /* R6 */
> -				 <STM32_PINMUX('J', 6, AF14)>, /* R7 */
> -				 <STM32_PINMUX('J', 7, AF14)>, /* G0 */
> -				 <STM32_PINMUX('J', 8, AF14)>, /* G1 */
> -				 <STM32_PINMUX('J', 9, AF14)>, /* G2 */
> -				 <STM32_PINMUX('J',10, AF14)>, /* G3 */
> -				 <STM32_PINMUX('J',11, AF14)>, /* G4 */
> -				 <STM32_PINMUX('J',13, AF14)>, /* B1 */
> -				 <STM32_PINMUX('J',14, AF14)>, /* B2 */
> -				 <STM32_PINMUX('J',15, AF14)>, /* B3 */
> -				 <STM32_PINMUX('K', 0, AF14)>, /* G5 */
> -				 <STM32_PINMUX('K', 1, AF14)>, /* G6 */
> -				 <STM32_PINMUX('K', 2, AF14)>, /* G7 */
> -				 <STM32_PINMUX('K', 4, AF14)>, /* B5 */
> -				 <STM32_PINMUX('K', 5, AF14)>, /* B6 */
> -				 <STM32_PINMUX('K', 6, AF14)>, /* B7 */
> -				 <STM32_PINMUX('K', 7, AF14)>; /* DE */
> -			slew-rate = <2>;
> -		};
> -	};
> -
>  	qspi_pins: qspi at 0 {
>  		pins {
>  			pinmux = <STM32_PINMUX('B', 2, AF9)>, /* 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 = <0xC0000000 0x800000>;
>  	};
>  
> +	reserved-memory {
> +		#address-cells = <1>;
> +		#size-cells = <1>;
> +		ranges;
> +
> +		linux,cma {
> +			compatible = "shared-dma-pool";
> +			no-map;
> +			size = <0x80000>;
> +			linux,dma-default;
> +		};
> +	};
> +
>  	aliases {
>  		serial0 = &usart1;
>  	};
> @@ -50,6 +63,25 @@
>  		regulator-min-microvolt = <3300000>;
>  		regulator-max-microvolt = <3300000>;
>  	};
> +
> +	backlight: backlight {
> +		compatible = "gpio-backlight";
> +		gpios = <&gpiok 3 GPIO_ACTIVE_HIGH>;
> +		status = "okay";
> +	};
> +
> +	panel_rgb: panel-rgb {
> +		compatible = "rocktech,rk043fn48h";
> +		power-supply = <&vcc_3v3>;
> +		backlight = <&backlight>;
> +		enable-gpios = <&gpioi 12 GPIO_ACTIVE_HIGH>;
> +		status = "okay";
> +		port {
> +			panel_in_rgb: endpoint {
> +				remote-endpoint = <&ltdc_out_rgb>;
> +			};
> +		};
> +	};
>  };
>  
>  &clk_hse {
> @@ -80,6 +112,18 @@
>  	};
>  };
>  
> +&ltdc {
> +	pinctrl-0 = <&ltdc_pins_a>;
> +	pinctrl-names = "default";
> +	status = "okay";
> +
> +	port {
> +		ltdc_out_rgb: endpoint {
> +			remote-endpoint = <&panel_in_rgb>;
> +		};
> +	};
> +};
> +
>  &sdio1 {
>  	status = "okay";
>  	vmmc-supply = <&vcc_3v3>;

Reviewed-by: Patrice Chotard <patrice.chotard at foss.st.com>

Thanks
Patrice