`alsactl init` unable to initialze audio device on G700

Bug #2049327 reported by Zhaoxuan Zhai
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
alsa-ucm-conf (Ubuntu)
Fix Released
High
Zhaoxuan Zhai
Jammy
Fix Released
Undecided
Unassigned
Mantic
Fix Released
Undecided
Unassigned

Bug Description

[Impact]
This bug cause "alsactl init" command unable to initialize mt8390-evk correctly and print these error message:

```
ceqa@ubuntu:/home/ceqa$ alsactl init
alsa-lib main.c:761:(execute_sequence) unable to execute cset 'name='DP_OUT_MUX' Disconnect'
alsa-lib main.c:2373:(set_boot_user) Unable to execute boot sequence
Found hardware: "mt8390-evk" "" "" "" ""
Hardware is initialized using a generic method
```

Thus, the audio devices on MediaTek G700 do not work.

This patch fix this bug by correcting 'DP_OUT_MUX' to 'DPTX_OUT_MUX'.

[ Test Plan ]
Reproduce:
1. Install Ubuntu 22.04 (jammy) on a MediaTek G700 device
2. Run `alsactl init` on MediaTek G700 device
3. alsactl print error message "unable to execute cset 'name='DP_OUT_MUX' Disconnect'" and audio devices do not work.

Verify:
By applying this patch, alsactl does not print this error message and audio devices work.

[Where problems could occur]
This patch fix the typo in ucm files, and it affects the device init process on Mediatek Genio boards.

[ Other Info ]
There is a typo in 'd/p/0001-Add-initial-support-for-MediaTek-Genio-boards.patch'.

The name of the flag that controls the audio output of the DP port is not 'DP_OUT_MUX'. It is 'DPTX_OUT_MUX'.

MediaTek fix this issue in the PR in upstream:
https://github.com/alsa-project/alsa-ucm-conf/compare/243f2d60740e41a0cfb7779388c17e4a00e1bfe9..a55f840ab2b13f32cb55cd1730933ef0f5a96079

mt8390-evk (G700) Pull Request: (Still under review)
https://github.com/alsa-project/alsa-ucm-conf/pull/321

Revision history for this message
Zhaoxuan Zhai (kxuan) wrote :
Revision history for this message
Zhaoxuan Zhai (kxuan) wrote :
Revision history for this message
Zhaoxuan Zhai (kxuan) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "debdiff_alsa-ucm-conf_jammy_from_1.2.6.3-1ubuntu1.9_to_1.2.6.3-1ubuntu1.11.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

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

Thank you for the work, you mention in the description another fix which is still under review upstream. Did you want to include it in the upload? If not it's not clear why you mention it?

Also we need to start by uploading to Noble or is it already fixed there?

Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

Hi Sebastien,

Here is the background information:
MediaTek sent two PRs[1][2][3] for enabling audio on Genio 700 and 1200 boards in Jun 2023. But, unfortunately the patch is still under review. In order to enable hardware in time, I discussed with Łukasz and he agreed to accept the patch as it was. If the final content accepted by upstream is different from what I backported from [1][2]. I'll create a bug to update it.

When QA verified audio on Core, we found an issue caused by a typo in the patch. So, Zhaoxuan created this bug and uploaded debdiff to fix the issue.

Hi Zhaoxuan,

Please upload the debdiff for noble as well. We have to upload the patch to the latest series, noble, first. After it gets merged, we can SRU the patch to mantic, lunar, and jammy.

---
[1] https://github.com/alsa-project/alsa-ucm-conf/pull/321
[2] https://github.com/alsa-project/alsa-ucm-conf/pull/322
[3] LP: #2000228 [SRU] Add initial support for Mediatek Genio boards

Revision history for this message
Zhaoxuan Zhai (kxuan) wrote :

Here is a debdiff for noble.

Thank you! @seb128 @ethan.hsieh

