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

Internal microphone (Conexant SN6140) detected but shows inactive/unplugged on Lenovo IdeaPad Slim 5 16AKP10 83HY after alsa-ucm-conf 1.2.14
by GitHub issues - edited 12 Sep '25

12 Sep '25

alsa-project/alsa-ucm-conf issue #612 was edited from JayJaber: The internal microphone on a Lenovo IdeaPad Slim 5 16AKP10 83HY with Conexant SN6140 codec behaves as follows: On alsa-ucm-conf 1.2.14 and newer: - Family 17h/19h/1ah HD Audio Controller Digital Microphone appears as a recording device but does not record. Only a small bit of static is audible at the start and end of recordings. - Family 17h/19h/1ah HD Audio Controller Stereo Microphone appears as inactive/unplugged. When headphones are plugged in, it becomes active and the headphone microphone works under that device. After downgrading to alsa-ucm-conf 1.2.13-2: - Both Family 17h/19h/1ah HD Audio Controller Internal Stereo Microphone and Family 17h/19h/1ah HD Audio Controller Digital Microphone appear as active recording devices without headphones plugged in. - Digital Microphone still does not record usable audio, only a small bit of static at the beginning and end. - Internal Stereo Microphone works correctly for the built-in mic when headphones are not plugged in. Plugging in headphones causes the headphone microphone to be used under the Internal Stereo Microphone device. Summary: The Digital Microphone is detected but never records usable audio. On 1.2.13-2, the Internal Stereo Microphone routes audio correctly from the built-in mic, so recording works, but the Digital Microphone itself remains nonfunctional. On 1.2.14+: - Digital Microphone appears but remains completely nonfunctional. - Stereo Microphone shows as inactive/unplugged until headphones are connected, making the internal microphone unusable. - Headphone microphone routing works correctly. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/612 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[PATCH v5 1/5] ASoC: ops: improve snd_soc_get_volsw
by Niranjan H Y 12 Sep '25

12 Sep '25

