- Alsa-devel - mailman.alsa-project.org

[GIT PULL] ASoC fixes for v6.14-rc6
by Mark Brown 13 Mar '25

13 Mar '25

The following changes since commit 7eb172143d5508b4da468ed59ee857c6e5e01da6: Linux 6.14-rc5 (2025-03-02 11:48:20 -0800) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.14-rc6 for you to fetch changes up to 658fb7fe8e7f4014ea17a4da0e0c1d9bc319fa35: ASoC: cs42l43: convert to SYSTEM_SLEEP_PM_OPS (2025-03-12 16:12:24 +0000) ---------------------------------------------------------------- ASoC: Fixes for v6.14 The bulk of this is driver specific fixes, mostly unremarkable. There's also one core fix from Charles, fixing up confusion around the limiting of maximum control values. ---------------------------------------------------------------- Alexey Klimov (1): ASoC: codecs: wsa884x: report temps to hwmon in millidegree of Celsius Arnd Bergmann (1): ASoC: cs42l43: convert to SYSTEM_SLEEP_PM_OPS Bard Liao (2): ASoC: rt1320: set wake_capable = 0 explicitly ASoC: rt722-sdca: add missing readable registers Charles Keepax (2): ASoC: ops: Consistently treat platform_max as control value ASoC: cs42l43: Fix maximum ADC Volume Christophe JAILLET (1): ASoC: codecs: wm0010: Fix error handling path in wm0010_spi_probe() Maciej Strozek (1): ASoC: cs42l43: Add jack delay debounce after suspend Peter Ujfalusi (1): ASoC: Intel: sof_sdw: Fix unlikely uninitialized variable use in create_sdw_dailinks() Thomas Mizrahi (1): ASoC: amd: yc: Support mic on another Lenovo ThinkPad E16 Gen 2 model Thorsten Blum (1): ASoC: tegra: Fix ADX S24_LE audio format include/sound/soc.h | 5 ++++- sound/soc/amd/yc/acp6x-mach.c | 7 +++++++ sound/soc/codecs/cs42l43-jack.c | 13 ++++++++++--- sound/soc/codecs/cs42l43.c | 17 +++++++++++++++-- sound/soc/codecs/cs42l43.h | 3 +++ sound/soc/codecs/rt1320-sdw.c | 3 +++ sound/soc/codecs/rt722-sdca-sdw.c | 4 ++++ sound/soc/codecs/wm0010.c | 13 +++++++++++-- sound/soc/codecs/wsa884x.c | 4 ++-- sound/soc/intel/boards/sof_sdw.c | 2 +- sound/soc/soc-ops.c | 15 +++++++-------- sound/soc/tegra/tegra210_adx.c | 4 ++-- 12 files changed, 69 insertions(+), 21 deletions(-)

2 1

Channel count in GoXLR SplitPCM configuration
by GitHub issues - edited 12 Mar '25

12 Mar '25

alsa-project/alsa-ucm-conf issue #518 was edited from pv: Similar issue as #515, and related to #444: We have a report https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/4602 that GoXLR UCM profile with SplitPCM=1 specifies always 23 channels, but the device can't be opened with exactly 23 channels. According to https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/4602#note_2819696 the device may have 21, 23, or 25 depending on firmware/etc details. In Pipewire, I think we add a workaround that just uses the available channel count even if it is different from what UCM profile specifies, but logs warnings if the counts are different so these issues in the profiles can be caught. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/518 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

Channel count in GoXLR SplitPCM configuration
by GitHub issues - opened 12 Mar '25

12 Mar '25

alsa-project/alsa-ucm-conf issue #518 was opened from pv: Similar issue as #515, and related to #444: We have a report https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/4602#note_2819696 that GoXLR UCM profile with SplitPCM=1 specifies always 23 channels, but the device can't be opened with exactly 23 channels. According to https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/4602#note_2819696 the device may have 21, 23, or 25 depending on firmware/etc details. In Pipewire, I think we add a workaround that just uses the available channel count even if it is different from what UCM profile specifies, but logs warnings if the counts are different so these issues in the profiles can be caught. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/518 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[GIT PULL] ASoC fixes for v6.14-rc6
by Mark Brown 12 Mar '25

12 Mar '25

The following changes since commit 7eb172143d5508b4da468ed59ee857c6e5e01da6: Linux 6.14-rc5 (2025-03-02 11:48:20 -0800) are available in the Git repository at: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git tags/asoc-fix-v6.14-rc6 for you to fetch changes up to ed92bc5264c4357d4fca292c769ea9967cd3d3b6: ASoC: codecs: wm0010: Fix error handling path in wm0010_spi_probe() (2025-03-11 13:51:52 +0000) ---------------------------------------------------------------- ASoC: Fixes for v6.14 The bulk of this is driver specific fixes, mostly unremarkable. There's also one core fix from Charles, fixing up confusion around the limiting of maximum control values. ---------------------------------------------------------------- Alexey Klimov (1): ASoC: codecs: wsa884x: report temps to hwmon in millidegree of Celsius Bard Liao (2): ASoC: rt1320: set wake_capable = 0 explicitly ASoC: rt722-sdca: add missing readable registers Charles Keepax (2): ASoC: ops: Consistently treat platform_max as control value ASoC: cs42l43: Fix maximum ADC Volume Christophe JAILLET (1): ASoC: codecs: wm0010: Fix error handling path in wm0010_spi_probe() Maciej Strozek (1): ASoC: cs42l43: Add jack delay debounce after suspend Peter Ujfalusi (1): ASoC: Intel: sof_sdw: Fix unlikely uninitialized variable use in create_sdw_dailinks() Thomas Mizrahi (1): ASoC: amd: yc: Support mic on another Lenovo ThinkPad E16 Gen 2 model Thorsten Blum (1): ASoC: tegra: Fix ADX S24_LE audio format include/sound/soc.h | 5 ++++- sound/soc/amd/yc/acp6x-mach.c | 7 +++++++ sound/soc/codecs/cs42l43-jack.c | 13 ++++++++++--- sound/soc/codecs/cs42l43.c | 17 +++++++++++++++-- sound/soc/codecs/cs42l43.h | 3 +++ sound/soc/codecs/rt1320-sdw.c | 3 +++ sound/soc/codecs/rt722-sdca-sdw.c | 4 ++++ sound/soc/codecs/wm0010.c | 13 +++++++++++-- sound/soc/codecs/wsa884x.c | 4 ++-- sound/soc/intel/boards/sof_sdw.c | 2 +- sound/soc/soc-ops.c | 15 +++++++-------- sound/soc/tegra/tegra210_adx.c | 4 ++-- 12 files changed, 69 insertions(+), 21 deletions(-)

2 1

[PATCH 5.10 1/1] ASoC: ops: Check for negative values before reading them
by d.privalov 12 Mar '25

12 Mar '25

From: Mark Brown <broonie(a)kernel.org> commit 1601033da2dd2052e0489137f7788a46a8fcd82f upstream. The controls allow inputs to be specified as negative but our manipulating them into register fields need to be done on unsigned variables so the checks for negative numbers weren't taking effect properly. Do the checks for negative values on the variable in the ABI struct rather than on our local unsigned copy. Signed-off-by: Mark Brown <broonie(a)kernel.org> Link: https://lore.kernel.org/r/20220128192443.3504823-1-broonie@kernel.org Signed-off-by: Mark Brown <broonie(a)kernel.org> Signed-off-by: Dmitriy Privalov <d.privalov(a)omp.ru> --- sound/soc/soc-ops.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/sound/soc/soc-ops.c b/sound/soc/soc-ops.c index a83cd8d8a9633..a1087dfee532d 100644 --- a/sound/soc/soc-ops.c +++ b/sound/soc/soc-ops.c @@ -316,26 +316,26 @@ int snd_soc_put_volsw(struct snd_kcontrol *kcontrol, if (sign_bit) mask = BIT(sign_bit + 1) - 1; + if (ucontrol->value.integer.value[0] < 0) + return -EINVAL; val = ucontrol->value.integer.value[0]; if (mc->platform_max && ((int)val + min) > mc->platform_max) return -EINVAL; if (val > max - min) return -EINVAL; - if (val < 0) - return -EINVAL; val = (val + min) & mask; if (invert) val = max - val; val_mask = mask << shift; val = val << shift; if (snd_soc_volsw_is_stereo(mc)) { + if (ucontrol->value.integer.value[1] < 0) + return -EINVAL; val2 = ucontrol->value.integer.value[1]; if (mc->platform_max && ((int)val2 + min) > mc->platform_max) return -EINVAL; if (val2 > max - min) return -EINVAL; - if (val2 < 0) - return -EINVAL; val2 = (val2 + min) & mask; if (invert) val2 = max - val2; @@ -429,13 +429,13 @@ int snd_soc_put_volsw_sx(struct snd_kcontrol *kcontrol, int err = 0; unsigned int val, val_mask, val2 = 0; + if (ucontrol->value.integer.value[0] < 0) + return -EINVAL; val = ucontrol->value.integer.value[0]; if (mc->platform_max && val > mc->platform_max) return -EINVAL; if (val > max) return -EINVAL; - if (val < 0) - return -EINVAL; val_mask = mask << shift; val = (val + min) & mask; val = val << shift; -- 2.34.1

1 0

Add SetLED macro to hide implementation details for LED handling
by GitHub pull_request - opened 11 Mar '25

11 Mar '25

alsa-project/alsa-ucm-conf pull request #517 was opened from perexg: This change also reduces modprobe calls when settings is split to multiple configuration files. Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/517 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/517.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

USB-Audio: Improve support for Focusrite 4th Gen devices
by GitHub pull_request - opened 10 Mar '25

10 Mar '25

alsa-project/alsa-ucm-conf pull request #516 was opened from perexg: There are 4 capture channels. Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/516 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/516.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH 00/14] Implement acp_common_hw_ops support for all platforms
by Venkata Prasad Potturu 10 Mar '25

10 Mar '25

This patch series is to implement acp_common_hw_ops support and refactor interrupt handler registration and interrupt handling. Venkata Prasad Potturu (14): ASoC: amd: acp: Remove redundant acp70 chip->name ASoC: amd: acp: Implement acp_common_hw_ops support for acp platforms ASoC: amd: acp: Refactor dmic-codec platform device creation ASoC: amd: acp: Refactor acp platform device creation ASoC: amd: acp: Refactor acp machine select ASoC: amd: acp: Add new interrupt handle callbacks in acp_common_hw_ops ASoC: amd: acp: Remove redundant acp_dev_data structure ASoC: amd: acp: Move spin_lock and list initialization to acp-pci driver ASoC: amd: acp: Remove white line ASoC: amd: acp: Refactor acp70 platform resource structure ASoC: amd: acp: Refactor acp63 platform resource structure ASoC: amd: acp: Refactor rembrant platform resource structure ASoC: amd: acp: Refactor renoir platform resource structure ASoC: amd: acp: Fix for enabling DMIC on acp platforms via _DSD entry sound/soc/amd/acp/acp-i2s.c | 185 +++++++++--------- sound/soc/amd/acp/acp-legacy-common.c | 254 ++++++++++++++++++------ sound/soc/amd/acp/acp-pci.c | 210 ++++++++++++-------- sound/soc/amd/acp/acp-pdm.c | 53 +++-- sound/soc/amd/acp/acp-platform.c | 145 ++++---------- sound/soc/amd/acp/acp-rembrandt.c | 111 +++-------- sound/soc/amd/acp/acp-renoir.c | 110 ++--------- sound/soc/amd/acp/acp63.c | 94 +++------ sound/soc/amd/acp/acp70.c | 84 +++----- sound/soc/amd/acp/amd.h | 270 +++++++++++++++++++++----- sound/soc/amd/acp/chip_offset_byte.h | 12 +- 11 files changed, 792 insertions(+), 736 deletions(-) -- 2.39.2

3 18

[PATCH alsa-ucm-conf] sof-soundwire: Split cs42l43 dmic initialisation
by Charles Keepax 10 Mar '25

10 Mar '25

Split the initialisation of cs42l43 into a headset part and a dmic part. This will prevent the dmic mute controls getting attached to the mute LED when the codec dmics are not being used. Signed-off-by: Charles Keepax <ckeepax(a)opensource.cirrus.com> --- ucm2/codecs/cs42l43-dmic/init.conf | 23 +++++++++++++++++++++++ ucm2/codecs/cs42l43/init.conf | 11 +---------- ucm2/sof-soundwire/sof-soundwire.conf | 2 +- 3 files changed, 25 insertions(+), 11 deletions(-) create mode 100644 ucm2/codecs/cs42l43-dmic/init.conf diff --git a/ucm2/codecs/cs42l43-dmic/init.conf b/ucm2/codecs/cs42l43-dmic/init.conf new file mode 100644 index 0000000..441d082 --- /dev/null +++ b/ucm2/codecs/cs42l43-dmic/init.conf @@ -0,0 +1,23 @@ +# cs42l43 specific control settings + +LibraryConfig.remap.Config { + ctl.default.map { + "name='cs42l43 Microphone Capture Switch'" { + "name='cs42l43 Decimator 3 Switch'".vindex.0 0 + "name='cs42l43 Decimator 4 Switch'".vindex.1 0 + } + "name='cs42l43 Microphone Capture Volume'" { + "name='cs42l43 Decimator 3 Volume'".vindex.0 0 + "name='cs42l43 Decimator 4 Volume'".vindex.1 0 + } + } +} + +FixedBootSequence [ + cset "name='cs42l43 Decimator 3 Switch' 0" + cset "name='cs42l43 Decimator 4 Switch' 0" + exec "-/sbin/modprobe snd_ctl_led" + sysw "-/class/sound/ctl-led/mic/card${CardNumber}/attach:cs42l43 Decimator 3 Switch" + sysw "-/class/sound/ctl-led/mic/card${CardNumber}/attach:cs42l43 Decimator 4 Switch" +] + diff --git a/ucm2/codecs/cs42l43/init.conf b/ucm2/codecs/cs42l43/init.conf index 0fbb517..8e41a0f 100644 --- a/ucm2/codecs/cs42l43/init.conf +++ b/ucm2/codecs/cs42l43/init.conf @@ -2,14 +2,6 @@ LibraryConfig.remap.Config { ctl.default.map { - "name='cs42l43 Microphone Capture Switch'" { - "name='cs42l43 Decimator 3 Switch'".vindex.0 0 - "name='cs42l43 Decimator 4 Switch'".vindex.1 0 - } - "name='cs42l43 Microphone Capture Volume'" { - "name='cs42l43 Decimator 3 Volume'".vindex.0 0 - "name='cs42l43 Decimator 4 Volume'".vindex.1 0 - } "name='cs42l43 Headset Microphone Capture Switch'" { "name='cs42l43 Decimator 1 Switch'".vindex.0 0 } @@ -20,8 +12,7 @@ LibraryConfig.remap.Config { } FixedBootSequence [ + cset "name='cs42l43 Decimator 1 Switch' 0" exec "-/sbin/modprobe snd_ctl_led" sysw "-/class/sound/ctl-led/mic/card${CardNumber}/attach:cs42l43 Decimator 1 Switch" - sysw "-/class/sound/ctl-led/mic/card${CardNumber}/attach:cs42l43 Decimator 3 Switch" - sysw "-/class/sound/ctl-led/mic/card${CardNumber}/attach:cs42l43 Decimator 4 Switch" ] diff --git a/ucm2/sof-soundwire/sof-soundwire.conf b/ucm2/sof-soundwire/sof-soundwire.conf index 96d2966..57e5610 100644 --- a/ucm2/sof-soundwire/sof-soundwire.conf +++ b/ucm2/sof-soundwire/sof-soundwire.conf @@ -114,7 +114,7 @@ If.hs_init { If.mic_init { Condition { Type RegexMatch - Regex "(rt712-dmic|rt713-dmic|rt715(-sdca)?)" + Regex "(cs42l43-dmic|rt712-dmic|rt713-dmic|rt715(-sdca)?)" String "${var:MicCodec1}" } True.Include.mic_init.File "/codecs/${var:MicCodec1}/init.conf" -- 2.39.5

