The rear panel of Lenovo P620 doesn't support more than one audio device at the same time

Bug #1902464 reported by jeremyszu
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
Fix Released
Undecided
Unassigned
OEM Priority Project
Fix Released
Critical
jeremyszu
alsa-ucm-conf (Ubuntu)
Fix Released
Undecided
Unassigned
Focal
Fix Released
Undecided
Unassigned
pulseaudio (Ubuntu)
Fix Released
Undecided
Kai-Heng Feng
Focal
Fix Released
Undecided
Unassigned

Bug Description

== SRU Justification ==
[Impact]
On P620, only single port can work on rear panel. For example, when the Line-Out is plugged, the Mic won't work anymore.

[Fix]
Use upstream version of UCM to handle port priority correctly, instead of separate ports into different profiles.

[Test]
Once the UCM is in place, all three ports of rear panel work correctly.

[Where problems could occur]
UCM is not a static thing - it's actually interpreted differently at higher level. So any change in userspace daemons other than PulseAudio may not like the change and can interpret the UCM in another way.

== Original Bug Report ==
After backporting following patches from PA and alsa-ucm-conf and then it works.

https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/290
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/354
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/355

https://github.com/alsa-project/alsa-ucm-conf/tree/master/ucm2/USB-Audio [landed by aa74f4c12eefcc98582572d2fc48982cf7478b51]

Here is the test PPA:
https://launchpad.net/~os369510/+archive/ubuntu/oem-package-test

Since the upstream not yet accepted those patches, the regression potential may quite high.

jeremyszu (os369510)
tags: added: oem-priority originate-from-1900094 sutton
jeremyszu (os369510)
Changed in oem-priority:
assignee: nobody → jeremyszu (os369510)
status: New → Confirmed
jeremyszu (os369510)
summary: The rear panel of Lenovo P620 doesn't support more than one audio device
- in at the same time
+ at the same time
Changed in pulseaudio (Ubuntu):
assignee: nobody → Hui Wang (hui.wang)
Changed in alsa-ucm-conf (Ubuntu):
assignee: nobody → Hui Wang (hui.wang)
Revision history for this message
Sebastien Bacher (seb128) wrote :

2 of the pulseaudio patches landed upstream now, maybe time to nag on the first one and then we should be able to get the fixes in Hirsute at least?

Revision history for this message
jeremyszu (os369510) wrote :

Since alsa-ucm-conf is updated by following the upstream version.

```
commit aa74f4c12eefcc98582572d2fc48982cf7478b51 (tag: pkg/import/1.2.4-1)
Author: Jordi Mallach <email address hidden>
Date: Mon Dec 7 22:13:23 2020 +0100

    1.2.4-1 (patches unapplied)

    Imported using git-ubuntu import.

Notes (changelog):
      * New upstream release.
      * Update Standards-Version to 4.5.1, with no changes needed.
```

Above commit already brought https://github.com/alsa-project/alsa-ucm-conf/tree/master/ucm2/USB-Audio for P620 parts.

Changed in alsa-ucm-conf (Ubuntu):
assignee: Hui Wang (hui.wang) → nobody
status: New → Fix Released
description: updated
Revision history for this message
jeremyszu (os369510) wrote :

@Seb,

do you think I can work on SRU

https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/354
https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/355

these two patches? since the #290 patch seems like still need a long time discussion on upstream and only this patch have high potential regression risk.

so far I need to backport these three patches and maintain on our archive. If we could reduce the scope then it will be helpful in current situation.

Revision history for this message
jeremyszu (os369510) wrote :

Although comment#2 is merged, this issue still needs PA corresponding (https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/290).

Therefore, we will deliver a special alsa-ucm-conf on our oem-archive as short-term solution and leave it be replaced after this ticket gets the merge.

Revision history for this message
Sebastien Bacher (seb128) wrote :

@Jeremyszu, backporting the patches that landed upstream in Ubuntu is alright if it helps you

Revision history for this message
jeremyszu (os369510) wrote :

@Seb,

I'll work on these two patches comment#3 mentioned in the other bug.
https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1929371

Let's leave this ticket to track https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/290

Revision history for this message
Sebastien Bacher (seb128) wrote :
Hui Wang (hui.wang)
Changed in pulseaudio (Ubuntu):
status: New → Incomplete
Revision history for this message
jeremyszu (os369510) wrote :
Alex Tu (alextu)
Changed in oem-priority:
importance: Undecided → Critical
Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

We only need to update UCM to solve this issue for Lenovo P620.

Changed in alsa-ucm-conf (Ubuntu Focal):
status: New → Confirmed
Changed in pulseaudio (Ubuntu Focal):
status: New → Fix Released
Changed in pulseaudio (Ubuntu):
status: Incomplete → Fix Released
Hui Wang (hui.wang)
Changed in pulseaudio (Ubuntu):
assignee: Hui Wang (hui.wang) → Kai-Heng Feng (kaihengfeng)
Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

alsa-ucm-conf debdiff for Focal

Revision history for this message
David Ober (dober60) wrote :

Following instruction in forwarded email after appling the alsa-ucm-conf=1.2.2-1ubuntu0.12sutton1 package the rear microphone jack works correctly

description: updated
Revision history for this message
Timo Aaltonen (tjaalton) wrote : Please test proposed package

Hello jeremyszu, 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.13 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: Confirmed → Fix Committed
tags: added: verification-needed verification-needed-focal
Revision history for this message
Kai-Heng Feng (kaihengfeng) wrote :

I enabled proposed and upgrade alsa-ucm-conf, then reboot the system.

I plugged a microphone to the rear panel, made sure it works, then I plugged a speaker to line-out, and made sure both speaker and mic are working. So the original issue is solved by this SRU.

tags: added: verification-done verification-done-focal
removed: verification-needed verification-needed-focal
Revision history for this message
Łukasz Zemczak (sil2100) 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.13

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

  * d/p/0035-Revert-PATCH-ucm2-USB-Audio-Add-more-support-to-Leno.patch
    d/p/0036-Revert-PATCH-Add-support-for-Lenovo-ThinkStation-P62.patch
    d/p/0037-Add-support-for-Lenovo-ThinkStation-P620-Main-Audio.patch
    d/p/0038-ucm2-USB-Audio-Add-support-for-Lenovo-ThinkStation-P.patch
    d/p/0039-ucm2-USB-Audio-Add-CaptureMixerElem-for-Lenovo-Think.patch
    Replace Lenovo P620 UCM files with upstream version to make port selection
    in PulseAudio work properly. (LP: #1902464)

 -- Kai-Heng Feng <email address hidden> Tue, 22 Mar 2022 20:43:51 +0800

Changed in alsa-ucm-conf (Ubuntu Focal):
status: Fix Committed → Fix Released
jeremyszu (os369510)
Changed in oem-priority:
status: Confirmed → Fix Released
Timo Aaltonen (tjaalton)
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.