diff mbox series

[1/1] ALSA: usb-audio: Disable Lenovo P620 Rear line-in volume control

Message ID 20200812082851.14274-2-kai.heng.feng@canonical.com
State New
Headers show
Series Disable Lenovo P620 Rear line-in volume control | expand

Commit Message

Kai-Heng Feng Aug. 12, 2020, 8:28 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1891281

The USB device (0x17aa:0x1046) that support Lenovo P620 rear panel
line-in claim to support volume control, but it doens't seem to have an
AMP, so when line-in volume lowers below 80, nothing gets recorded
anymore.

Disable the volume control to workaround the issue.

Fixes: f8c11eb7da4a ("ALSA: usb-audio: Add support for Lenovo ThinkStation P620")
Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Link: https://lore.kernel.org/r/20200810133108.31580-1-kai.heng.feng@canonical.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
(cherry picked from commit 34dedd2a83b241ba6aeb290260313c65dc58660e linux-next)
Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
---
 sound/usb/mixer_maps.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Stefan Bader Aug. 13, 2020, 9:07 a.m. UTC | #1
On 12.08.20 10:28, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1891281
> 
> The USB device (0x17aa:0x1046) that support Lenovo P620 rear panel
> line-in claim to support volume control, but it doens't seem to have an
> AMP, so when line-in volume lowers below 80, nothing gets recorded
> anymore.
> 
> Disable the volume control to workaround the issue.
> 
> Fixes: f8c11eb7da4a ("ALSA: usb-audio: Add support for Lenovo ThinkStation P620")
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
> Link: https://lore.kernel.org/r/20200810133108.31580-1-kai.heng.feng@canonical.com
> Signed-off-by: Takashi Iwai <tiwai@suse.de>
> (cherry picked from commit 34dedd2a83b241ba6aeb290260313c65dc58660e linux-next)
> Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Acked-by: Stefan Bader <stefan.bader@canonical.com>
> ---

on the better-regression-potential mission: maybe I would say

The change modifies volume controls of a specific device. Potentially this could
lead to unexpected effects controlling volumes for that device.

-Stefan

>  sound/usb/mixer_maps.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/sound/usb/mixer_maps.c b/sound/usb/mixer_maps.c
> index 08eb230e2bff..7e7afd71b356 100644
> --- a/sound/usb/mixer_maps.c
> +++ b/sound/usb/mixer_maps.c
> @@ -371,6 +371,7 @@ static const struct usbmix_name_map asus_rog_map[] = {
>  };
>  
>  static const struct usbmix_name_map lenovo_p620_rear_map[] = {
> +	{ 19, NULL, 2 }, /* FU, Volume */
>  	{ 19, NULL, 12 }, /* FU, Input Gain Pad */
>  	{}
>  };
>
diff mbox series

Patch

diff --git a/sound/usb/mixer_maps.c b/sound/usb/mixer_maps.c
index 08eb230e2bff..7e7afd71b356 100644
--- a/sound/usb/mixer_maps.c
+++ b/sound/usb/mixer_maps.c
@@ -371,6 +371,7 @@  static const struct usbmix_name_map asus_rog_map[] = {
 };
 
 static const struct usbmix_name_map lenovo_p620_rear_map[] = {
+	{ 19, NULL, 2 }, /* FU, Volume */
 	{ 19, NULL, 12 }, /* FU, Input Gain Pad */
 	{}
 };