Re: [PATCH 4/4] ARM: dts: qcom: apq8074-dragonboard: enable DSI panel

[Konrad Dybcio <konrad.dybcio@linaro.org>]

On 7.05.2023 21:07, Dmitry Baryshkov wrote:
> Enable MDSS, GPU and DSI panel output on the APQ8074 dragonboard.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  .../arm/boot/dts/qcom-apq8074-dragonboard.dts | 51 +++++++++++++++++++
>  1 file changed, 51 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts
> index c893afc00eb4..72f7e09a5bbf 100644
> --- a/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts
> +++ b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts
> @@ -48,6 +48,57 @@ eeprom: eeprom@52 {
>  	};
>  };
>  
> +&dsi0 {
old junk could use some mdss_ prefixing to keep the nodes together

Could you please take care of that?

> +	vdda-supply = <&pm8941_l2>;
> +	vdd-supply = <&pm8941_l22>;
> +	vddio-supply = <&pm8941_l12>;
> +
> +	status = "okay";
> +
> +	panel: panel@0 {
> +		compatible = "sharp,ls043t1le01-qhd";
> +		reg = <0>;
> +
> +		avdd-supply = <&pm8941_l22>;
> +		backlight = <&pm8941_wled>;
> +		reset-gpios = <&pm8941_gpios 19 GPIO_ACTIVE_HIGH>;
> +
> +		port {
> +			panel_in: endpoint {
> +				remote-endpoint = <&dsi0_out>;
> +			};
> +		};
> +	};
> +};
> +
> +&dsi0_out {
> +	remote-endpoint = <&panel_in>;
> +	data-lanes = <0 1 2 3>;
> +};
> +
> +&dsi0_phy {
> +	status = "okay";
> +
> +	vddio-supply = <&pm8941_l12>;
status last

Konrad
> +};
> +
> +&gpu {
> +	status = "okay";
> +};
> +
> +&mdss {
> +	status = "okay";
> +};
> +
> +&pm8941_wled {
> +	qcom,cs-out;
> +	qcom,switching-freq = <3200>;
> +	qcom,ovp = <32>;
> +	qcom,num-strings = <1>;
> +
> +	status = "okay";
> +};
> +
>  &remoteproc_adsp {
>  	cx-supply = <&pm8841_s2>;
>