[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Thu, 7 Jul 2022 21:35:52 +0530
From: "Reddy, V sujith kumar" <vsujithkumar.reddy@....com>
To: Randy Dunlap <rdunlap@...radead.org>, broonie@...nel.org,
alsa-devel@...a-project.org
Cc: Vijendar.Mukunda@....com, Basavaraj.Hiregoudar@....com,
Sunil-kumar.Dommati@....com, venkataprasad.potturu@....com,
Liam Girdwood <lgirdwood@...il.com>,
Jaroslav Kysela <perex@...ex.cz>,
Takashi Iwai <tiwai@...e.com>,
Ajit Kumar Pandey <AjitKumar.Pandey@....com>,
Geert Uytterhoeven <geert+renesas@...der.be>,
Arnd Bergmann <arnd@...db.de>,
Akihiko Odaki <akihiko.odaki@...il.com>,
Jia-Ju Bai <baijiaju1990@...il.com>,
Syed Saba kareem <ssabakar@....com>,
Yang Yingliang <yangyingliang@...wei.com>,
Meng Tang <tangmeng@...ontech.com>,
Uwe Kleine-König <u.kleine-koenig@...gutronix.de>,
Dan Carpenter <dan.carpenter@...cle.com>,
open list <linux-kernel@...r.kernel.org>
Subject: 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
Powered by blists - more mailing lists