Re: [PATCH] ALSA: hda/hdmi: let new platforms assign the pcm slot dynamically

[Hui Wang <hui.wang@canonical.com>]

On 2/23/21 9:09 PM, Takashi Iwai wrote:
> On Tue, 23 Feb 2021 13:22:05 +0100,
> Hui Wang wrote:
>> If the platform set the dyn_pcm_assign to true, it will call
>> hdmi_find_pcm_slot() to find a pcm slot when hdmi/dp monitor is
>> connected and need to create a pcm.
>>
>> So far only intel_hsw_common_init() and patch_nvhdmi() set the
>> dyn_pcm_assign to true, here we let tgl platforms assign the pcm slot
>> dynamically first, if the driver runs for a period of time and there
>> is no regression reported, we could set no_fixed_assgin to true in
>> the intel_hsw_common_init(), and then set it to true in the
>> patch_nvhdmi().
>>
>> This change comes from the discussion between Takashi and
>> Kai Vehmanen. Please refer to:
>> https://github.com/alsa-project/alsa-lib/pull/118
>>
>> Signed-off-by: Hui Wang <hui.wang@canonical.com>
> As this would "fix" actual use cases, I'd love to merge this for 5.12,
> but of course it needs to be verified beforehand.
>
> So this was actually tested in your side, right?
>
I tested it on TGL machines with Ubuntu,  both sof and legacy hda, so 
far there is no regression.

Regards,

Hui.

> thanks,
>
> Takashi
>