Support for Cirrus Logic audio solution CS42L45 with amplifiers on new Dell PTL platform

Bug #2131725 reported by Chris Chiu
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
alsa-ucm-conf (Ubuntu)
Status tracked in Resolute
Noble
Fix Committed
Undecided
Unassigned
Plucky
Won't Fix
Undecided
Unassigned
Questing
Fix Committed
Undecided
Unassigned
Resolute
Fix Released
Undecided
Unassigned
linux-firmware (Ubuntu)
Status tracked in Resolute
Noble
Fix Released
Undecided
Unassigned
Plucky
Won't Fix
Undecided
Unassigned
Questing
Fix Released
Undecided
Unassigned
Resolute
Fix Released
Undecided
Unassigned
linux-oem-6.17 (Ubuntu)
Status tracked in Resolute
Noble
Fix Released
Undecided
Unassigned
Plucky
Invalid
Undecided
Unassigned
Questing
Invalid
Undecided
Unassigned
Resolute
Invalid
Undecided
Unassigned

Bug Description

SRU Justification:

[Impact]
New Dell platform with CirrusLogic audio solution CS42L43+CS35L57/CS35L63 powered by Intel PantherLake will have no basic audio functions w/o Soundwire Driver Support.

================ Linux Kernel ========================

[Fix]
Backport new Intel soundwire drivers of the following
Add SDCA class driver:
3af1815a2f9c ASoC: SDCA: Add basic SDCA function driver
2d877d0659cb ASoC: SDCA: Add basic SDCA class driver
4496d1c65bad ASoC: SDCA: add function devices
5acf17b6df5e ASoC: SDCA: Add helper to write initialization writes
193c65ec8ed4 ASoC: SDCA: Populate regmap cache for readable Controls
222cbe172e5f ASoC: SDCA: Factor out helper to process Control defaults
fb62da31fad2 ASoC: SDCA: Use helper macros for control identification
edb82534ee65 ASoC: SDCA: Move most of the messages from info to debug
336beae295bf ASoC: SDCA: Add comment for function reset polling
cc58055bfefa ASoC: SDCA: Correct FDL locking in sdca_fdl_process()
5fe65824b74c ASoC: SDCA: Add missing forward declaration in header
3508311f2e1c ASoC: SDCA: Fix missing dash in HIDE DisCo property
549f618516a1 ASoC: SDCA: Remove duplicated module macros
43a3adb6dd39 spi: spidev: add compatible for arduino spi mcu interface
a7bde7c10902 dt-bindings: trivial-devices: add arduino spi mcu interface
3efee7362dbf ASoC: SDCA: Add stubs for FDL helper functions

Add SDCA UMP/FDL support:
ef042df96d0e ASoC: SDCA: Add HID button IRQ
12aa3160c10a ASoC: SDCA: Add early IRQ handling
e92e25f77748 ASoC: SDCA: Add UMP timeout handling for FDL
0723affa1bee ASoC: SDCA: Add completion for FDL start and stop
aeaf27ec6571 ASoC: SDCA: Add FDL-specific IRQ processing
71f7990a34cd ASoC: SDCA: Add FDL library for XU entities
c4d096c3ca42 ASoC: SDCA: Add SDCA FDL data parsing
daab108504be ASoC: SDCA: Add UMP buffer helper functions
7b6be935e7ef ASoC: SDCA: Parse Function Reset max delay
0a5e9769d088 ASoC: SDCA: Parse XU Entity properties
c7b6c6b60594 ASoC: SDCA: Force some SDCA Controls to be volatile
dfe7c3401ed3 ASoC: SDCA: Rely less on the ASoC component in IRQ handling
8d557cc4867f ASoC: SDCA: Factor out a helper to find SDCA IRQ data
56bbda23d4be ASoC: SDCA: Update externally_requested flag to cover all requests
390c05f47d07 ASoC: SDCA: Pass device register map from IRQ alloc to handlers
7159816707dc ASoC: SDCA: Pass SoundWire slave to HID
907364ea3db4 ASoC: SDCA: Add manual PM runtime gets to IRQ handlers
013a3a66f25a regmap: sdw-mbq: Don't assume the regmap device is the SoundWire slave
715159314dfa ASoC: SDCA: Rename SoundWire struct device variables

