Re: [PATCH 3/3] ASoC: amd: acp: Add legacy audio driver support for Rembrandt platform

[Reddy, V sujith kumar]

On 6/25/2022 4:34 AM, Randy Dunlap wrote:
> [CAUTION: External Email]
>
> Hi--
>
> On 6/24/22 12:48, V sujith kumar Reddy wrote:
> > diff --git a/sound/soc/amd/acp/Kconfig b/sound/soc/amd/acp/Kconfig
> > index 7e56d2644105..abc089f38b6c 100644
> > --- a/sound/soc/amd/acp/Kconfig
> > +++ b/sound/soc/amd/acp/Kconfig
> > @@ -40,6 +40,17 @@ config SND_AMD_ASOC_RENOIR
> >        help
> >          This option enables Renoir I2S support on AMD platform.
> >
> > +config SND_AMD_ASOC_REMBRANDT
> > +        tristate "AMD ACP ASOC Rembrandt Support"
> > +        select SND_SOC_AMD_ACP_PCM
> > +        select SND_SOC_AMD_ACP_I2S
> > +        select SND_SOC_AMD_ACP_PDM
> > +        depends on X86 && PCI
> > +        help
> > +          This option enables Rembrandt I2S support on AMD platform.
> > +       Say Y if you want to enable AUDIO on Rembrandt
> > +       If unsure select "N".
Done
> Documentation/process/coding-style.rst says:
>
> 10) Kconfig configuration files
> -------------------------------
>
> For all of the Kconfig* configuration files throughout the source tree,
> the indentation is somewhat different.  Lines under a ``config`` definition
> are indented with one tab, while help text is indented an additional two
> spaces.
>
>
> so please indent as expected, and consistently.
>
> --
> ~Randy