Hi Geert,
> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: Friday, August 23, 2024 1:33 PM
> Subject: Re: [PATCH v4 4/4] arm64: dts: renesas: r9a07g043u11-smarc: Enable DU
>
> Hi Biju,
>
> On Thu, Aug 22, 2024 at 6:23 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote:
> > Enable DU and link with the HDMI add-on board connected with the
> > parallel connector on RZ/G2UL SMARC EVK.
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> > ---
> > v3->v4:
> > * Restored ports property in du node and port@0 for DPI interface.
>
> Thanks for the update!
>
> > --- a/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts
> > +++ b/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc.dts
> > @@ -35,4 +35,115 @@
> > / {
> > model = "Renesas SMARC EVK based on r9a07g043u11";
> > compatible = "renesas,smarc-evk", "renesas,r9a07g043u11",
> > "renesas,r9a07g043";
> > +
> > + hdmi-out {
> > + compatible = "hdmi-connector";
>
> [...]
>
> > +&i2c1 {
> > + adv7513: adv7513@39 {
> > + compatible = "adi,adv7513";
>
> The HDMI connector and ADV7513 bridge are not present on the SMARC Carrier board, but on a separate
> board. Hence I think this should be moved to a separate DT overlay .dtso, with a suitable Makefile
> rule to build a DTS for SMARC Carrier + HDMI add-on board.
OK, will send DT overlay in the next version. The only reason for sending like this is VLP package using cip kernel
still don't use DT overlay. So, the backporting will be straight forward for the users of VLP package.
Cheers,
Biju
>
> The actual DT description LGTM.
