The analogue audio does not work on the Dell USB Dock

Bug #1718824 reported by Hui Wang on 2017-09-22
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
HWE Next
Critical
Hui Wang
pulseaudio (Ubuntu)
High
Hui Wang
Nominated for Xenial by Daniel van Vugt

Bug Description

SRU Document:

[Impact]

If users use the latest Dell USB Dock, e.g. TB16, they will found the analog audio (lineout jack) can't work under ubuntu linux

[Test Case]

After applying the fix, users can play sound via analog speaker or lineout freely. Without the fix, users can't play sound from lineout jack.

[Regression Potential]

No any possibility to introduce regression since this fix just adding a new dock's support, it does not change any existing code.

[Other Info]

No more info here

Steps:
1. Cold boot system with BME/IE connected
2. Plug in a speaker to Line-out jack
3. Set Analog Stereo Output from Sound settings

Expected results: Line-out port can work

Actual results: Line-out port not work

Hui Wang (hui.wang) on 2017-09-22
Changed in hwe-next:
assignee: nobody → Hui Wang (hui.wang)
importance: Undecided → Critical
Hui Wang (hui.wang) on 2017-09-22
tags: added: originate-from-1715596 somerville xenial
Hui Wang (hui.wang) on 2017-09-25
Changed in hwe-next:
status: New → In Progress
Changed in pulseaudio (Ubuntu):
status: New → In Progress
Daniel van Vugt (vanvugt) wrote :

Fix committed for xenial:
https://git.launchpad.net/~ubuntu-audio-dev/pulseaudio/commit/?h=ubuntu-xenial&id=312b9b8d36b90c6bae520722cf68e01b53832eeb

But I forgot to ask: Does the same fix exist in artful already? It's part of the SRU process that the next release needs to have the fix first, before xenial.

Hui Wang (hui.wang) wrote :

@Daniel,

The fix is not in the artful and zesty, they also need this patch.

And here is the merge request for Artful and Zesty.
https://code.launchpad.net/~hui.wang/pulseaudio/+git/pulseaudio/+merge/331390

If you need me to do anything, please let me know.

Thanks.

Daniel van Vugt (vanvugt) wrote :

Sorry for the delay. More comments in the merge request.

Hui Wang (hui.wang) wrote :

Got it, i will change the merge request.

Hui Wang (hui.wang) wrote :

The new merge request for Artful and Zesty is here:
https://code.launchpad.net/~hui.wang/pulseaudio/+git/pulseaudio/+merge/332335

thanks.

Daniel van Vugt (vanvugt) wrote :
Changed in pulseaudio (Ubuntu):
status: In Progress → Fix Committed
importance: Critical → High
Daniel van Vugt (vanvugt) wrote :

In theory next step is to create a debdiff, attach it here, and subscribe ubuntu-sponsors to the bug.

In practice being release week for 17.10 this might now be called an SRU so you might also be asked to add SRU documentation in the bug description:

https://wiki.ubuntu.com/StableReleaseUpdates#SRU_Bug_Template

Hui Wang (hui.wang) on 2017-10-18
description: updated
Hui Wang (hui.wang) on 2017-10-18
description: updated
Hui Wang (hui.wang) wrote :

debdiff for Artful and Zesty

Hui Wang (hui.wang) wrote :

This is for Xenial.

tags: added: ubuntu-sponsors
Hui Wang (hui.wang) wrote :

@ubuntu-sponsors team,

We added a fix to pulseaudio (xenial and artful), Please help build a new SRU version.

thx.

Hui Wang (hui.wang) wrote :

@Daniel,

Who will trigger the SRU build for pulseaudio?

thx.

Daniel van Vugt (vanvugt) wrote :

A member of ubuntu-sponsors will. That's all I know :)

https://launchpad.net/~ubuntu-sponsors

Jonathan Riddell (jr) wrote :

I added your xenial debdiff to the one from #1728791 and uploaded

Your artful/zenial debdiff seems to be confused and removes 0001-Cherrypick-one-upstream-commit-to-support-audio-on-t.patch and stuff from debian/changelog

Jonathan Riddell (jr) on 2017-11-09
tags: added: verification-needed
tags: added: verification-needed-xenial
removed: verification-needed
Hui Wang (hui.wang) wrote :

@Daniel,

OK, will do it today.

Hui Wang (hui.wang) wrote :

Upgraded the pulseaudio from -proposed, and verified the dock analog audio (lineout + speaker) worked well.

changed verification-needed-xenial to verification-done-xenial

tags: added: verification-done-xenial
removed: verification-needed-xenial
Changed in hwe-next:
status: In Progress → Fix Committed
Jonathan Riddell (jr) wrote :

I'm afraid I had to upload it again as the bug 1728791 patch was incorrect. Please verify again
https://launchpad.net/ubuntu/+source/pulseaudio/1:8.0-0ubuntu3.6

tags: added: verification-needed-xenial
removed: verification-done-xenial
Hui Wang (hui.wang) wrote :

Sorry, forgot to add conf file to build sys, then the conf file (dell-dock-tb16-usb-audio.conf) is not installed to /usr/share/pulseaudio/profile-sets/.

I will send a patch to upstream to fix it first, then backport it to Xenial and Artful.

Changed in pulseaudio (Ubuntu):
status: Fix Committed → Triaged
tags: added: verification-failed-xenial
removed: verification-needed-xenial
Changed in hwe-next:
status: Fix Committed → Triaged
Hui Wang (hui.wang) on 2017-11-14
Changed in pulseaudio (Ubuntu):
status: Triaged → In Progress
Changed in hwe-next:
status: Triaged → In Progress
Hui Wang (hui.wang) wrote :

@Daniel,

The commit https://git.launchpad.net/~ubuntu-audio-dev/pulseaudio/commit/?h=ubuntu-xenial&id=312b9b8d36b90c6bae520722cf68e01b53832eeb is not sufficient to fix this problem, this commit only adds the conf file in the source repository, but forgot to add the conf in the build-sys. I already submitted a simple patch to upstream to fix it, but looks like the maintainer is on vacation, he has not shown up in the maillist for several days.

I am wondering if we could merge this simple fix ahead of upstream, if we could, may I send an independent commit and set the version to 1:8.0-0ubuntu3.6; or you revert my previous commit, then I send one commit which contains both my previous commit and this new simple fix?

And the simple fix like this:

src/Makefile.am | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index e610db7..0348250 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -1312,7 +1312,8 @@ dist_alsaprofilesets_DATA = \
   modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf \
   modules/alsa/mixer/profile-sets/kinect-audio.conf \
   modules/alsa/mixer/profile-sets/sb-omni-surround-5.1.conf \
- modules/alsa/mixer/profile-sets/steelseries-arctis-usb-audio.conf
+ modules/alsa/mixer/profile-sets/steelseries-arctis-usb-audio.conf \
+ modules/alsa/mixer/profile-sets/dell-dock-tb16-usb-audio.conf

 if HAVE_UDEV
 dist_udevrules_DATA = \
--
2.7.4

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers