RE: [PATCH v2 24/24] arm64: dts: renesas: r9a09g047e57-smarc: add DA7212 audio codec support

[John Madieu]

Hi Geert,

Thank you for your review.

> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: Wednesday, April 8, 2026 11:41 AM
> To: John Madieu <john.madieu.xa@xxxxxxxxxxxxxx>
> Subject: Re: [PATCH v2 24/24] arm64: dts: renesas: r9a09g047e57-smarc: add
> DA7212 audio codec support
> 
> Hi John,
> 
> On Thu, 2 Apr 2026 at 11:10, John Madieu <john.madieu.xa@xxxxxxxxxxxxxx>
> wrote:
> > RZ/G3E SMARC board has a DA7212 audio codec connected via I2C1 for
> > sound input/output using SSI3/SSI4 where:
> >
> >  - The codec receives its master clock from the Versa3 clock
> >    generator present on the SoM
> >  - SSI4 shares clock pins with SSI3 to provide a separate data
> >    line for full-duplex audio capture.
> >
> > Enable audio support on RZ/G3E SMARC2 EVK boards with a DA7212 audio
> codec.
> >
> > Signed-off-by: John Madieu <john.madieu.xa@xxxxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> > --- a/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> > +++ b/arch/arm64/boot/dts/renesas/r9a09g047e57-smarc.dts
> 
> > @@ -280,6 +358,42 @@ &sdhi1 {
> >         vqmmc-supply = <&vqmmc_sd1_pvdd>;  };
> >
> > +&snd_rzg3e {
> 
> Please preserve sort order (alphabetical, by label).

Noted for v3.

Regards,
John

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
> 
> 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