Re: [PATCH v4 4/4] arm64: dts: renesas: r9a07g043u11-smarc: Enable DU

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

The actual DT description LGTM.

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





[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux