Re: [PATCH 1/1] arm64: dts: rockchip: disable display subsystem only for Radxa E25

[Dragan Simic <dsimic@manjaro.org>]

Hello Chukun,

On 2024-08-20 14:00, Chukun Pan wrote:
> The SoM board has reserved HDMI output, while the Radxa E25
> is not connected. So disable the display subsystem only for
> Radxa E25.
> 
> Signed-off-by: Chukun Pan <amadeus@jmu.edu.cn>

Looking good to me, thanks for the patch.  The carrier board is what's
responsible for exposing the actual HDMI port from the SoM.

Reviewed-by: Dragan Simic <dsimic@manjaro.org>

> ---
>  arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3i.dtsi | 4 ----
>  arch/arm64/boot/dts/rockchip/rk3568-radxa-e25.dts   | 4 ++++
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3i.dtsi
> b/arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3i.dtsi
> index 45b03dcbbad4..19d309654bdb 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3i.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3568-radxa-cm3i.dtsi
> @@ -108,10 +108,6 @@ &cpu3 {
>  	cpu-supply = <&vdd_cpu>;
>  };
> 
> -&display_subsystem {
> -	status = "disabled";
> -};
> -
>  &gpu {
>  	mali-supply = <&vdd_gpu>;
>  	status = "okay";
> diff --git a/arch/arm64/boot/dts/rockchip/rk3568-radxa-e25.dts
> b/arch/arm64/boot/dts/rockchip/rk3568-radxa-e25.dts
> index 72ad74c38a2b..84a0789fad96 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3568-radxa-e25.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3568-radxa-e25.dts
> @@ -103,6 +103,10 @@ &combphy1 {
>  	phy-supply = <&vcc3v3_pcie30x1>;
>  };
> 
> +&display_subsystem {
> +	status = "disabled";
> +};
> +
>  &pcie2x1 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pcie20_reset_h>;