[PATCH v2 2/8] sound: soc: sh: Add RZ/G2L SSIF-2 driver

[Biju Das]

Hi Geert,

Thanks for the feedback.

> Subject: Re: [PATCH v2 2/8] sound: soc: sh: Add RZ/G2L SSIF-2 driver
> 
> Hi Biju,
> 
> On Mon, Jul 19, 2021 at 3:40 PM Biju Das <biju.das.jz at bp.renesas.com>
> wrote:
> > Add serial sound interface(SSIF-2) driver support for RZ/G2L SoC.
> >
> > Based on the work done by Chris Brandt for RZ/A SSI driver.
> >
> > Signed-off-by: Biju Das <biju.das.jz at bp.renesas.com>
> > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj at bp.renesas.com>
> 
> Thanks for your patch!
> 
> > --- a/sound/soc/sh/Kconfig
> > +++ b/sound/soc/sh/Kconfig
> > @@ -45,6 +45,16 @@ config SND_SOC_RCAR
> >         help
> >           This option enables R-Car SRU/SCU/SSIU/SSI sound support
> >
> > +config SND_SOC_RZ
> > +       tristate "RZ/G2L series SSIF-2 support"
> > +       depends on OF || COMPILE_TEST
> 
> ARCH_R9A07G044 implies OF
> 
> > +       depends on ARCH_R9A07G044
> 
> Is there any hard compile-time dependency on ARCH_R9A07G044?
> 
> Perhaps you meant
> 
>     depends on ARCH_R9A07G044 || COMPILE_TEST

OK, Will update this in the next version.

Regards,
Biju

> 
> ?
> 
> > +       select SND_SIMPLE_CARD
> > +       help
> > +         This option enables ASoC sound support for the RZ/G2L MPUs.
> > +         The simple-audio-card driver and the RZ/G2L built-in serial
> > +         sound interface (SSIF-2) driver are used.
> > +
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at 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