On Mon, 5 Dec 2022 at 17:44, Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> wrote: > > On 05/12/2022 18:37, Robert Foss wrote: > > Enable the display subsystem and the dsi0 output for > > the sm8350-hdk board. > > > > Signed-off-by: Robert Foss <robert.foss@xxxxxxxxxx> > > --- > > arch/arm64/boot/dts/qcom/sm8350-hdk.dts | 22 ++++++++++++++++++++++ > > 1 file changed, 22 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/qcom/sm8350-hdk.dts b/arch/arm64/boot/dts/qcom/sm8350-hdk.dts > > index e6deb08c6da0..39462c659c58 100644 > > --- a/arch/arm64/boot/dts/qcom/sm8350-hdk.dts > > +++ b/arch/arm64/boot/dts/qcom/sm8350-hdk.dts > > @@ -213,10 +213,32 @@ &cdsp { > > firmware-name = "qcom/sm8350/cdsp.mbn"; > > }; > > > > +&dispcc { > > + status = "okay"; > > +}; > > + > > +&dsi0 { > > Bjorn suggested using mdss_dsi0 / mdss_dsi0_phy labels for DSI host and > PHY, as it allows us to group them nicely. WDYT? Sounds quite reasonable, fixing it in dts/dtsi/binding. > > > + vdda-supply = <&vreg_l6b_1p2>; > > + status = "okay"; > > +}; > > + > > +&dsi0_phy { > > + vdds-supply = <&vreg_l5b_0p88>; > > + status = "okay"; > > +}; > > + > > &gpi_dma1 { > > status = "okay"; > > }; > > > > +&mdss { > > + status = "okay"; > > +}; > > + > > +&mdss_mdp { > > + status = "okay"; > > +}; > > + > > &mpss { > > status = "okay"; > > firmware-name = "qcom/sm8350/modem.mbn"; > > -- > With best wishes > Dmitry >