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

Device rename regression fixes
by GitHub pull_request - opened 15 May '25

15 May '25

alsa-project/alsa-ucm-conf pull request #563 was opened from jhovold: A recent change renaming device sections introduced a name collision which broke the internal microphones on the Lenovo ThinkPad X13s. A number of jack hw mute and conflicting devices properties are now also referring to non-existing devices. Please consider doing a bug fix release as soon as possible to address the X13s regression in v1.2.14. Request URL : https://github.com/alsa-project/alsa-ucm-conf/pull/563 Patch URL : https://github.com/alsa-project/alsa-ucm-conf/pull/563.patch Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH] conf: aliases: add hda-acpi -> HDA-Intel alias
by Daniel Dadap 15 May '25

15 May '25

The new snd_hda_acpi driver in Linux exposes the existing Azalia interface to non-PCI devices advertised over ACPI. Add an alias to the existing HDA-Intel configuration file so that devices using this driver can be discovered properly. Signed-off-by: Daniel Dadap <ddadap(a)nvidia.com> --- src/conf/cards/aliases.conf | 1 + 1 file changed, 1 insertion(+) diff --git a/src/conf/cards/aliases.conf b/src/conf/cards/aliases.conf index a54824ae..e2d59aa7 100644 --- a/src/conf/cards/aliases.conf +++ b/src/conf/cards/aliases.conf @@ -57,6 +57,7 @@ CMI8786 cards.CMI8788 CMI8787 cards.CMI8788 pistachio cards.pistachio-card VC4-HDMI cards.vc4-hdmi +hda-acpi cards.HDA-Intel <confdir:ctl/default.conf> <confdir:pcm/default.conf> -- 2.39.5

1 0

MIDI 2.0 Network UDP server/client programs for ALSA
by Takashi Iwai 14 May '25

14 May '25

Hi, I recently wrote a set of small programs for the new Network MIDI 2.0 UDP transport with ALSA sequencer / rawmidi API, based on v1.0 spec. The current version (v0.3) is found at: https://github.com/tiwai/amidi2net A network server can be invoked simply via: % amidi2net-server and this will create an ALSA sequencer client/port. This ALSA sequencer port can be connected by any other ALSA sequencer programs or devices. The server opens a UDP port, so that it can be connected by a client on another machine on the net, e.g.: % amidi2net-client -l amidi2net which creates an ALSA sequencer client/port similarly, and this can be connected by other apps, too. After the network connection, server and client can pass the UMP data between them freely, so they work as a bridge or hub for applications on both machines. Some details are found in README and man pages. For the builds, you'll need openssl and avahi devel packages. (It allows to build without those, but basic functions will be missing.) As it's a quite initial version, there can be bugs and subject to change. But the stuff seems working stably. Feel free to reach me if you find any issues. thanks, Takashi

1 0

[PATCH v4 0/5] Add DisplayPort sound support for Fairphone 5 smartphone
by Luca Weiss 14 May '25

14 May '25

Add the necessary sound card bits and some dts additions to enable sound over DisplayPort-over-USB-C, e.g. to a connected TV or monitor. The UCM files can be found here: https://gitlab.postmarketos.org/postmarketOS/pmaports/-/tree/master/device/… This series - in spirit - depends on the series enabling DisplayPort in the first place, but can land pretty independently, especially the ASoC bits: https://lore.kernel.org/linux-arm-msm/20250312-fp5-pmic-glink-dp-v2-0-a5592… The ASoC bits depend on the patches for USB audio offloading merged through Greg's tree so this cannot easily be applied for v6.16 through the sound tree. Either it waits for v6.17 or also goes through Greg's tree. Signed-off-by: Luca Weiss <luca.weiss(a)fairphone.com> --- Changes in v4: - Use "sm4250" for qcom,qrb4210-rb2-sndcard (Srinivas) - Expand cover letter to mention dependencies for ASoC patches - Pick up tags - Link to v3: https://lore.kernel.org/r/20250425-fp5-dp-sound-v3-0-7cb45180091b@fairphone… Changes in v3: - Simplify return qcom_snd_wcd_jack_setup in machine driver - Pick up tags - Link to v2: https://lore.kernel.org/r/20250418-fp5-dp-sound-v2-0-05d65f084b05@fairphone… Changes in v2: - Revamp series based on comments on v1, doesn't have much too much in common anymore - Use sm8250 instead of sc8280xp sndcard file, so port other required changes from sc8280xp.c to sm8250.c - This also changes the sound card compatible to from qcom,qcm6490-sndcard to fairphone,fp5-sndcard - Link to v1: https://lore.kernel.org/r/20240809-fp5-dp-sound-v1-0-d7ba2c24f6b9@fairphone… --- Luca Weiss (5): ASoC: dt-bindings: qcom,sm8250: Add Fairphone 5 sound card ASoC: qcom: sm8250: set card driver name from match data ASoC: qcom: sm8250: add DisplayPort Jack support ASoC: qcom: sm8250: Add Fairphone 5 soundcard compatible arm64: dts: qcom: qcm6490-fairphone-fp5: Add DisplayPort sound support .../devicetree/bindings/sound/qcom,sm8250.yaml | 1 + arch/arm64/boot/dts/qcom/qcm6490-fairphone-fp5.dts | 31 ++++++++++++++++++++++ sound/soc/qcom/sm8250.c | 27 ++++++++++--------- 3 files changed, 47 insertions(+), 12 deletions(-) --- base-commit: f819a667fa466d3b96b542b147053b6873f18ea1 change-id: 20240809-fp5-dp-sound-b3768f3019bd Best regards, -- Luca Weiss <luca.weiss(a)fairphone.com>

3 8

[PATCH v4] ALSA: hda/tas2781: Fix the ld issue reported by kernel test robot
by Shenghao Ding 13 May '25

13 May '25

After commit 9fa6a693ad8d ("ALSA: hda/tas2781: Remove tas2781_spi_fwlib.c and leverage SND_SOC_TAS2781_FMWLIB")created a separated lib for i2c, However, tasdevice_remove() used for not only for I2C but for SPI being still in that lib caused ld issue. All errors (new ones prefixed by >>): >> ld.lld: error: undefined symbol: tasdevice_remove >>> referenced by tas2781_hda.c:33 (sound/pci/hda/tas2781_hda.c:33) >>> vmlinux.o:(tas2781_hda_remove) To fix this issue, the implementation of tasdevice_remove was moved from tas2781-comlib-i2c.c to tas2781-comlib.c. Fixes: 9fa6a693ad8d ("ALSA: hda/tas2781: Remove tas2781_spi_fwlib.c and leverage SND_SOC_TAS2781_FMWLIB") Reported-by: kernel test robot <lkp(a)intel.com> Closes: https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/202505111… Signed-off-by: Shenghao Ding <shenghao-ding(a)ti.com> --- v4: - Put report tag around Signed-off-by without quotes. v3: - Drop the report tag in the patch description. v2: - put the Fixes tag to point out the commit that introduced the regression - Add compiling error information reported by kernel test robot into patch description. v1: - | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/202505111… --- sound/soc/codecs/tas2781-comlib-i2c.c | 6 ------ sound/soc/codecs/tas2781-comlib.c | 6 ++++++ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/sound/soc/codecs/tas2781-comlib-i2c.c b/sound/soc/codecs/tas2781-comlib-i2c.c index a75b812618ee..c078bb0a8437 100644 --- a/sound/soc/codecs/tas2781-comlib-i2c.c +++ b/sound/soc/codecs/tas2781-comlib-i2c.c @@ -364,12 +364,6 @@ int tascodec_init(struct tasdevice_priv *tas_priv, void *codec, } EXPORT_SYMBOL_GPL(tascodec_init); -void tasdevice_remove(struct tasdevice_priv *tas_priv) -{ - mutex_destroy(&tas_priv->codec_lock); -} -EXPORT_SYMBOL_GPL(tasdevice_remove); - MODULE_DESCRIPTION("TAS2781 common library for I2C"); MODULE_AUTHOR("Shenghao Ding, TI, <shenghao-ding(a)ti.com>"); MODULE_LICENSE("GPL"); diff --git a/sound/soc/codecs/tas2781-comlib.c b/sound/soc/codecs/tas2781-comlib.c index ad2f65359b15..076c1d955972 100644 --- a/sound/soc/codecs/tas2781-comlib.c +++ b/sound/soc/codecs/tas2781-comlib.c @@ -225,6 +225,12 @@ void tasdevice_dsp_remove(void *context) } EXPORT_SYMBOL_GPL(tasdevice_dsp_remove); +void tasdevice_remove(struct tasdevice_priv *tas_priv) +{ + mutex_destroy(&tas_priv->codec_lock); +} +EXPORT_SYMBOL_GPL(tasdevice_remove); + MODULE_DESCRIPTION("TAS2781 common library"); MODULE_AUTHOR("Shenghao Ding, TI, <shenghao-ding(a)ti.com>"); MODULE_LICENSE("GPL"); -- 2.43.0

3 3

[PATCH] ASoC: tlv320aic3x: Use dev_err_probe
by Nishanth Menon 13 May '25

13 May '25

During probe the regulator supply drivers may not yet be available. Use dev_err_probe to provide just the pertinent log. Signed-off-by: Nishanth Menon <nm(a)ti.com> --- Testing based on next-20250512 (includes additional patch) on AM62a7-sk Before: https://gist.github.com/nmenon/be94f21e83b4260ad3f89e1ae8f0d188#file-before… After: https://gist.github.com/nmenon/be94f21e83b4260ad3f89e1ae8f0d188#file-after-… sound/soc/codecs/tlv320aic3x.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c index 56e795a00e22..f1649df19738 100644 --- a/sound/soc/codecs/tlv320aic3x.c +++ b/sound/soc/codecs/tlv320aic3x.c @@ -1818,10 +1818,8 @@ int aic3x_probe(struct device *dev, struct regmap *regmap, kernel_ulong_t driver ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(aic3x->supplies), aic3x->supplies); - if (ret) { - dev_err(dev, "Failed to request supplies: %d\n", ret); - return ret; - } + if (ret) + return dev_err_probe(dev, ret, "Failed to request supplies\n"); aic3x_configure_ocmv(dev, aic3x); -- 2.47.0

2 1

Re: [Bug 220109] Audio disappears on HP 15-fc000 after warm boot again (ALC256 code path)
by Ruslan Nikolaev 13 May '25

13 May '25

Kailiang, I added alsa-info outputs to https://bugzilla.kernel.org/show_bug.cgi?id=220109 (see attachments for both the regression and reverted changes) by following your instructions. Please let me know if there is any other information you need. Thanks, Ruslan On Tuesday, May 13, 2025 at 02:30:17 AM EDT, <bugzilla-daemon(a)kernel.org> wrote: https://bugzilla.kernel.org/show_bug.cgi?id=220109 Kailiang Yang (kailang(a)realtek.com) changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kailang(a)realtek.com --- Comment #3 from Kailiang Yang (kailang(a)realtek.com) --- Could you run alsa-info.sh to dump codec info? Please do ==> echo 1 > /sys/module/snd_hda_codec/parameters/dump_coef before do alsa-info.sh. -- You may reply to this email to add a comment. You are receiving this mail because: You are on the CC list for the bug. You reported the bug.

1 0

[PATCH v1] ALSA: hda/tas2781: Fix the symbol was not declare warning reported by kernel test robot
by Shenghao Ding 13 May '25

13 May '25

