[SRU] The internal mic of Dell Precision 5750 can't record sound

Bug #1936009 reported by Hui Wang
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
Invalid
Undecided
Unassigned
OEM Priority Project
Fix Released
Critical
Unassigned
alsa-ucm-conf (Ubuntu)
Fix Released
High
Hui Wang
Focal
Fix Released
High
Hui Wang
Hirsute
Fix Released
High
Hui Wang
Impish
Fix Released
High
Hui Wang

Bug Description

[Impact]
After upgrade the linux kernel, the mixer names of rt715 is changed, the current alsa-ucm-conf will fail to initialize that codec, then users can't record sound from internal mic since the internal mic is routed to the codec rt715.

[Fix]
Backport 3 upstream commits, after applying these 3 commits, the ucm conf file dosn't initialize the codec with hardcoded mixer names anymore, it will check the existence of the mixer name first, then conditionally initialize the codec.

[Test]
run 'alsactl init 1' on the precision 5750 with old and new kernel, there is no error logs, use the internal mic to record sound, it could record successfully.

[Where problems will occur]
The change only affects the rt715 codec, so far, that codec only connects to the internal mic on Dell soundwire audio machines, so it could introduce regression on the internal mic like users can't record sound via internal mic anymore on the Dell soundwire audio machines. But this possibility is very low since we already tested these 3 commits on different Dell soundwire audio machines (latitude and precision). We tested it with old kernel and new kernel, all worked well.

Hui Wang (hui.wang)
Changed in alsa-ucm-conf (Ubuntu):
importance: Undecided → High
Changed in alsa-ucm-conf (Ubuntu Hirsute):
importance: Undecided → High
Changed in alsa-ucm-conf (Ubuntu Focal):
importance: Undecided → High
status: New → In Progress
Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: New → In Progress
Changed in alsa-ucm-conf (Ubuntu Impish):
status: New → In Progress
tags: added: oem-priority originate-from-1935805 somerville
Revision history for this message
Hui Wang (hui.wang) wrote :

This is the debdiff for impish.

thx.

description: updated
Revision history for this message
Hui Wang (hui.wang) wrote :

This is the debdiff for focal.

thx.

Rex Tsai (chihchun)
Changed in oem-priority:
importance: Undecided → Critical
Hui Wang (hui.wang)
summary: - The internal mic of Dell Precision 5750 can't record sound
+ [SRU] The internal mic of Dell Precision 5750 can't record sound
Revision history for this message
Hui Wang (hui.wang) wrote :

update the debdiff for hirsute. When uploading to hirsute, please use this one.

thx.

Changed in alsa-ucm-conf (Ubuntu Impish):
status: In Progress → Fix Committed
Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: In Progress → Fix Committed
Changed in alsa-ucm-conf (Ubuntu Focal):
status: In Progress → Fix Committed
Changed in alsa-ucm-conf (Ubuntu Hirsute):
assignee: nobody → Hui Wang (hui.wang)
Changed in alsa-ucm-conf (Ubuntu Focal):
assignee: nobody → Hui Wang (hui.wang)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package alsa-ucm-conf - 1.2.4-2ubuntu3

