[SRU] Support soundwire speaker AMP-ALC1318 codec

Bug #1997946 reported by Kai-Chuan Hsieh
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OEM Priority Project
Fix Released
Critical
Kai-Chuan Hsieh
alsa-ucm-conf (Ubuntu)
Fix Released
High
Unassigned
Jammy
Fix Released
Undecided
Unassigned
Kinetic
Fix Released
Undecided
Unassigned

Bug Description

[ Impact ]

 * Add ALC1318 speaker codec ucm file
 * Make internal speaker working on the platform with the codec

[ Test Plan ]

 * On platform with the codec, the internal speaker should work after
   boot into Ubuntu

[ Where problems could occur ]

 * The config only be executed on the platform with the codec exported

 * It needs to add the codec combination to the machine driver to make the function work
   ex. https://github.com/thesofproject/linux/blob/topic/sof-dev/sound/soc/intel/common/soc-acpi-intel-rpl-match.c#L225

[ Other Info ]

 * Upstream commit soundwire speaker AMP-ALC1318 codec.
   https://github.com/alsa-project/alsa-ucm-conf/commit/c82c400fb653292bbf0570e278d60e1ba14cc341

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

Thank you for taking the time to report this bug and helping to make Ubuntu better. Unfortunately, we cannot work on this bug because your description didn't include enough information. You may find it helpful to read "How to report bugs effectively" http://www.chiark.greenend.org.uk/~sgtatham/bugs.html. We'd be grateful if you would then provide a more complete description of the problem.

We have instructions on debugging some types of problems at http://wiki.ubuntu.com/DebuggingProcedures.

At a minimum, we need:

1. The specific steps or actions you took that caused you to encounter the problem.
2. The behavior you expected.
3. The behavior you actually encountered (in as much detail as possible).

Please also ensure that you include the release and flavour of Ubuntu that you are using.

Thank you!

Changed in alsa-ucm-conf (Ubuntu):
status: New → Incomplete
description: updated
Revision history for this message
Kai-Chuan Hsieh (kchsieh) wrote (last edit ):

Upload the debdiff file for jammy.

description: updated
Revision history for this message
Kai-Chuan Hsieh (kchsieh) wrote :

Upload debdiff file for kinetic.

Changed in alsa-ucm-conf (Ubuntu):
status: Incomplete → Confirmed
tags: added: oem-priority originate-from-1998737 somerville
Changed in oem-priority:
assignee: nobody → Kai-Chuan Hsieh (kchsieh)
status: New → Triaged
importance: Undecided → Critical
description: updated
description: updated
description: updated
Hui Wang (hui.wang)
summary: - Support soundwire speaker AMP-ALC1318 codec
+ [SRU] Support soundwire speaker AMP-ALC1318 codec
description: updated
description: updated
Revision history for this message
Sebastien Bacher (seb128) wrote (last edit ):

the 22.04 patch modifies 0005-acp6x-add-initial-support-for-AMD-Yellow-Carp-ACP6x.patch is that wanted?

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

I've uploaded to lunar the same changes that kinetic now

Changed in alsa-ucm-conf (Ubuntu):
status: Confirmed → Fix Committed
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
alsa-ucm-conf (1.2.6.3-1ubuntu3.1) lunar; urgency=medium

  * Backport patch for speaker AMP-ALC1318 support. (LP: #1997946)

 -- Kai-Chuan Hsieh <email address hidden> Mon, 26 Dec 2022 15:12:24 +0800

Changed in alsa-ucm-conf (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Kai-Chuan Hsieh (kchsieh) wrote :

reply #4

The diff is because that quilt suggests to do quilt refresh on 0005-acp6x-add-initial-support-for-AMD-Yellow-Carp-ACP6x.patch.
I did the comparison to the modified files which changed by old and new 0005-acp6x-add-initial-support-for-AMD-Yellow-Carp-ACP6x.patch, there is no difference. I think it can help to apply patch more cleanly if someone would like to propose another SRU next time, so I include it into the debdiff too.

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

Uploaded to the different series now

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

Hello Kai-Chuan, or anyone else affected,

Accepted alsa-ucm-conf into kinetic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/alsa-ucm-conf/1.2.6.3-1ubuntu3 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-kinetic to verification-done-kinetic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-kinetic. 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 Kinetic):
status: New → Fix Committed
tags: added: verification-needed verification-needed-kinetic
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 Kai-Chuan, 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.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-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
Andy Chi (andch) wrote :

Install alsa-ucm-conf version 1.2.6.3-1ubuntu1.2 on MYBY-DVT1-C5_202212-31006 with 6.1.0-1004-oem kernel. Speaker works fine.

Revision history for this message
Andy Chi (andch) wrote :

Install alsa-ucm-conf version 1.2.6.3-1ubuntu3 on MYBY-DVT1-C5_202212-31006 with 6.1.0-1004-oem kernel. Speaker works fine.

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

This bug was fixed in the package alsa-ucm-conf - 1.2.6.3-1ubuntu3

---------------
alsa-ucm-conf (1.2.6.3-1ubuntu3) kinetic; urgency=medium

  * Backport patch for speaker AMP-ALC1318 support. (LP: #1997946)

  [ Andy Chi ]
  * d/p/sof-soundwire-set-PGA-capture-switch-for-rt715-mic-in-Boo.patch:
    - fix microphone recording on rt715 (LP: #2000465)

 -- Kai-Chuan Hsieh <email address hidden> Mon, 26 Dec 2022 15:12:24 +0800

Changed in alsa-ucm-conf (Ubuntu Kinetic):
status: Fix Committed → Fix Released
Revision history for this message
Timo Aaltonen (tjaalton) 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.2

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

  * Backport patch for speaker AMP-ALC1318 support. (LP: #1997946)

  [ Andy Chi ]
  * d/p/sof-soundwire-set-PGA-capture-switch-for-rt715-mic-in-Boo.patch:
    - fix microphone recording on rt715 (LP: #2000465)

 -- Kai-Chuan Hsieh <email address hidden> Mon, 26 Dec 2022 14:53:41 +0800

Changed in alsa-ucm-conf (Ubuntu Jammy):
status: Fix Committed → Fix Released
Andy Chi (andch)
Changed in oem-priority:
status: Triaged → 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.