[SRU] the speaker doesn't work on Dell sdw sdca machine with kernel-5.13

Bug #1937980 reported by Hui Wang
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
Fix Released
Undecided
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 to 5.13, the mixer names of rt1316 is changed, the current alsa-ucm-conf will fail to initialize that codec, then users can't output sound from internal speaker since the internal speaker is routed to the codec rt1316.

[Fix]
Backport 1 commit (https://github.com/alsa-project/alsa-ucm-conf/pull/109), after applying the commit, the ucm conf file doesn'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]
sudo rm /var/lib/alsa/*; alsactl init; reboot, play the sound from internal speaker, the speaker works well.

[Where problems will occur]
The change only affects the rt1316 codec, so far, that codec only connects to the internal speaker on Dell soundwire sdca audio machine, so it could introduce regression on the internal speaker like users can't output sound via internal speaker anymore on the Dell soundwire sdca audio machine. But this possibility is very low since we already tested the commit on the Dell soundwire sdca audio machine. 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 Focal):
importance: Undecided → High
Changed in alsa-ucm-conf (Ubuntu Hirsute):
importance: Undecided → High
Hui Wang (hui.wang)
tags: added: oem-priority originate-from-1934865 somerville
Hui Wang (hui.wang)
description: updated
Revision history for this message
Hui Wang (hui.wang) wrote :

Re-write this SRU based on the latest ubuntu alsa-ucm-conf, and also integrate another SRU (https://bugs.launchpad.net/ubuntu/+source/alsa-ucm-conf/+bug/1940788)

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

Hi sponsor team, This is the debdiff for hirsute. thx.

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

Hi sponsor team, This is the debdiff for impish. thx.

Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: New → In Progress
Changed in alsa-ucm-conf (Ubuntu Impish):
status: New → In Progress
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)
status: New → In Progress
Revision history for this message
Hui Wang (hui.wang) wrote :

Hi sponsor team, This is the debdiff for focal. thx.

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

Thanks, uploaded to impish now. A small side comment since that's the first time I notice that but somehow you end up adding empty lines between debian/patches/series entries

Changed in alsa-ucm-conf (Ubuntu Impish):
status: In Progress → Fix Committed
Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: In Progress → Fix Committed
Revision history for this message
Sebastien Bacher (seb128) wrote :

Upload to hirsute and focal as well now

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

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

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

  * d/p/0006-sof-soundwire-rt1316-DAC-L-and-R-is-replaced-by-DAC.patch
    The mixer name is changed in the codec driver rt1316-sdw.c, so need
    to handle the new mixer name.
    https://github.com/alsa-project/alsa-ucm-conf/pull/109
    (LP: #1937980)
  * d/p/0007-sof-hda-dsp-Set-Capture-Switch-on-in-the-BootSequenc.patch
    The input volume of the external microphone is 0 by default for all
    machines with sof audio driver, that is because the capture switch
    is set to off by default, need to set it to on in the ucm.
    https://github.com/alsa-project/alsa-ucm-conf/pull/107
    (LP: #1940788)

 -- Hui Wang <email address hidden> Mon, 23 Aug 2021 10:36:38 +0800

Changed in alsa-ucm-conf (Ubuntu Impish):
status: Fix Committed → Fix Released
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.3 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.

tags: added: verification-needed verification-needed-hirsute
tags: added: verification-needed-focal
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.10 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.

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

For hirsute:

On the Dell soundwire-sdca audio machine,
enable hirsute-proposed in the /etc/apt/sources.list
apt-get update
apt install alsa-ucm-conf, now the alsa-ucm-conf all 1.2.4-2ubuntu1.3 is installed.
install linux-image-unsigned-5.13.0-oem kernel
sudo rm /var/lib/alsa/*; rm ~/.config/pulse/*;alsactl init; reboot machine
After the system boots up, play sound through internal speaker, the sound could be outputted via speaker successfully.

Verified done for hirsute.

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

For focal:

On the Dell soundwire-sdca audio machine,
enable focal-proposed in the /etc/apt/sources.list
apt-get update
apt install alsa-ucm-conf, now the alsa-ucm-conf all 1.2.2-1ubuntu0.10 is installed.
install linux-image-unsigned-5.13.0-oem kernel
sudo rm /var/lib/alsa/*; rm ~/.config/pulse/*;alsactl init; reboot machine
After the system boots up, play sound through internal speaker, the sound could be outputted via speaker successfully.

Verification done for focal.

tags: added: verification-done verification-done-focal verification-done-hirsute
removed: verification-needed verification-needed-focal verification-needed-hirsute
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

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

  * d/p/0005-sof-soundwire-rt1316-DAC-L-and-R-is-replaced-by-DAC.patch
    The mixer name is changed in the codec driver rt1316-sdw.c, so need
    to handle the new mixer name.
    https://github.com/alsa-project/alsa-ucm-conf/pull/109
    (LP: #1937980)
  * d/p/0006-sof-hda-dsp-Set-Capture-Switch-on-in-the-BootSequenc.patch
    The input volume of the external microphone is 0 by default for all
    machines with sof audio driver, that is because the capture switch
    is set to off by default, need to set it to on in the ucm.
    https://github.com/alsa-project/alsa-ucm-conf/pull/107
    (LP: #1940788)

 -- Hui Wang <email address hidden> Mon, 23 Aug 2021 10:05:50 +0800

Changed in alsa-ucm-conf (Ubuntu Hirsute):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) 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.10

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

  * d/p/0031-sof-soundwire-rt1316-DAC-L-and-R-is-replaced-by-DAC.patch
    The mixer name is changed in the codec driver rt1316-sdw.c, so need
    to handle the new mixer name.
    https://github.com/alsa-project/alsa-ucm-conf/pull/109
    (LP: #1937980)
  * d/p/0032-sof-hda-dsp-Set-Capture-Switch-on-in-the-BootSequenc.patch
    The input volume of the external microphone is 0 by default for all
    machines with sof audio driver, that is because the capture switch
    is set to off by default, need to set it to on in the ucm.
    https://github.com/alsa-project/alsa-ucm-conf/pull/107
    (LP: #1940788)

 -- Hui Wang <email address hidden> Mon, 23 Aug 2021 11:00:48 +0800

Changed in alsa-ucm-conf (Ubuntu Focal):
status: Fix Committed → Fix Released
Changed in hwe-next:
status: New → Fix Released
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.