Changed in alsa-ucm-conf (Ubuntu):
status: New → Fix Committed
importance: Undecided → High
assignee: nobody → Zhaoxuan Zhai (kxuan)
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, I've uploaded to N/M/J now after adding the bug reference to the changelog (please try to think about doing that next time), I don't think it makes sense to SRU to Lunar at this point unless you had an explicit need for it?

I've also changed the J version to be .10 instead of .9 but let me know if there was a reason you did skip one revision

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

This bug was fixed in the package alsa-ucm-conf - 1.2.10-1ubuntu2

---------------
alsa-ucm-conf (1.2.10-1ubuntu2) noble; urgency=medium

  * d/p/0001-Add-initial-support-for-MediaTek-Genio-boards.patch
    - fix typo to make the patch work on mtk8390-evk (lp: #2049327)

 -- Zhaoxuan Zhai <email address hidden> Tue, 16 Jan 2024 11:23:07 +0100

Changed in alsa-ucm-conf (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Zhaoxuan Zhai (kxuan) wrote (last edit ):

Thank you @seb128 .

We don't currently have an explicit need to SRU to Lunar.

And I think skip '.10' is a mistake. :)

tags: added: oem-priority originate-from-2047345
Revision history for this message
Timo Aaltonen (tjaalton) wrote : Please test proposed package

Hello Zhaoxuan, or anyone else affected,

Accepted alsa-ucm-conf into mantic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/alsa-ucm-conf/1.2.9-1ubuntu3.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-mantic to verification-done-mantic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-mantic. 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 Mantic):
status: New → Fix Committed
tags: added: verification-needed verification-needed-mantic
Changed in alsa-ucm-conf (Ubuntu Jammy):
status: New → Fix Committed
tags: added: verification-needed-jammy
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

Hello Zhaoxuan, or anyone else affected,

Accepted alsa-ucm-conf into jammy-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/alsa-ucm-conf/1.2.6.3-1ubuntu1.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-jammy to verification-done-jammy. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-jammy. 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
ethan.hsieh (ethan.hsieh) wrote :

Verified packages on jammy- and mantic-proposed. Here is the test result:

$ alsactl init
alsa-lib main.c:761:(execute_sequence) unable to execute cset 'name='DP_OUT_MUX' Disconnect'
alsa-lib main.c:2373:(set_boot_user) Unable to execute boot sequence
Found hardware: "mt8390-evk" "" "" "" ""
Hardware is initialized using a generic method

# Install the package on jammy-proposed
$ alsactl init
# no error message
$ apt-cache policy alsa-ucm-conf | grep Installed
  Installed: 1.2.6.3-1ubuntu1.10

# Install the package on mantic-proposed
$ alsactl init
# no error message
$ apt-cache policy alsa-ucm-conf | grep Installed
  Installed: 1.2.9-1ubuntu3.2

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

This bug was fixed in the package alsa-ucm-conf - 1.2.9-1ubuntu3.2

---------------
alsa-ucm-conf (1.2.9-1ubuntu3.2) mantic; urgency=medium

  * d/p/0001-Add-initial-support-for-MediaTek-Genio-boards.patch
    - fix typo to make the patch work on mtk8390-evk (lp: #2049327)

 -- Zhaoxuan Zhai <email address hidden> Fri, 12 Jan 2024 16:26:31 +0800

Changed in alsa-ucm-conf (Ubuntu Mantic):
status: Fix Committed → Fix Released
Revision history for this message
Andreas Hasenack (ahasenack) 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.6.3-1ubuntu1.10

---------------
alsa-ucm-conf (1.2.6.3-1ubuntu1.10) jammy; urgency=medium

  * d/p/0001-Add-initial-support-for-MediaTek-Genio-boards.patch
    - fix typo to make the patch work on mt8390-evk (lp: #2049327)

 -- Zhaoxuan Zhai <email address hidden> Fri, 12 Jan 2024 16:24:24 +0800

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