* add error handling in case register read fails * clamp the values if the register value read is out of range Signed-off-by: Niranjan H Y <niranjan.hy(a)ti.com> -- v5: - remove clamp parameter - move the boundary check after sign-bit extension --- sound/soc/soc-ops.c | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/sound/soc/soc-ops.c b/sound/soc/soc-ops.c index a629e0eac..03f74c32d 100644 --- a/sound/soc/soc-ops.c +++ b/sound/soc/soc-ops.c @@ -118,6 +118,7 @@ static int soc_mixer_reg_to_ctl(struct soc_mixer_control *mc, unsigned int reg_v if (mc->sign_bit) val = sign_extend32(val, mc->sign_bit); + val = clamp(val, mc->min, mc->max); val -= mc->min; if (mc->invert) @@ -249,9 +250,15 @@ static int soc_get_volsw(struct snd_kcontrol *kcontrol, { struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); unsigned int reg_val; - int val; + int val, ret = 0; reg_val = snd_soc_component_read(component, mc->reg); + val = reg_val; + if (val < 0) { + ret = val; + goto get_volsw_done; + } + val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->shift, max); ucontrol->value.integer.value[0] = val; @@ -261,13 +268,19 @@ static int soc_get_volsw(struct snd_kcontrol *kcontrol, val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->rshift, max); } else { reg_val = snd_soc_component_read(component, mc->rreg); + val = reg_val; + if (val < 0) { + ret = val; + goto get_volsw_done; + } val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->shift, max); } ucontrol->value.integer.value[1] = val; } - return 0; +get_volsw_done: + return ret; } /** -- 2.45.2

2 6

Re: [PATCH v4 2/6] ASoC: ops: add snd_soc_get_volsw_clamped
by Holalu Yogendra, Niranjan 12 Sep '25

12 Sep '25

> From: Mark Brown <broonie(a)kernel.org> > Sent: Friday, September 12, 2025 12:02 AM > Subject: Re: [EXTERNAL] Re: [PATCH v4 2/6] ASoC: ops: add > snd_soc_get_volsw_clamped > > On Thu, Sep 11, 2025 at 05:21:14PM +0000, Holalu Yogendra, Niranjan wrote: > > > From: Mark Brown <broonie(a)kernel.org> > > > Since few of the drivers as still using the original API, and was not sure if > > everyone wanted to clamp the register value, did not want to disturb the > > original API snd_soc_get_volsw. Per my analysis, if register read fails due to > In what situation would anything want to report an invalid value? I am not sure if I fully follow. But in my case, for one of the cases, valid amp volume is 0x0 to 0x14 with mask 0x1f - so if the register value read is 0x15 to 0x1f, it is invalid. But in the next patch, I will add the logic to clamp the value. > > Are you thinking that this patch could be dropped and clamp the value to max in > > 'soc_get_volsw' (as in 1st patch) and use the same API ? Please suggest. > Yes, everything should report valid values. I/O errors should be > propagated as errors. I will update the next patch accordingly. Thanks Niranjan H Y

1 0

Lenovo Yoga 7 2-in-1 16AKP10 volume control bug
by Sandro Gähler 12 Sep '25

12 Sep '25

Hello I would like to report a bug for the brand new Lenovo Yoga 7 2-in-1 16AKP10. It has the same sound chip and also the same volume control bug as several older models. I thus assume that the same workaround, written by Philipp Jungkamp and later by Pascal Gross, will also work for this model. There are several lines with laptops that apply this workaround, for example this line in linux/sound/pci/hda/patch_realtek.c: SND_PCI_QUIRK(0x17aa, 0x3801, "Lenovo Yoga9 14IAP7", ALC287_FIXUP_YOGA9_14IAP7_BASS_SPK_PIN) Could you please add this line for my model? SND_PCI_QUIRK(0x????, 0x????, "Lenovo Yoga 7 2-in-1 16AKP10", ALC287_FIXUP_YOGA9_14IAP7_BASS_SPK_PIN), Please advise on how to determine the first two parameters. Thanks a lot for your great work and support. Best regards, --Sandro Gähler

1 1

[PATCH v4 1/6] ASoC: ops: improve snd_soc_get_volsw
by Niranjan H Y 11 Sep '25

11 Sep '25

* add error handling in case register read fails * add support for clamping the values if the register value read is greater than max value Signed-off-by: Niranjan H Y <niranjan.hy(a)ti.com> --- sound/soc/soc-ops.c | 38 +++++++++++++++++++++++++++++--------- 1 file changed, 29 insertions(+), 9 deletions(-) diff --git a/sound/soc/soc-ops.c b/sound/soc/soc-ops.c index a629e0eac..59e91741b 100644 --- a/sound/soc/soc-ops.c +++ b/sound/soc/soc-ops.c @@ -111,10 +111,14 @@ int snd_soc_put_enum_double(struct snd_kcontrol *kcontrol, EXPORT_SYMBOL_GPL(snd_soc_put_enum_double); static int soc_mixer_reg_to_ctl(struct soc_mixer_control *mc, unsigned int reg_val, - unsigned int mask, unsigned int shift, int max) + unsigned int mask, unsigned int shift, int max, + bool clamp) { int val = (reg_val >> shift) & mask; + if (clamp) + val = clamp(val, 0, mc->max); + if (mc->sign_bit) val = sign_extend32(val, mc->sign_bit); @@ -245,29 +249,44 @@ static int soc_put_volsw(struct snd_kcontrol *kcontrol, static int soc_get_volsw(struct snd_kcontrol *kcontrol, struct snd_ctl_elem_value *ucontrol, - struct soc_mixer_control *mc, int mask, int max) + struct soc_mixer_control *mc, int mask, int max, + bool clamp) { struct snd_soc_component *component = snd_kcontrol_chip(kcontrol); unsigned int reg_val; - int val; + int val, ret = 0; reg_val = snd_soc_component_read(component, mc->reg); - val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->shift, max); + val = reg_val; + if (val < 0) { + ret = val; + goto get_volsw_done; + } + + val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->shift, max, clamp); ucontrol->value.integer.value[0] = val; if (snd_soc_volsw_is_stereo(mc)) { if (mc->reg == mc->rreg) { - val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->rshift, max); + val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->rshift, + max, clamp); } else { reg_val = snd_soc_component_read(component, mc->rreg); - val = soc_mixer_reg_to_ctl(mc, reg_val, mask, mc->shift, max); + val = reg_val; + if (val < 0) { + ret = val; + goto get_volsw_done; + } + val = soc_mixer_reg_to_ctl(mc, reg_val, mask, + mc->shift, max, clamp); } ucontrol->value.integer.value[1] = val; } - return 0; +get_volsw_done: + return ret; } /** @@ -330,7 +349,8 @@ int snd_soc_get_volsw(struct snd_kcontrol *kcontrol, (struct soc_mixer_control *)kcontrol->private_value; unsigned int mask = soc_mixer_mask(mc); - return soc_get_volsw(kcontrol, ucontrol, mc, mask, mc->max - mc->min); + return soc_get_volsw(kcontrol, ucontrol, mc, mask, + mc->max - mc->min, false); } EXPORT_SYMBOL_GPL(snd_soc_get_volsw); @@ -372,7 +392,7 @@ int snd_soc_get_volsw_sx(struct snd_kcontrol *kcontrol, (struct soc_mixer_control *)kcontrol->private_value; unsigned int mask = soc_mixer_sx_mask(mc); - return soc_get_volsw(kcontrol, ucontrol, mc, mask, mc->max); + return soc_get_volsw(kcontrol, ucontrol, mc, mask, mc->max, false); } EXPORT_SYMBOL_GPL(snd_soc_get_volsw_sx); -- 2.45.2

3 7

[PATCH v1] ALSA: hda/tas2781: Fix a potential race condition that causes a NULL pointer in case no efi.get_variable exsits
by Shenghao Ding 11 Sep '25

11 Sep '25

A a potential race condition reported by one of my customers that leads to a NULL pointer dereference, where the call to efi.get_variable should be guarded with efi_rt_services_supported() to ensure that function exists. Fixes: 4fe238513407 ("ALSA: hda/tas2781: Move and unified the calibrated-data getting function for SPI and I2C into the tas2781_hda lib") Signed-off-by: Shenghao Ding <shenghao-ding(a)ti.com> --- sound/hda/codecs/side-codecs/tas2781_hda.c | 5 +++++ sound/hda/codecs/side-codecs/tas2781_hda_i2c.c | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/sound/hda/codecs/side-codecs/tas2781_hda.c b/sound/hda/codecs/side-codecs/tas2781_hda.c index 536940c78f00..96e6d82dc69e 100644 --- a/sound/hda/codecs/side-codecs/tas2781_hda.c +++ b/sound/hda/codecs/side-codecs/tas2781_hda.c @@ -193,6 +193,11 @@ int tas2781_save_calibration(struct tas2781_hda *hda) efi_status_t status; int i; + if (!efi_rt_services_supported(EFI_RT_SUPPORTED_GET_VARIABLE)) { + dev_err(p->dev, "%s: NO EFI FOUND!\n", __func__); + return -EINVAL; + } + if (hda->catlog_id < LENOVO) efi_guid = tasdev_fct_efi_guid[hda->catlog_id]; diff --git a/sound/hda/codecs/side-codecs/tas2781_hda_i2c.c b/sound/hda/codecs/side-codecs/tas2781_hda_i2c.c index 008dbe1490a7..4dea442d8c30 100644 --- a/sound/hda/codecs/side-codecs/tas2781_hda_i2c.c +++ b/sound/hda/codecs/side-codecs/tas2781_hda_i2c.c @@ -317,6 +317,11 @@ static int tas2563_save_calibration(struct tas2781_hda *h) unsigned int attr; int ret, i, j, k; + if (!efi_rt_services_supported(EFI_RT_SUPPORTED_GET_VARIABLE)) { + dev_err(p->dev, "%s: NO EFI FOUND!\n", __func__); + return -EINVAL; + } + cd->cali_dat_sz_per_dev = TAS2563_CAL_DATA_SIZE * TASDEV_CALIB_N; /* extra byte for each device is the device number */ -- 2.43.0

