[alsa-devel] [PATCH] ALSA: hda - fix the Headset Mic Jack name for Realtek

[alsa-devel] [PATCH] ALSA: hda - fix the Headset Mic Jack name for Realtek

[Jaroslav Kysela]

The HDA auto parser creates kcontrol 'Headset Mic Jack'. Use this
name also for the alc_fixup_headset_jack() function. It is necessary
to have common name for the user space like the UCM configs:

https://github.com/alsa-project/alsa-ucm-conf/pull/5

Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Cc: Kailang Yang <kailang@realtek.com>
---
 sound/pci/hda/patch_realtek.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 80f66ba85f87..2633e45144d5 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -5679,7 +5679,7 @@ static void alc_fixup_headset_jack(struct hda_codec *codec,
 	case HDA_FIXUP_ACT_PRE_PROBE:
 		snd_hda_jack_detect_enable_callback(codec, 0x55,
 						    alc_headset_btn_callback);
-		snd_hda_jack_add_kctl(codec, 0x55, "Headset Jack", false,
+		snd_hda_jack_add_kctl(codec, 0x55, "Headset Mic Jack", false,
 				      SND_JACK_HEADSET, alc_headset_btn_keymap);
 		break;
 	case HDA_FIXUP_ACT_INIT:
-- 
2.20.1
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [alsa-devel] [PATCH] ALSA: hda - fix the Headset Mic Jack name for Realtek

[Takashi Iwai]

On Fri, 01 Nov 2019 12:57:18 +0100,
Jaroslav Kysela wrote:
> 
> The HDA auto parser creates kcontrol 'Headset Mic Jack'. Use this
> name also for the alc_fixup_headset_jack() function. It is necessary
> to have common name for the user space like the UCM configs:
> 
> https://github.com/alsa-project/alsa-ucm-conf/pull/5

Please make sure that this really doesn't break the existing device
especially with PulseAudio.  I'll merge this once when it's confirmed
that it still works as is after the change.


thanks,

Takashi


> 
> Signed-off-by: Jaroslav Kysela <perex@perex.cz>
> Cc: Kailang Yang <kailang@realtek.com>
> ---
>  sound/pci/hda/patch_realtek.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index 80f66ba85f87..2633e45144d5 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -5679,7 +5679,7 @@ static void alc_fixup_headset_jack(struct hda_codec *codec,
>  	case HDA_FIXUP_ACT_PRE_PROBE:
>  		snd_hda_jack_detect_enable_callback(codec, 0x55,
>  						    alc_headset_btn_callback);
> -		snd_hda_jack_add_kctl(codec, 0x55, "Headset Jack", false,
> +		snd_hda_jack_add_kctl(codec, 0x55, "Headset Mic Jack", false,
>  				      SND_JACK_HEADSET, alc_headset_btn_keymap);
>  		break;
>  	case HDA_FIXUP_ACT_INIT:
> -- 
> 2.20.1
> 
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [alsa-devel] [PATCH] ALSA: hda - fix the Headset Mic Jack name for Realtek

[Jaroslav Kysela]

Dne 01. 11. 19 v 13:30 Takashi Iwai napsal(a):
> On Fri, 01 Nov 2019 12:57:18 +0100,
> Jaroslav Kysela wrote:
>>
>> The HDA auto parser creates kcontrol 'Headset Mic Jack'. Use this
>> name also for the alc_fixup_headset_jack() function. It is necessary
>> to have common name for the user space like the UCM configs:
>>
>> https://github.com/alsa-project/alsa-ucm-conf/pull/5
> 
> Please make sure that this really doesn't break the existing device
> especially with PulseAudio.  I'll merge this once when it's confirmed
> that it still works as is after the change.

Just for the reference: Please, drop this patch. I mangled the names somehow 
(Headset / Headphone). The Headphone Jack names are not consistent, too. We 
are going to handle this in UCM (user space) rather than the driver side.

						Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel