[SRU][Focal][OEM-5.6][Groovy][PATCH 0/1] alsa/sof: support 1 and 3 dmics

[Hui Wang]

On 2020/8/14 下午3:44, Stefan Bader wrote:
> On 14.08.20 07:39, Hui Wang wrote:
>> BugLink: https://bugs.launchpad.net/bugs/1891585
>>
>> [Impact]
>> On a Dell tigerlake laptop, the sof driver fails to detect the dmic,
>> it alwasys report 0 dmics on the machine. So users doesn't have
>> internal mic to use.
> This does not warrant a "high" importance for SRU. I lowered it to "medium".
> Also I saw on the bug report that there are tasks for linux-firmware. Is there
> any hard dependency or ordering restriction between the kernel and the
> linux-firmware change?

OK, it is fine to set it to medium.  There is no hard dependency or 
ordering restriction between kernel and firmware.

If users install the firmware first, it will not bring any impact to old 
kernel since the old kernel doesn't load these 2 new files.

If users install the new kernel on the machines with 0 dmics, 2 dmics or 
4 dmics first, it will not bring any impact too since the kernel will 
not load those 2 new files.

If users install the new kernel on the machines with 1 dmic or 3 dmics 
first, the kernel will try to load the -1ch.tplg or -3ch.tplg, but there 
is no these 2 files since users has not updated linux-firmware, then the 
sof driver will fail. But this is not a regression since the audio never 
work with old kernel .

Thanks,

Hui.

>
> -Stefan
>
>> [Fix]
>> Intel write a patch to fix this issue, if NHLT table returns 1 or
>> 3 dmics, the sof driver will not fail like before, it can support
>> 1 or 3 dmics after applying this patch.
>>
>> [Test Case]
>> Boot the kernel with this patch, and put sof-hda-generic-1ch.tplg in
>> the linux-firmware, the sof driver could detect the dmic and users
>> have a workable internal mic after booting up.
>>
>> [Regression Risk]
>> This patch adds new support for 1 and 3 dmics, it doesn't change
>> anything on existing functions, so the regression posibility is very
>> low.
>>
>>
>> Jaska Uimonen (1):
>>    UBUNTU: SAUCE: ASoC: SOF: intel: hda: support also devices with 1 and
>>      3 dmics
>>
>>   sound/soc/sof/intel/hda.c | 8 +++++++-
>>   1 file changed, 7 insertions(+), 1 deletion(-)
>>
>