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

["Mukunda,Vijendar" <vijendar.mukunda@amd.com>]

On 17/03/26 09:30, Mukunda,Vijendar wrote:
> On 16/03/26 21:16, Hasun Park wrote:
>>> 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.
> Hi Park, There is a confusion. We don't have a platform to cross-check.  Could
> you please double confirm,
> when you record audio stream from UI, which capture PCM device being used ?  Or
> else you can do
> command line testing using arecord and confirm the pcm device.
>
> As per our CE team, the platform you have mentioned uses different hardware
> configuration.
> Could you please double confirm exact model details?  Share us dmidecode and
> acpidump.
> Share us aplay -l and arecord -l listing.
 
Got the confirmation on hw configuration being used for this ASUS platform from
our CE team.
ACP PDM DMIC being used for this platform. 


>
>