2 1

Channel count in Focusrite Scarlett Gen 4 (1235:8218) SplitPCM configuration
by GitHub issues - edited 09 Mar '25

09 Mar '25

alsa-project/alsa-ucm-conf issue #515 was edited from pv: The configuration https://github.com/alsa-project/alsa-ucm-conf/blob/master/ucm2/USB-Audio/Fo… specifies `HWChannels 2` for the Mic1 and Mic2 devices. This gets translated to `CaptureChannels 2` for `alsaucm open '<<<SplitPCM=1>>>hw:2'`. However, we've got a report https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/4597#note_2815906 that looks like snd_pcm_hw_params_set_channels(pcm_handle, hwparams, 2) for the CapturePCM "_ucm0001.hw:Gen,0" fails with error. The PCM can be opened with 4 channels, though. Without `SplitPCM=1`, it also probably works but I'm not sure what channel count the `dshare` plugin is using. The issue also affects the `Direct` profile, which also specifies `CaptureChannels 2`. Is the `HWChannels 2` for Mic1/2 input direction right? Shall `CaptureChannels 2` in UCM device be interpreted as that the device can be opened with exactly 2 channels, or that it can be opened with some number of channels >= 2? <details> ``` $ alsaucm open '<<<SplitPCM=1>>>hw:2' dump text Verb.HiFi { Comment Default Device.Line1 { Comment "Headphones / Line 1-2" Values { CaptureCTL "_ucm0001.hw:Gen" PlaybackCTL "_ucm0001.hw:Gen" PlaybackPCM "_ucm0001.hw:Gen" PlaybackPriority 200 } } Device.Mic1 { Comment "Input 1 Mic/Inst/Line" Values { CaptureCTL "_ucm0001.hw:Gen" CaptureChannel0 0 CaptureChannelPos0 MONO CaptureChannels 2 CapturePCM "_ucm0001.hw:Gen,0" CapturePriority 200 PlaybackCTL "_ucm0001.hw:Gen" } } Device.Mic2 { Comment "Input 2 Mic/Inst/Line" Values { CaptureCTL "_ucm0001.hw:Gen" CaptureChannel0 1 CaptureChannelPos0 MONO CaptureChannels 2 CapturePCM "_ucm0001.hw:Gen,0" CapturePriority 100 PlaybackCTL "_ucm0001.hw:Gen" } } } Verb.Direct { Comment "Direct Scarlett Solo 4th Gen" Device.Direct { Comment "Direct Scarlett Solo 4th Gen" Values { CaptureCTL "_ucm0001.hw:Gen" CaptureChannels 2 CapturePCM "_ucm0001.hw:Gen" CapturePriority 1000 PlaybackCTL "_ucm0001.hw:Gen" PlaybackChannels 2 PlaybackPCM "_ucm0001.hw:Gen" PlaybackPriority 1000 } } } ``` </details> Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/515 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0