Hi Biju, On Mon, Apr 24, 2023 at 6:39 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > Enable DU and link with DSI on RZ/{G2L,V2L} SMARC EVK. > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> Thanks for your patch! > --- a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi > +++ b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi > @@ -49,6 +49,7 @@ ports { > port@0 { > reg = <0>; > dsi0_in: endpoint { > + remote-endpoint = <&du_out_dsi>; > }; > }; > > @@ -62,6 +63,26 @@ dsi0_out: endpoint { > }; > }; > > +&du { > + status = "okay"; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + du_out_dsi: endpoint { > + remote-endpoint = <&dsi0_in>; > + }; > + }; > + > + port@1 { > + reg = <1>; > + }; Do you need port@1 if it is not connected to anything? If yes, don't the port@ subnodes and reg properties belong in the SoC-specific .dtsi? > + }; > +}; > + > &i2c1 { > adv7535: hdmi@3d { > compatible = "adi,adv7535"; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds