linux-kernel - Re: [PATCH v2 1/8] ASoC: amd: Add common framework to support I2S on ACP SOC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 12 Oct 2021 12:47:44 +0530
From:   Ajit Kumar Pandey <AjitKumar.Pandey@....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, Alexander.Deucher@....com,
        Liam Girdwood <lgirdwood@...il.com>,
        Jaroslav Kysela <perex@...ex.cz>,
        Takashi Iwai <tiwai@...e.com>,
        Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>,
        open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/8] ASoC: amd: Add common framework to support I2S on
 ACP SOC



On 10/12/2021 1:46 AM, Randy Dunlap wrote:
> [CAUTION: External Email]
> 
> On 10/11/21 8:44 AM, Ajit Kumar Pandey wrote:
>> We are using legacy way of exposing dais and DMA configuration that
>> requires separate driver modules for various ACP SOC with almost
>> similar hw configuration. Moreover the legacy approach requires
>> separate I2S and DMA module platform devices registration and need
>> machine specific quirk to control various I2S endpoints. Add generic
>> dai driver and platform driver for I2S controller on ACP hw block.
>> This common framework can be used by various ACP platform devices
>> that shares common specs.
>>
>> Signed-off-by: Ajit Kumar Pandey<AjitKumar.Pandey@....com>
>> ---
> 
>> diff --git a/sound/soc/amd/acp/Kconfig b/sound/soc/amd/acp/Kconfig
>> new file mode 100644
>> index 000000000000..6b993074e2c4
>> --- /dev/null
>> +++ b/sound/soc/amd/acp/Kconfig
>> @@ -0,0 +1,18 @@
>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-3-Clause)
>> +# This file is provided under a dual BSD/GPLv2 license. When using or
>> +# redistributing this file, you may do so under either license.
>> +#
>> +# Copyright(c) 2021 Advanced Micro Devices, Inc. All rights reserved.
>> +#
>> +
>> +config SND_SOC_AMD_ACP_COMMON
>> +     tristate "AMD Audio ACP Common support"
>> +     select SND_AMD_ACP_CONFIG
>> +     help
>> +       This option enables ACP support on AMD platform.
>> +
>> +config SND_SOC_AMD_ACP_I2S
>> +     tristate
>> +
>> +config SND_SOC_AMD_ACP_PCM
>> +     tristate
> 
> Hi,
> I don't see anything that tells us what "ACP" is.
> Please add that somewhere -- either in the Kconfig file or
> at least in the commit description.
> 
> thanks.
> 
> -- 
> ~Randy
ACP stands for Audio Coprocessor in AMD platforms, will update
Kconfig text with that in v3 patch chain