2 1

Internal microphone (Conexant SN6140) detected but shows inactive/unplugged on Lenovo IdeaPad Slim 5 16AKP10 83HY after alsa-ucm-conf 1.2.14
by GitHub issues - edited 11 Sep '25

11 Sep '25

alsa-project/alsa-ucm-conf issue #612 was edited from JayJaber: On a Lenovo IdeaPad Slim 5 16AKP10 83HY with Conexant SN6140 codec, the internal microphone works correctly with alsa-ucm-conf 1.2.13-2, but with 1.2.14 and newer it is detected but shows up as inactive/unplugged. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/612 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

Internal microphone (Conexant SN6140) detected but shows inactive/unplugged on Lenovo IdeaPad Slim 5 16AKP10 83HY after alsa-ucm-conf 1.2.14
by GitHub issues - edited 11 Sep '25

11 Sep '25

alsa-project/alsa-ucm-conf issue #612 was edited from JayJaber: On a Lenovo IdeaPad Slim 5 16AKP10 83HY with Conexant SN6140 codec, the internal microphone works correctly with alsa-ucm-conf 1.2.13-2, but with 1.2.14 and newer it is not detected and shows up as inactive/unplugged. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/612 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

Internal microphone (Conexant SN6140) not detected on Lenovo IdeaPad Slim 5 16AKP10 83HY after alsa-ucm-conf 1.2.14
by GitHub issues - opened 11 Sep '25