---------------
alsa-ucm-conf (1.2.4-2ubuntu3) impish; urgency=medium

  * d/p/0003-rt715-don-t-initialize-obsolete-ADC-controls.patch
  * d/p/0004-sof-soundwire-rt715-use-the-hw-volume-control.patch
  * d/p/0005-rt715-add-the-missing-sof-prefix-to-the-hw-control-n.patch
    Conditionally initialize the codec rt715 according to mixer names
    (LP: #1936009)

 -- Hui Wang <email address hidden> Wed, 14 Jul 2021 08:03:50 +0800

Changed in alsa-ucm-conf (Ubuntu Impish):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

The status should be Fix Committed only when accepted into -proposed, reverting statuses for now o/

Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: Fix Committed → In Progress
Changed in alsa-ucm-conf (Ubuntu Focal):
status: Fix Committed → In Progress
Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-hirsute
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Please test proposed package

Hello Hui, or anyone else affected,

Accepted alsa-ucm-conf into hirsute-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/alsa-ucm-conf/1.2.4-2ubuntu1.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-hirsute to verification-done-hirsute. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-hirsute. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Hello Hui, or anyone else affected,

Accepted alsa-ucm-conf into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/alsa-ucm-conf/1.2.2-1ubuntu0.9 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in alsa-ucm-conf (Ubuntu Focal):
status: In Progress → Fix Committed
tags: added: verification-needed-focal
Revision history for this message
Hui Wang (hui.wang) wrote :

for hirsute:

on the Dell Precision 5760
enabled hirsute-proposed in the /etc/apt/sources.list
sudo apt-get update
sudo apt install alsa-ucm-conf, now the alsa-ucm-conf-1.2.4-2ubuntu1.2 is installed.
rm ~/.config/pulse/*;sudo rm -rf /var/lib/alsa/*; echo b | sudo tee /proc/sysrq-trigger

After the system boot up, check the internal mic, could record sound successfully.

verified done on hirsute.

tags: added: verification-done-hirsute
removed: verification-needed-hirsute
Revision history for this message
Hui Wang (hui.wang) wrote :

for focal:

on the Dell Precision 5760
enabled focal-proposed in the /etc/apt/sources.list
sudo apt-get update
sudo apt install alsa-ucm-conf, now the alsa-ucm-conf-1.1.2-1ubuntu0.9 is installed.
rm ~/.config/pulse/*;sudo rm -rf /var/lib/alsa/*; echo b | sudo tee /proc/sysrq-trigger

After the system boot up, check the internal mic, could record sound successfully.

verified done on focal.

tags: added: verification-done verification-done-focal
removed: verification-needed verification-needed-focal
Revision history for this message
Brian Murray (brian-murray) wrote :

Comments #9 and #10 don't indicate which versions of the kernel were tested while the description specifically mentions testing with "the old and new kernel". Was this tested with both kernels?

Revision history for this message
Hui Wang (hui.wang) wrote :

Oh, forgot to add testing info of 2 kernels:

With hirsute: the old kernel is linux-5.10.0-1027-oem, the new kernel is linux-5.11.0-26.28-generic.

With focal: the old kernel is linux-5.10.0-1027-oem, the new kernel is linux-5.10.0-1042-oem.

The internal mic could work well with all kernels.

Revision history for this message
Chris Halse Rogers (raof) wrote : Update Released

The verification of the Stable Release Update for alsa-ucm-conf has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package alsa-ucm-conf - 1.2.2-1ubuntu0.9

---------------
alsa-ucm-conf (1.2.2-1ubuntu0.9) focal; urgency=medium

  * d/p/0028-rt715-don-t-initialize-obsolete-ADC-controls.patch
  * d/p/0029-sof-soundwire-rt715-use-the-hw-volume-control.patch
  * d/p/0030-rt715-add-the-missing-sof-prefix-to-the-hw-control-n.patch
    Conditionally initialize the codec rt715 according to mixer names
    (LP: #1936009)

 -- Hui Wang <email address hidden> Wed, 14 Jul 2021 08:59:30 +0800

Changed in alsa-ucm-conf (Ubuntu Focal):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package alsa-ucm-conf - 1.2.4-2ubuntu1.2

---------------
alsa-ucm-conf (1.2.4-2ubuntu1.2) hirsute; urgency=medium

  * d/p/0002-rt715-don-t-initialize-obsolete-ADC-controls.patch
  * d/p/0003-sof-soundwire-rt715-use-the-hw-volume-control.patch
  * d/p/0004-rt715-add-the-missing-sof-prefix-to-the-hw-control-n.patch
    Conditionally initialize the codec rt715 according to mixer names
    (LP: #1936009)

 -- Hui Wang <email address hidden> Mon, 26 Jul 2021 12:49:55 +0800

Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: Fix Committed → Fix Released
Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

Per what I know, this can be closed.

Changed in oem-priority:
status: New → Fix Released
Changed in hwe-next:
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.