Re: [PATCH 3/3] ASoC: amd: acp: add ASUS HN7306EA quirk for legacy SDW machine

[Hasun Park <hasunpark@gmail.com>]

>On 11/03/26 23:02, Mukunda,Vijendar wrote:
>> On 11/03/26 22:51, Mark Brown wrote:
>>> On Mon, Mar 09, 2026 at 11:07:30AM +0530, Mukunda,Vijendar wrote:
>>>> On 08/03/26 20:46, hasunpark@gmail.com wrote:
>>>>> From: Hasun Park <hasunpark@gmail.com>
>>>>> Add a DMI quirk entry for ASUS HN7306EA in the ACP SoundWire legacy
>>>>> machine driver.
>>>>> Set driver_data to RT711_JD1 | ASOC_SDW_ACP_DMIC for this board so the
>>>>> platform-specific jack/DMIC quirk path is selected.
>>>> As RT721 comes up with UAJ + DMIC, then why we need ACP PDM DMIC in this case?
>>>> If you are using RT721 (UAJ + DMIC combination) below quirk is not required.
>>> Presumably nothing stops the system integrator wiring a DMIC directly to
>>> the SoC even if the CODEC can also use it?  It doesn't seem an obviously
>>> silly configuration.
>> On this platform Soundwire Codec RT721 already comes up with DMIC
>> endpoint. As per our understanding current soundwire topology points to
>> use RT721 + TI codecs. ACP DMIC is not being used.
>> If we want to use ACP PDM DMIC where DMIC's are connected to SOC and 
>> routed to ACP PDM controller instead of Soundwire Codec (Which supports
>> DMIC path), then ACP_PDM_DMIC quirk should be enabled.
>
>We have cross-checked hw configuration. This platform uses DMIC connected
>to Soundwire codec(ALC721). This patch should be dropped.

Thanks for double-checking the hw configs.

I'm not sure to drop this patch because when i test it dmic only works when i add quirk and enable BE link to ACP.

But one thing i have to update is both jack and dmic both works fine with ASOC_SDW_ACP_DMIC not RT711_JD1|ASOC_SDW_ACP_DMIC.