and CS42L45 driver patch from CirrusLogic
1e645bca9d1e ASoC: intel: sof_sdw: Add codec_info for cs42l45
3f6b562f2107 ASoC: sdw_utils: Add cs42l45 support functions
c66297d09e1a ASoC: intel: sof_sdw: Add ability to have auxiliary devices
2ae4659533d8 ASoC: sdw_utils: Move codec_name to dai info
26ee34d2f5c7 ASoC: sdw_utils: Add codec_conf for every DAI

9be609d4f429 - Maciej Strozek, 8 weeks ago : soundwire: intel_auxdevice: add cs42l45 codec to wake_capable_list -

5edcc5451766 - Richard Fitzgerald, 6 weeks ago : ASoC: cs35l56: Add support for CS35L56 B2 silicon -
ad36fbb6ee4b - Richard Fitzgerald, 6 weeks ago : ASoC: cs35l56: Set fw_regs table after getting REVID -

and patch set of [PATCH 0/7] SDCA jack and system suspend fixups

<=============== linux-firmware ========================

[Fix]
Backport the patch firmware from CirrusLogic
https://gitlab.com/kernel-firmware/linux-firmware/-/commit/18672039c39ac71ecc17592999016311cff901ec

<=============== alsa-ucm-conf ========================
Backport the patch firmware from CirrusLogic
f47b9418c81a ("ucm2: soundwire: cs42l45: Add support for CS42L45 codec")
2e090676c9a4 ("ucm2: Add setting LED Mode in SetLED macro")
7bab30e73569 ("ucm2: sof-soundwire: Update cs42l45 JackControls)
51bada5f0c1b ("ucm2: sof-soundwire: Simplify cs42l45 configs")

[Test Case]
1. Boot up the machine of new Dell Intel PantherLake platform with CirrusLogic CS42L43 + new amplifiers
2. Open settings->Sound->Output Device and make sure it's not Dummy audio devices
3. Click test icon for basic audio output functions
4. Use `arecord` to record and play the recorded audio file with `aplay` for basic speaker/microphone functions.

[Where problems could occur]
It adds the support for new CirrusLogic codec+amplifier on PTL platform. No regression expected at the current stage.

CVE References

Chris Chiu (mschiu77)
no longer affects: linux-firmware
tags: added: kernel-daily-bug
Chris Chiu (mschiu77)
description: updated
summary: - Support for CirrusLogic audio solution CS42L43 with amplifiers on new
+ Support for Cirrus Logic audio solution CS42L43 with amplifiers on new
Dell PTL platform
Revision history for this message
Chris Chiu (mschiu77) wrote : Re: Support for Cirrus Logic audio solution CS42L43 with amplifiers on new Dell PTL platform
Revision history for this message
Anthony Wong (anthonywong) wrote :

Confirmed with Chris that firmware already in Resolute.

Changed in linux-firmware (Ubuntu Resolute):
status: New → Fix Released
Chris Chiu (mschiu77)
description: updated
Juerg Haefliger (juergh)
Changed in linux-firmware (Ubuntu Questing):
status: New → Fix Committed
Changed in linux-firmware (Ubuntu Plucky):
status: New → Won't Fix
Juerg Haefliger (juergh)
Changed in linux-firmware (Ubuntu Noble):
status: New → Fix Committed
Revision history for this message
Timo Aaltonen (tjaalton) wrote : Please test proposed package

Hello Chris, or anyone else affected,

Accepted linux-firmware into questing-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/linux-firmware/20250901.git993ff19b-0ubuntu1.5 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-questing to verification-done-questing. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-questing. 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
Timo Aaltonen (tjaalton) wrote : Re: Support for Cirrus Logic audio solution CS42L43 with amplifiers on new Dell PTL platform

what about the resolute kernel?

Changed in linux-oem-6.17 (Ubuntu Plucky):
status: New → Invalid
Changed in linux-oem-6.17 (Ubuntu Questing):
status: New → Invalid
Changed in linux-oem-6.17 (Ubuntu Resolute):
status: New → Invalid
Revision history for this message
Timo Aaltonen (tjaalton) wrote : Please test proposed package

Hello Chris, or anyone else affected,

Accepted linux-firmware into noble-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/linux-firmware/20240318.git3b128b60-0ubuntu2.22 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-noble to verification-done-noble. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-noble. 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
Ubuntu SRU Bot (ubuntu-sru-bot) wrote : Autopkgtest regression report (linux-firmware/20250901.git993ff19b-0ubuntu1.5)

All autopkgtests for the newly accepted linux-firmware (20250901.git993ff19b-0ubuntu1.5) for questing have finished running.
The following regressions have been reported in tests triggered by the package:

initramfs-tools/0.150ubuntu3.1 (s390x)

Please visit the excuses page listed below and investigate the failures, proceeding afterwards as per the StableReleaseUpdates policy regarding autopkgtest regressions [1].

https://people.canonical.com/~ubuntu-archive/proposed-migration/questing/update_excuses.html#linux-firmware

[1] https://wiki.ubuntu.com/StableReleaseUpdates#Autopkgtest_Regressions

Thank you!

Revision history for this message
Chris Chiu (mschiu77) wrote : Re: Support for Cirrus Logic audio solution CS42L43 with amplifiers on new Dell PTL platform

Verified on HUR14-DVT2-C1 with linux-firmware/20240318.git3b128b60-0ubuntu2.22 + alsa-ucm-conf 1.2.15-1ubuntu1 (built from https://code.launchpad.net/~mschiu77/ubuntu/+source/alsa-ucm-conf/+git/alsa-ucm-conf/+merge/496002) + exp kernel

Verified on HUR14-DVT2-C1 with linux-firmware/20250901.git993ff19b-0ubuntu1.5 + alsa-ucm-conf 1.2.15-1ubuntu1 (built from https://code.launchpad.net/~mschiu77/ubuntu/+source/alsa-ucm-conf/+git/alsa-ucm-conf/+merge/496002) + exp kernel

audio play/record functions work as expected.

aplay shows
ubuntu@localhost:~$ aplay -l
**** List of PLAYBACK Hardware Devices ****
card 0: sofsoundwire [sof-soundwire], device 0: Jack Out (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 2: Speaker (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 5: HDMI1 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 6: HDMI2 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 7: HDMI3 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 31: Deepbuffer Jack Out (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0

arecord shows
ubuntu@localhost:~$ arecord -l
**** List of CAPTURE Hardware Devices ****
card 0: sofsoundwire [sof-soundwire], device 1: Jack In (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 4: Microphone (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0

description: updated
Changed in linux-oem-6.17 (Ubuntu Noble):
status: New → In Progress
tags: added: verification-done-noble
tags: added: jira-somerville-1948 somerville
Chris Chiu (mschiu77)
tags: added: verification-done-questing
summary: - Support for Cirrus Logic audio solution CS42L43 with amplifiers on new
+ Support for Cirrus Logic audio solution CS42L45 with amplifiers on new
Dell PTL platform
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package linux-firmware - 20240318.git3b128b60-0ubuntu2.22

---------------
linux-firmware (20240318.git3b128b60-0ubuntu2.22) noble; urgency=medium

  * Miscellaneous Ubuntu changes
    - [Workflow] Update the gitea workflow
    - [Workflow] Move checkout to top of workflow
  * [SRU] Upgrade Intel xe GUC to v70.49.4 for Intel Panther Lake (LP: #2127969)
    - xe: Update GUC to v70.49.4 for BMG, LNL, PTL
  * Support for Realtek audio solution ALC3329B+ALC1708B on new Dell PTL platform (LP: #2129952)
    - realtek: rt1321: Add patch firmware of MCU
  * [SRU][R/Q/N] mt7925: Support 802.11d and CQM events for roaming (LP: #2130381)
    - linux-firmware: update firmware for MT7925 WiFi device
  * RTL8922A BT USB: Bluetooth: hci0: RTL: download fw command failed (-13) (LP: #2131732)
    - rtl_bt: Update RTL8922A BT USB firmware to 0x41C0_C905
  * Support for Cirrus Logic audio solution CS42L43 with amplifiers on new Dell PTL platform (LP: #2131725)
    - cirrus: cs42l45: Add firmware for Cirrus Logic CS42L45 SDCA codec
    - cirrus: cs35l57: Add firmware for a few Dell products
  * Add TI tas2781 support for HP platforms (LP: #2133664)
    - tas2781: Upload dsp firmware for ASUS laptop 1EB30 & 1EB31
    - Add a link from TAS2XXX1EB3.bin -> ti/tas2781/TAS2XXX1EB30.bin
    - ASoC: tas2781: fix the license issue for tas781 firmware
    - linux-firmware: Create audio folder in ti folder, and move all the audio firmwares into it
    - ASoC: tas2781: Update dsp firmware for HP and ASUS projects
  * Update aic100 fw for power issues (LP: #2131936)
    - qcom: Update aic100 firmware files
    - qcom: Update aic100 firmware files
  * Support Qualcomm RB4 graphics firmware (LP: #2133787)
    - qcom: Update gpu firmwares for qcs8300 chipset
    - qcom: Update A623 GMU fw
  * Linux-firmware: Add qualcomm serial engine firmware qupv3fw.elf (LP: #2133923)
    - qcom: add QUPv3 firmware for QCS9100 platform
    - qcom: add QUPv3 firmware for QCM6490 platform
    - Adjust QUPv3 driver name
    - qcom: add QUPv3 firmware for QCS8300 platform
    - qcom: Add QCS6490 symlink for QUPv3 firmware

 -- Juerg Haefliger <email address hidden> Mon, 08 Dec 2025 13:57:49 +0100

Changed in linux-firmware (Ubuntu Noble):
status: Fix Committed → Fix Released
Revision history for this message
Chris Halse Rogers (raof) wrote : Update Released

The verification of the Stable Release Update for linux-firmware 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 linux-firmware - 20250901.git993ff19b-0ubuntu1.5

---------------
linux-firmware (20250901.git993ff19b-0ubuntu1.5) questing; urgency=medium

  * Miscellaneous Ubuntu changes
    - [Workflow] Update the gitea workflow
    - [Workflow] Move checkout to top of workflow
  * [SRU] Upgrade Intel xe GUC to v70.49.4 for Intel Panther Lake (LP: #2127969)
    - xe: Update GUC to v70.49.4 for BMG, LNL, PTL
  * [SRU][R/Q/N] mt7925: Support 802.11d and CQM events for roaming (LP: #2130381)
    - linux-firmware: update firmware for MT7925 WiFi device
  * RTL8922A BT USB: Bluetooth: hci0: RTL: download fw command failed (-13) (LP: #2131732)
    - rtl_bt: Update RTL8922A BT USB firmware to 0x41C0_C905
  * Support for Cirrus Logic audio solution CS42L43 with amplifiers on new Dell PTL platform (LP: #2131725)
    - cirrus: cs42l45: Add firmware for Cirrus Logic CS42L45 SDCA codec
    - cirrus: cs35l57: Add firmware for a few Dell products
  * Add TI tas2781 support for HP platforms (LP: #2133664)
    - linux-firmware: Create audio folder in ti folder, and move all the audio firmwares into it
    - ASoC: tas2781: Update dsp firmware for HP and ASUS projects
  * Update aic100 fw for power issues (LP: #2131936)
    - qcom: Update aic100 firmware files
    - qcom: Update aic100 firmware files
  * Support Qualcomm RB4 graphics firmware (LP: #2133787)
    - qcom: Update A623 GMU fw

 -- Juerg Haefliger <email address hidden> Mon, 08 Dec 2025 13:47:50 +0100

Changed in linux-firmware (Ubuntu Questing):
status: Fix Committed → Fix Released
Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Ubuntu 25.04 (Plucky Puffin) has reached end of life, so this bug will not be fixed for that specific release.

Changed in alsa-ucm-conf (Ubuntu Plucky):
status: New → Won't Fix
LEE KUAN-YING (kyyc0426)
Changed in linux-oem-6.17 (Ubuntu Noble):
status: In Progress → Fix Committed
Revision history for this message
Ubuntu Kernel Bot (ubuntu-kernel-bot) wrote :

This bug is awaiting verification that the linux-oem-6.17/6.17.0-1011.11 kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-noble-linux-oem-6.17' to 'verification-done-noble-linux-oem-6.17'. If the problem still exists, change the tag 'verification-needed-noble-linux-oem-6.17' to 'verification-failed-noble-linux-oem-6.17'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: kernel-spammed-noble-linux-oem-6.17-v2 verification-needed-noble-linux-oem-6.17
Revision history for this message
Chris Chiu (mschiu77) wrote :

Verified on HUR14-DVT2-C1 with linux-oem-6.17/6.17.0-1011.11 + linux-firmware/20250901.git993ff19b-0ubuntu1.5 + 1.2.10-1ubuntu5.9

audio play/record functions work as expected.

aplay shows
ubuntu@localhost:~$ aplay -l
**** List of PLAYBACK Hardware Devices ****
card 0: sofsoundwire [sof-soundwire], device 0: Jack Out (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 2: Speaker (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 5: HDMI1 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 6: HDMI2 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 7: HDMI3 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 31: Deepbuffer Jack Out (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0

arecord shows
ubuntu@localhost:~$ arecord -l
**** List of CAPTURE Hardware Devices ****
card 0: sofsoundwire [sof-soundwire], device 1: Jack In (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 4: Microphone (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0

tags: added: verification-done-noble-linux-oem-6.17
removed: verification-needed-noble-linux-oem-6.17
Revision history for this message
Kai-Chuan Hsieh (kchsieh) wrote :

Upload debdiff for noble.

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

Upload debdiff for questing.

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

Resolute contains all required patches already, no need to update.

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

The attachment "alsa-ucm-conf_1.2.10-1ubuntu5.10.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
Changed in alsa-ucm-conf (Ubuntu Resolute):
status: New → Fix Released
Revision history for this message
Matthew Ruffell (mruffell) wrote :

Hi Kai-Chuan Hsieh,

Is "2e09067 ucm2: Add setting LED Mode in SetLED macro" really necessary?

If you look closer, the line:

+diff --git a/ucm2/codecs/cs42l45-dmic/init.conf b/ucm2/codecs/cs42l45-dmic/init.conf
+index 88b19c8..6dd19bb 100644
+--- a/ucm2/codecs/cs42l45-dmic/init.conf
++++ b/ucm2/codecs/cs42l45-dmic/init.conf
+@@ -5,5 +5,5 @@ BootSequence [
+ ]
+
+ Macro [
+- { SetLED { LED="mic" Action="attach" CtlId="cs42l45 FU 113 Mute Switch" } }
++ { SetLED { LED="mic" Action="attach" CtlId="cs42l45 FU 113 Mute Switch" Mode="follow-route"} }
+ ]

just gets deleted anyway in "51bada5 ucm2: sof-soundwire: Simplify cs42l45 configs".

+--- a/ucm2/codecs/cs42l45-dmic/init.conf
++++ b/ucm2/codecs/cs42l45-dmic/init.conf
+
+ Macro [
+- { SetLED { LED="mic" Action="attach" CtlId="cs42l45 FU 113 Mute Switch" Mode="follow-route"} }
++ { SetLED { LED="mic" Action="attach" CtlId="cs42l45 FU 113 Channel Switch"} }
+ ]

Whats the point of this patch?

If you look more closely:

+diff --git a/ucm2/common/ctl/led.conf b/ucm2/common/ctl/led.conf
+index f10ad87..7f47daa 100644
+--- a/ucm2/common/ctl/led.conf
++++ b/ucm2/common/ctl/led.conf
+@@ -30,6 +30,7 @@
+ # LED - LED selection - "speaker" or "mic"
+ # Action - action for given control - "attach" or "detach"
+ # CtlId - control identifier (as for amixer, e.g. "Capture Switch" or "name='CaptureSw
itch',index=1")
++# Mode - (optional) LED mode - "on", "off", "follow-mute" or "follow-route"
+ #
+
+ DefineMacro.SetLED {
+@@ -46,6 +47,15 @@ DefineMacro.SetLED {
+ }
+ }
+ If.1 {
++ Condition {
++ Type String
++ Empty "${var:-__Mode}"
++ }
++ False.FixedBootSequence [
++ sysw "-/class/sound/ctl-led/${var:__LED}/mode:${var:__Mode}"
++ ]
++ }
++ If.2 {
+ Condition { Type AlwaysTrue }
+ True.FixedBootSequence [
+ sysw "-/class/sound/ctl-led/${var:__LED}/card${CardNumber}/${var:__
Action}:${var:__CtlId}"

It moves the existing logic for If.1 into If.2, and adds completely new logic
for If.1.

Is this going to change behaviour of everyones Mute LEDS? How does alsa-ucm-conf
know to use this new If.1 or If.2 for existing behaviour?

I think this doesn't pass the "minimal patch" rules, and creates unnecessary risk.

Can you just delete the hunk from "51bada5 ucm2: sof-soundwire: Simplify cs42l45 configs".
instead, and delete "2e09067 ucm2: Add setting LED Mode in SetLED macro"
entirely?

When we backport patches to stable releases, they don't have to always be
straight cherry picks from upstream. Sometimes its better to modify patches so
that we only have the smallest fix possible. Within reason anyway.

That way it would be less likely to regress existing behaviour of the Mute LED.

PS, your patch format and naming is much better, well done!

Thanks,
Matthew

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

@mruffell

Hello,

Thanks for looking into it carefully. I've updated the debdiff, please check it again.

Revision history for this message
Matthew Ruffell (mruffell) wrote :

Uploaded to questing along with bug 2139651

Uploading alsa-ucm-conf_1.2.12-1ubuntu2.3.dsc
Uploading alsa-ucm-conf_1.2.12-1ubuntu2.3.debian.tar.xz
Uploading alsa-ucm-conf_1.2.12-1ubuntu2.3_source.buildinfo
Uploading alsa-ucm-conf_1.2.12-1ubuntu2.3_source.changes

Changed in alsa-ucm-conf (Ubuntu Noble):
status: New → In Progress
Changed in alsa-ucm-conf (Ubuntu Questing):
status: New → In Progress
Revision history for this message
Matthew Ruffell (mruffell) wrote :

Note to SRU Team:
The first questing upload contains a extra file that should not have been there. Please reject the first questing upload and look at the second one instead.

Revision history for this message
Matthew Ruffell (mruffell) wrote :

Hi Kai-Chaun,

Your noble debdiff contains an extra file "debian/patches/0001-ucm2-sof-soundwire-Add-support-for-CS42L43B-variant.patch".

This is not mentioned in d/changelog, and is not added to the series file. Since this bug is for a different chipset I just assumed you made a mistake and did not intend it to be there.

If you still need this file, file another LP bug and make another debdiff.

I have removed this file from the uploads.

Thanks,
Matthew

Revision history for this message
Matthew Ruffell (mruffell) wrote :

Uploaded to noble along with bug 2139651

Uploading alsa-ucm-conf_1.2.10-1ubuntu5.10.dsc
Uploading alsa-ucm-conf_1.2.10-1ubuntu5.10.debian.tar.xz
Uploading alsa-ucm-conf_1.2.10-1ubuntu5.10_source.buildinfo
Uploading alsa-ucm-conf_1.2.10-1ubuntu5.10_source.changes

Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (178.6 KiB)

This bug was fixed in the package linux-oem-6.17 - 6.17.0-1011.11

---------------
linux-oem-6.17 (6.17.0-1011.11) noble; urgency=medium

  * noble/linux-oem-6.17: 6.17.0-1011.11 -proposed tracker (LP: #2137848)

  * Packaging resync (LP: #1786013)
    - [Packaging] update variants
    - [Packaging] debian.oem/dkms-versions -- update from kernel-versions
      (main/2026.01.12)

  * Pre-build Synaptics display link evdi kernel driver (LP: #2110634)
    - [Packaging] debian.oem/dkms-versions -- add evdi

  * Support for Cirrus Logic audio solution CS42L45 with amplifiers on new
    Dell PTL platform (LP: #2131725)
    - ACPICA: Add SoundWire File Table (SWFT) signature
    - ASoC: cs35l56: Set fw_regs table after getting REVID
    - ASoC: cs35l56: Add support for CS35L56 B2 silicon
    - ASoC: soc_sdw_utils: add cs35l57 support
    - ASoC: SDCA: Rename SoundWire struct device variables
    - regmap: sdw-mbq: Don't assume the regmap device is the SoundWire slave
    - ASoC: SDCA: Add manual PM runtime gets to IRQ handlers
    - ASoC: SDCA: Pass SoundWire slave to HID
    - ASoC: SDCA: Pass device register map from IRQ alloc to handlers
    - ASoC: SDCA: Update externally_requested flag to cover all requests
    - ASoC: SDCA: Factor out a helper to find SDCA IRQ data
    - ASoC: SDCA: Rely less on the ASoC component in IRQ handling
    - ASoC: SDCA: Force some SDCA Controls to be volatile
    - ASoC: SDCA: Parse XU Entity properties
    - ASoC: SDCA: Parse Function Reset max delay
    - ASoC: SDCA: Add UMP buffer helper functions
    - ASoC: SDCA: Add SDCA FDL data parsing
    - ASoC: SDCA: Add FDL library for XU entities
    - ASoC: SDCA: Add FDL-specific IRQ processing
    - ASoC: SDCA: Add completion for FDL start and stop
    - ASoC: SDCA: Add UMP timeout handling for FDL
    - ASoC: SDCA: Add early IRQ handling
    - ASoC: SDCA: Add HID button IRQ
    - ASoC: SOF: Intel: remove hyphen from AMP<index> name_prexix
    - regmap: sdw-mbq: Reorder regmap_mbq_context struct for better packing
    - ASoC: SDCA: Add stubs for FDL helper functions
    - ASoC: SDCA: Remove duplicated module macros
    - ASoC: SDCA: Fix missing dash in HIDE DisCo property
    - ASoC: SDCA: Add missing forward declaration in header
    - ASoC: SDCA: Correct FDL locking in sdca_fdl_process()
    - ASoC: SDCA: Add comment for function reset polling
    - ASoC: SDCA: Move most of the messages from info to debug
    - ASoC: SDCA: Use helper macros for control identification
    - ASoC: SDCA: Factor out helper to process Control defaults
    - ASoC: SDCA: Populate regmap cache for readable Controls
    - ASoC: SDCA: Add helper to write initialization writes
    - ASoC: SDCA: add function devices
    - ASoC: SDCA: Add basic SDCA class driver
    - ASoC: SDCA: Add basic SDCA function driver
    - SAUCE: add config for SDCA class drivers
    - ASoC: SDCA: Align mute controls to ALSA expectations
    - ASoC: SDCA: Add terminal type into input/output widget name
    - ASoC: sdw_utils: Add codec_conf for every DAI
    - ASoC: sdw_utils: Move codec_name to dai info
    - ASoC: intel: sof_sdw: Add ability to have auxiliary devices
    - ASoC: sdw_utils: Add cs42l45 support functio...

Changed in linux-oem-6.17 (Ubuntu Noble):
status: Fix Committed → Fix Released
Revision history for this message
Timo Aaltonen (tjaalton) wrote : Please test proposed package

Hello Chris, or anyone else affected,

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

Hello Chris, or anyone else affected,

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

Verified on HUR14-DVT2-C1 with linux-firmware/20250901.git993ff19b-0ubuntu1.5 + alsa-ucm-conf/1.2.12-1ubuntu2.3 + linux-oem-6.17/6.17.0-1013.13
Verified on HUR14-DVT2-C1 with linux-firmware/20250901.git993ff19b-0ubuntu1.5 + alsa-ucm-conf/1.2.10-1ubuntu5.10 + linux-oem-6.17/6.17.0-1013.13

audio play/record functions work as expected.

aplay shows
ubuntu@localhost:~$ aplay -l
**** List of PLAYBACK Hardware Devices ****
card 0: sofsoundwire [sof-soundwire], device 0: Jack Out (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 2: Speaker (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 5: HDMI1 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 6: HDMI2 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 7: HDMI3 (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 31: Deepbuffer Jack Out (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0

arecord shows
ubuntu@localhost:~$ arecord -l
**** List of CAPTURE Hardware Devices ****
card 0: sofsoundwire [sof-soundwire], device 1: Jack In (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: sofsoundwire [sof-soundwire], device 4: Microphone (*) []
  Subdevices: 1/1
  Subdevice #0: subdevice #0

tags: added: verification-done
removed: verification-needed
Chris Chiu (mschiu77)
tags: added: done-needed-noble verification-done-questing
removed: verification-needed-noble verification-needed-questing
tags: added: verification-done-noble
removed: done-needed-noble
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.