RE: [PATCH v3 12/13] arm64: dts: renesas: Add DSI overlay for RZ/G3L SMARC EVK with ADV7535

[Biju Das]

Hi Geert,

Thanks for the feedback.

> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: 08 September 2026 14:25
> Subject: Re: [PATCH v3 12/13] arm64: dts: renesas: Add DSI overlay for RZ/G3L SMARC EVK with ADV7535
> 
> Hi Biju,
> 
> On Wed, 26 Aug 2026 at 19:49, Biju <biju.das.au@xxxxxxxxx> wrote:
> > From: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> >
> > Add a Device Tree overlay (r9a08g046l48-smarc-dsi-adv7535.dtso) for
> > the RZ/G3L (R9A08G046) SMARC EVK board to support DSI-to-HDMI output
> > via the Analog Devices ADV7535 HDMI transmitter.
> >
> > The overlay enables the DSI controller with a 4-lane data path and the
> > display unit (DU/LCDC), and configures the ADV7535 on I2C2.
> >
> > Update the Makefile to build the overlay as both a standalone .dtbo
> > and a composite .dtb (base DTB + overlay).
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/renesas/r9a08g046l48-smarc-dsi-adv7535.dtso
> > @@ -0,0 +1,95 @@
> > +// SPDX-License-Identifier: GPL-2.0
> 
> Any chance this can be "(GPL-2.0-only OR BSD-2-Clause)", like all already existing
> arch/arm64/boot/dts/renesas/r9a08g046* files (and arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi)?

OK will fix.
> 
> 
> > +/*
> > + * Device Tree overlay for the RZ/G3L SMARC EVK with ADV7535
> > + * connected to DSI and LCDC enabled.
> > + *
> > + * Copyright (C) 2026 Renesas Electronics Corp.
> > + */
> > +
> > +/dts-v1/;
> > +/plugin/;
> > +
> > +#include <dt-bindings/gpio/gpio.h>
> > +#include <dt-bindings/interrupt-controller/arm-gic.h>
> > +#include <dt-bindings/pinctrl/renesas,r9a08g046-pinctrl.h>
> > +
> > +&{/} {
> > +       osc1: cec-clock {
> > +               compatible = "fixed-clock";
> > +               #clock-cells = <0>;
> > +               clock-frequency = <12000000>;
> > +       };
> > +
> > +       dsi-to-hdmi-out {
> > +               compatible = "hdmi-connector";
> > +               type = "d";
> 
> No ddc-i2c-bus?
> 
> > +
> > +               port {
> > +                       dsi_to_hdmi_con_out: endpoint {
> > +                               remote-endpoint = <&adv7535_out>;
> > +                       };
> > +               };
> > +       };
> > +};
> > +
> > +&dsi {
> > +       status = "okay";
> > +
> > +       ports {
> > +               #address-cells = <1>;
> > +               #size-cells = <0>;
> > +
> > +               port@1 {
> > +                       reg = <1>;
> > +                       dsi_out: endpoint {
> > +                               data-lanes = <1 2 3 4>;
> > +                               remote-endpoint = <&adv7535_in>;
> > +                       };
> > +               };
> > +       };
> > +};
> > +
> > +&du {
> > +       status = "okay";
> > +};
> > +
> > +&i2c2 {
> > +       #address-cells = <1>;
> > +       #size-cells = <0>;
> > +
> > +       adv7535: hdmi@3d {
> > +               compatible = "adi,adv7535";
> > +               reg = <0x3d>;
> > +
> > +               interrupts-extended = <&pinctrl RZG3L_GPIO(K, 3) IRQ_TYPE_EDGE_FALLING>;
> > +               clocks = <&osc1>;
> > +               clock-names = "cec";
> > +               avdd-supply = <&reg_1p8v>;
> > +               dvdd-supply = <&reg_1p8v>;
> > +               pvdd-supply = <&reg_1p8v>;
> > +               a2vdd-supply = <&reg_1p8v>;
> > +               v3p3-supply = <&reg_3p3v>;
> > +               v1p2-supply = <&reg_1p8v>;
> > +
> > +               adi,dsi-lanes = <4>;
> > +
> > +               ports {
> > +                       #address-cells = <1>;
> > +                       #size-cells = <0>;
> > +
> > +                       port@0 {
> > +                               reg = <0>;
> > +                               adv7535_in: endpoint {
> > +                                       remote-endpoint = <&dsi_out>;
> > +                               };
> > +                       };
> > +
> > +                       port@1 {
> > +                               reg = <1>;
> > +                               adv7535_out: endpoint {
> > +                                       remote-endpoint = <&dsi_to_hdmi_con_out>;
> > +                               };
> > +                       };
> 
> No audio?

Ok, Will add HDMI audio support.

Cheers,
Biju