mbox series

[SRU,linux-firmware,Focal/Groovy,0/1] sof-firmware: support 1 and 3 dmics

Message ID 20200814055425.11316-1-hui.wang@canonical.com
Headers show
Series sof-firmware: support 1 and 3 dmics | expand

Message

Hui Wang Aug. 14, 2020, 5:54 a.m. UTC
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.

[Fix]
Intel provides 2 new topology files to us, they could support 1
dmic and 3 dmics.

[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 is adding 2 new topology files, not changing anything on
current sof-firmware, so the regression posibility is very low.


Hui Wang (1):
  UBUNTU: SAUCE: sof-firmware: add 1ch and 3ch topology files

 WHENCE.ubuntu                           |   2 ++
 intel/sof-tplg/sof-hda-generic-1ch.tplg | Bin 0 -> 48091 bytes
 intel/sof-tplg/sof-hda-generic-3ch.tplg | Bin 0 -> 48227 bytes
 3 files changed, 2 insertions(+)
 create mode 100644 intel/sof-tplg/sof-hda-generic-1ch.tplg
 create mode 100644 intel/sof-tplg/sof-hda-generic-3ch.tplg

Comments

Seth Forshee Aug. 18, 2020, 8:21 p.m. UTC | #1
On Fri, Aug 14, 2020 at 01:54:24PM +0800, 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.
> 
> [Fix]
> Intel provides 2 new topology files to us, they could support 1
> dmic and 3 dmics.
> 
> [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 is adding 2 new topology files, not changing anything on
> current sof-firmware, so the regression posibility is very low.

Just to confirm, as we've had problems with sof firmware files in the
past due to signing problems. Have you confirmed that these files have
been signed appropriately, if they require signatures?

Thanks,
Seth
Hui Wang Aug. 19, 2020, 6:27 a.m. UTC | #2
On 2020/8/19 上午4:21, Seth Forshee wrote:
> On Fri, Aug 14, 2020 at 01:54:24PM +0800, 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.
>>
>> [Fix]
>> Intel provides 2 new topology files to us, they could support 1
>> dmic and 3 dmics.
>>
>> [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 is adding 2 new topology files, not changing anything on
>> current sof-firmware, so the regression posibility is very low.
> Just to confirm, as we've had problems with sof firmware files in the
> past due to signing problems. Have you confirmed that these files have
> been signed appropriately, if they require signatures?

Confirmed with Intel, all topology files (*.tplg) are not signed by any 
keys. Only firmware files (*.ri) need to be signed.

Thanks,

Hui.


>
> Thanks,
> Seth
Seth Forshee Aug. 19, 2020, 5:58 p.m. UTC | #3
On Fri, Aug 14, 2020 at 01:54:24PM +0800, 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.
> 
> [Fix]
> Intel provides 2 new topology files to us, they could support 1
> dmic and 3 dmics.
> 
> [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 is adding 2 new topology files, not changing anything on
> current sof-firmware, so the regression posibility is very low.

Applied for focal and groovy, thanks!