After commit 28a09d9e42c5 ("ALSA: hda/tas2781: Create an independent lib to save the shared parts for both SPI and I2C driver") created a separated lib for both hda-i2c and hda-spi driver, which caused this issue. sparse warnings: (new ones prefixed by >>) >> sound/pci/hda/tas2781_hda_spi.c:496:25: sparse: sparse: symbol 'tas2781_snd_ctls' was not declared. Should it be static? >> sound/pci/hda/tas2781_hda_spi.c:505:25: sparse: sparse: symbol 'tas2781_prof_ctl' was not declared. Should it be static? >> sound/pci/hda/tas2781_hda_spi.c:512:25: sparse: sparse: symbol 'tas2781_dsp_ctls' was not declared. Should it be static? Fixes: 28a09d9e42c5 ("ALSA: hda/tas2781: Create an independent lib to save the shared parts for both SPI and I2C driver") Reported-by: kernel test robot <lkp(a)intel.com> Closes: https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/202505130… Signed-off-by: Shenghao Ding <shenghao-ding(a)ti.com> --- v1: - | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/202505130… --- sound/pci/hda/tas2781_hda_spi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/pci/hda/tas2781_hda_spi.c b/sound/pci/hda/tas2781_hda_spi.c index 50b82ecabb90..e0f8603c4854 100644 --- a/sound/pci/hda/tas2781_hda_spi.c +++ b/sound/pci/hda/tas2781_hda_spi.c @@ -493,7 +493,7 @@ static int tas2781_force_fwload_put(struct snd_kcontrol *kcontrol, return change; } -struct snd_kcontrol_new tas2781_snd_ctls[] = { +static struct snd_kcontrol_new tas2781_snd_ctls[] = { ACARD_SINGLE_RANGE_EXT_TLV(NULL, TAS2781_AMP_LEVEL, 1, 0, 20, 0, tas2781_amp_getvol, tas2781_amp_putvol, amp_vol_tlv), ACARD_SINGLE_RANGE_EXT_TLV(NULL, TAS2781_DVC_LVL, 0, 0, 200, 1, @@ -502,14 +502,14 @@ struct snd_kcontrol_new tas2781_snd_ctls[] = { tas2781_force_fwload_put), }; -struct snd_kcontrol_new tas2781_prof_ctl = { +static struct snd_kcontrol_new tas2781_prof_ctl = { .iface = SNDRV_CTL_ELEM_IFACE_CARD, .info = tasdevice_info_profile, .get = tasdevice_get_profile_id, .put = tasdevice_set_profile_id, }; -struct snd_kcontrol_new tas2781_dsp_ctls[] = { +static struct snd_kcontrol_new tas2781_dsp_ctls[] = { /* Speaker Program */ { .iface = SNDRV_CTL_ELEM_IFACE_CARD, -- 2.43.0

2 1

[PATCH v3] ALSA: hda/tas2781: Fix the ld issue reported by kernel test robot
by Shenghao Ding 13 May '25

13 May '25

After commit 9fa6a693ad8d ("ALSA: hda/tas2781: Remove tas2781_spi_fwlib.c and leverage SND_SOC_TAS2781_FMWLIB")created a separated lib for i2c, However, tasdevice_remove() used for not only for I2C but for SPI being still in that lib caused ld issue. All errors (new ones prefixed by >>): >> ld.lld: error: undefined symbol: tasdevice_remove >>> referenced by tas2781_hda.c:33 (sound/pci/hda/tas2781_hda.c:33) >>> vmlinux.o:(tas2781_hda_remove) To fix this issue, the implementation of tasdevice_remove was moved from tas2781-comlib-i2c.c to tas2781-comlib.c. Fixes: 9fa6a693ad8d ("ALSA: hda/tas2781: Remove tas2781_spi_fwlib.c and leverage SND_SOC_TAS2781_FMWLIB") Signed-off-by: Shenghao Ding <shenghao-ding(a)ti.com> --- v3: - Drop the report tag in the patch description. v2: - put the Fixes tag to point out the commit that introduced the regression - Add compiling error information reported by kernel test robot into patch description. v1: - | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://urldefense.com/v3/__https://lore.kernel.org/oe-kbuild-all/202505111… --- sound/soc/codecs/tas2781-comlib-i2c.c | 6 ------ sound/soc/codecs/tas2781-comlib.c | 6 ++++++ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/sound/soc/codecs/tas2781-comlib-i2c.c b/sound/soc/codecs/tas2781-comlib-i2c.c index a75b812618ee..c078bb0a8437 100644 --- a/sound/soc/codecs/tas2781-comlib-i2c.c +++ b/sound/soc/codecs/tas2781-comlib-i2c.c @@ -364,12 +364,6 @@ int tascodec_init(struct tasdevice_priv *tas_priv, void *codec, } EXPORT_SYMBOL_GPL(tascodec_init); -void tasdevice_remove(struct tasdevice_priv *tas_priv) -{ - mutex_destroy(&tas_priv->codec_lock); -} -EXPORT_SYMBOL_GPL(tasdevice_remove); - MODULE_DESCRIPTION("TAS2781 common library for I2C"); MODULE_AUTHOR("Shenghao Ding, TI, <shenghao-ding(a)ti.com>"); MODULE_LICENSE("GPL"); diff --git a/sound/soc/codecs/tas2781-comlib.c b/sound/soc/codecs/tas2781-comlib.c index ad2f65359b15..076c1d955972 100644 --- a/sound/soc/codecs/tas2781-comlib.c +++ b/sound/soc/codecs/tas2781-comlib.c @@ -225,6 +225,12 @@ void tasdevice_dsp_remove(void *context) } EXPORT_SYMBOL_GPL(tasdevice_dsp_remove); +void tasdevice_remove(struct tasdevice_priv *tas_priv) +{ + mutex_destroy(&tas_priv->codec_lock); +} +EXPORT_SYMBOL_GPL(tasdevice_remove); + MODULE_DESCRIPTION("TAS2781 common library"); MODULE_AUTHOR("Shenghao Ding, TI, <shenghao-ding(a)ti.com>"); MODULE_LICENSE("GPL"); -- 2.43.0

2 1

Fwd: [Bug 220109] New: Audio disappears on HP 15-fc000 after warm boot again (ALC256 code path)
by Ruslan Nikolaev 13 May '25

13 May '25

Hi, Forwarding my regression bug report for HP 15-fc000 / ALC236 (ALC256 code path). #regzbot introduced: v6.12.2 https://bugzilla.kernel.org/show_bug.cgi?id=220109 Sincerely, Ruslan Forwarded Message (see logs in the bug report above): There was a similar bug in the past (Bug 217440), which was fixed for this laptop. (I believe the code path is for ALC256, see below.) The same issue is occurring again as of kernel v.6.12.2. The symptoms are very similar - initially audio works but after a warm reboot, the audio completely disappears until the computer is powered off (there is no audio output at all). The issue is caused by a different change now. By bisecting different kernel versions, I found that reverting cc3d0b5dd989d3238d456f9fd385946379a9c13d in patch_realtek.c ( see https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/sou… ) restores the sound and it works fine after the reboot.

2 2