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

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