11 Sep '25

alsa-project/alsa-ucm-conf issue #612 was opened from JayJaber: On a Lenovo IdeaPad Slim 5 16AKP10 83HY with Conexant SN6140 codec, the internal microphone works correctly with alsa-ucm-conf 1.2.13-2, but with 1.2.14 and newer it is not detected and shows up as inactive/unplugged. Issue URL : https://github.com/alsa-project/alsa-ucm-conf/issues/612 Repository URL: https://github.com/alsa-project/alsa-ucm-conf

1 0

[tiwai-sound:test/auto-cleanup 208/210] sound/sparc/amd7930.c:357:45: error: 'flags' undeclared
by kernel test robot 10 Sep '25

10 Sep '25

tree: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git test/auto-cleanup head: f29244cff0185975fee4f55293cca61cfd83af3e commit: db8ed7441c876cac679db2869b87b26633c89bde [208/210] ALSA: sparc/amd7930: Use guard() for spin locks config: sparc64-randconfig-002-20250910 (https://download.01.org/0day-ci/archive/20250911/202509110130.PlGyYo4k-lkp@…) compiler: sparc64-linux-gcc (GCC) 12.5.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250911/202509110130.PlGyYo4k-lkp@…) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp(a)intel.com> | Closes: https://lore.kernel.org/oe-kbuild-all/202509110130.PlGyYo4k-lkp@intel.com/ All errors (new ones prefixed by >>): sound/sparc/amd7930.c: In function 'amd7930_enable_ints': >> sound/sparc/amd7930.c:357:45: error: 'flags' undeclared (first use in this function) 357 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ sound/sparc/amd7930.c:357:45: note: each undeclared identifier is reported only once for each function it appears in In file included from include/linux/module.h:17, from sound/sparc/amd7930.c:33: >> include/linux/cleanup.h:291:17: error: too many arguments to function 'class_spinlock_irqsave_constructor' 291 | class_##_name##_constructor | ^~~~~~ include/linux/cleanup.h:401:9: note: in expansion of macro 'CLASS' 401 | CLASS(_name, __UNIQUE_ID(guard)) | ^~~~~ sound/sparc/amd7930.c:357:9: note: in expansion of macro 'guard' 357 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ include/linux/cleanup.h:477:33: note: declared here 477 | static inline class_##_name##_t class_##_name##_constructor(_type *l) \ | ^~~~~~ include/linux/cleanup.h:496:1: note: in expansion of macro '__DEFINE_LOCK_GUARD_1' 496 | __DEFINE_LOCK_GUARD_1(_name, _type, _lock) | ^~~~~~~~~~~~~~~~~~~~~ include/linux/spinlock.h:585:1: note: in expansion of macro 'DEFINE_LOCK_GUARD_1' 585 | DEFINE_LOCK_GUARD_1(spinlock_irqsave, spinlock_t, | ^~~~~~~~~~~~~~~~~~~ sound/sparc/amd7930.c: In function 'amd7930_disable_ints': sound/sparc/amd7930.c:365:45: error: 'flags' undeclared (first use in this function) 365 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ >> include/linux/cleanup.h:291:17: error: too many arguments to function 'class_spinlock_irqsave_constructor' 291 | class_##_name##_constructor | ^~~~~~ include/linux/cleanup.h:401:9: note: in expansion of macro 'CLASS' 401 | CLASS(_name, __UNIQUE_ID(guard)) | ^~~~~ sound/sparc/amd7930.c:365:9: note: in expansion of macro 'guard' 365 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ include/linux/cleanup.h:477:33: note: declared here 477 | static inline class_##_name##_t class_##_name##_constructor(_type *l) \ | ^~~~~~ include/linux/cleanup.h:496:1: note: in expansion of macro '__DEFINE_LOCK_GUARD_1' 496 | __DEFINE_LOCK_GUARD_1(_name, _type, _lock) | ^~~~~~~~~~~~~~~~~~~~~ include/linux/spinlock.h:585:1: note: in expansion of macro 'DEFINE_LOCK_GUARD_1' 585 | DEFINE_LOCK_GUARD_1(spinlock_irqsave, spinlock_t, | ^~~~~~~~~~~~~~~~~~~ -- amd7930.c: In function 'amd7930_enable_ints': amd7930.c:357:45: error: 'flags' undeclared (first use in this function) 357 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ amd7930.c:357:45: note: each undeclared identifier is reported only once for each function it appears in In file included from include/linux/module.h:17, from amd7930.c:33: >> include/linux/cleanup.h:291:17: error: too many arguments to function 'class_spinlock_irqsave_constructor' 291 | class_##_name##_constructor | ^~~~~~ include/linux/cleanup.h:401:9: note: in expansion of macro 'CLASS' 401 | CLASS(_name, __UNIQUE_ID(guard)) | ^~~~~ amd7930.c:357:9: note: in expansion of macro 'guard' 357 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ include/linux/cleanup.h:477:33: note: declared here 477 | static inline class_##_name##_t class_##_name##_constructor(_type *l) \ | ^~~~~~ include/linux/cleanup.h:496:1: note: in expansion of macro '__DEFINE_LOCK_GUARD_1' 496 | __DEFINE_LOCK_GUARD_1(_name, _type, _lock) | ^~~~~~~~~~~~~~~~~~~~~ include/linux/spinlock.h:585:1: note: in expansion of macro 'DEFINE_LOCK_GUARD_1' 585 | DEFINE_LOCK_GUARD_1(spinlock_irqsave, spinlock_t, | ^~~~~~~~~~~~~~~~~~~ amd7930.c: In function 'amd7930_disable_ints': amd7930.c:365:45: error: 'flags' undeclared (first use in this function) 365 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ >> include/linux/cleanup.h:291:17: error: too many arguments to function 'class_spinlock_irqsave_constructor' 291 | class_##_name##_constructor | ^~~~~~ include/linux/cleanup.h:401:9: note: in expansion of macro 'CLASS' 401 | CLASS(_name, __UNIQUE_ID(guard)) | ^~~~~ amd7930.c:365:9: note: in expansion of macro 'guard' 365 | guard(spinlock_irqsave)(&amd->lock, flags); | ^~~~~ include/linux/cleanup.h:477:33: note: declared here 477 | static inline class_##_name##_t class_##_name##_constructor(_type *l) \ | ^~~~~~ include/linux/cleanup.h:496:1: note: in expansion of macro '__DEFINE_LOCK_GUARD_1' 496 | __DEFINE_LOCK_GUARD_1(_name, _type, _lock) | ^~~~~~~~~~~~~~~~~~~~~ include/linux/spinlock.h:585:1: note: in expansion of macro 'DEFINE_LOCK_GUARD_1' 585 | DEFINE_LOCK_GUARD_1(spinlock_irqsave, spinlock_t, | ^~~~~~~~~~~~~~~~~~~ vim +/flags +357 sound/sparc/amd7930.c 353 354 /* Enable chip interrupts. The amd->lock is not held. */ 355 static __inline__ void amd7930_enable_ints(struct snd_amd7930 *amd) 356 { > 357 guard(spinlock_irqsave)(&amd->lock, flags); 358 sbus_writeb(AMR_INIT, amd->regs + AMD7930_CR); 359 sbus_writeb(AM_INIT_ACTIVE, amd->regs + AMD7930_DR); 360 } 361 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki

1 0