Re: [PATCH 1/2] ASoC: amd: acp: update dmic_num logic for acp pdm dmic

From: Mukunda,Vijendar

Date: Mon Mar 30 2026 - 11:16:39 EST


On 30/03/26 19:13, Mark Brown wrote:
> On Mon, Mar 30, 2026 at 12:50:27PM +0530, Vijendar Mukunda wrote:
>
>> +++ b/sound/soc/amd/acp/acp-sdw-legacy-mach.c
>> @@ -535,11 +535,11 @@ static int mc_probe(struct platform_device *pdev)
>> " cfg-amp:%d", amp_num);
>> if (!card->components)
>> return -ENOMEM;
>> - if (mach->mach_params.dmic_num) {
>> + if (soc_sdw_quirk & ASOC_SDW_ACP_DMIC) {
> soc_card_dai_links_create() (which could perhaps use better namespacing)
> checks both dmic_num and the quirk. This doesn't seem properly joined
> up (and perhaps the code shouldn't be as spread out as it is)?
Need to drop the mach_params->dmic_num check in soc_card_dai_links_create().
Will push a separate patch for the same.