Commit c1b55029 authored by Daniel Cordova A's avatar Daniel Cordova A Committed by Takashi Iwai
Browse files

ALSA: hda: fixup headset for ASUS GU502 laptop 



The GU502 requires a few steps to make headset i/o works properly:
pincfg, verbs to unmute headphone out and callback to toggle output
between speakers and headphone using jack.

Signed-off-by: default avatarDaniel Cordova A <danesc87@gmail.com>
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/20210507173116.12043-1-danesc87@gmail.com


Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 8822702f

sound/pci/hda/patch_realtek.c

+62 −0
Original line number Diff line number Diff line
@@ -6254,6 +6254,35 @@ static void alc294_fixup_gx502_hp(struct hda_codec *codec, 
	}

}



static void alc294_gu502_toggle_output(struct hda_codec *codec,

				       struct hda_jack_callback *cb)

{

	/* Windows sets 0x10 to 0x8420 for Node 0x20 which is

	 * responsible from changes between speakers and headphones

	 */

	if (snd_hda_jack_detect_state(codec, 0x21) == HDA_JACK_PRESENT)

		alc_write_coef_idx(codec, 0x10, 0x8420);

	else

		alc_write_coef_idx(codec, 0x10, 0x0a20);

}



static void alc294_fixup_gu502_hp(struct hda_codec *codec,

				  const struct hda_fixup *fix, int action)

{

	if (!is_jack_detectable(codec, 0x21))

		return;



	switch (action) {

	case HDA_FIXUP_ACT_PRE_PROBE:

		snd_hda_jack_detect_enable_callback(codec, 0x21,

				alc294_gu502_toggle_output);

		break;

	case HDA_FIXUP_ACT_INIT:

		alc294_gu502_toggle_output(codec, NULL);

		break;

	}

}



static void  alc285_fixup_hp_gpio_amp_init(struct hda_codec *codec,

			      const struct hda_fixup *fix, int action)

{
@@ -6471,6 +6500,9 @@ enum { 
	ALC294_FIXUP_ASUS_GX502_HP,

	ALC294_FIXUP_ASUS_GX502_PINS,

	ALC294_FIXUP_ASUS_GX502_VERBS,

	ALC294_FIXUP_ASUS_GU502_HP,

	ALC294_FIXUP_ASUS_GU502_PINS,

	ALC294_FIXUP_ASUS_GU502_VERBS,

	ALC285_FIXUP_HP_GPIO_LED,

	ALC285_FIXUP_HP_MUTE_LED,

	ALC236_FIXUP_HP_GPIO_LED,
@@ -7712,6 +7744,35 @@ static const struct hda_fixup alc269_fixups[] = { 
		.type = HDA_FIXUP_FUNC,

		.v.func = alc294_fixup_gx502_hp,

	},

	[ALC294_FIXUP_ASUS_GU502_PINS] = {

		.type = HDA_FIXUP_PINS,

		.v.pins = (const struct hda_pintbl[]) {

			{ 0x19, 0x01a11050 }, /* rear HP mic */

			{ 0x1a, 0x01a11830 }, /* rear external mic */

			{ 0x21, 0x012110f0 }, /* rear HP out */

			{ }

		},

		.chained = true,

		.chain_id = ALC294_FIXUP_ASUS_GU502_VERBS

	},

	[ALC294_FIXUP_ASUS_GU502_VERBS] = {

		.type = HDA_FIXUP_VERBS,

		.v.verbs = (const struct hda_verb[]) {

			/* set 0x15 to HP-OUT ctrl */

			{ 0x15, AC_VERB_SET_PIN_WIDGET_CONTROL, 0xc0 },

			/* unmute the 0x15 amp */

			{ 0x15, AC_VERB_SET_AMP_GAIN_MUTE, 0xb000 },

			/* set 0x1b to HP-OUT */

			{ 0x1b, AC_VERB_SET_PIN_WIDGET_CONTROL, 0x24 },

			{ }

		},

		.chained = true,

		.chain_id = ALC294_FIXUP_ASUS_GU502_HP

	},

	[ALC294_FIXUP_ASUS_GU502_HP] = {

		.type = HDA_FIXUP_FUNC,

		.v.func = alc294_fixup_gu502_hp,

	},

	[ALC294_FIXUP_ASUS_COEF_1B] = {

		.type = HDA_FIXUP_VERBS,

		.v.verbs = (const struct hda_verb[]) {
@@ -8256,6 +8317,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { 
	SND_PCI_QUIRK(0x1043, 0x1ccd, "ASUS X555UB", ALC256_FIXUP_ASUS_MIC),

	SND_PCI_QUIRK(0x1043, 0x1d4e, "ASUS TM420", ALC256_FIXUP_ASUS_HPE),

	SND_PCI_QUIRK(0x1043, 0x1e11, "ASUS Zephyrus G15", ALC289_FIXUP_ASUS_GA502),

	SND_PCI_QUIRK(0x1043, 0x1e51, "ASUS Zephyrus M15", ALC294_FIXUP_ASUS_GU502_PINS),

	SND_PCI_QUIRK(0x1043, 0x1e8e, "ASUS Zephyrus G15", ALC289_FIXUP_ASUS_GA401),

	SND_PCI_QUIRK(0x1043, 0x1f11, "ASUS Zephyrus G14", ALC289_FIXUP_ASUS_GA401),

	SND_PCI_QUIRK(0x1043, 0x3030, "ASUS ZN270IE", ALC256_FIXUP_ASUS_AIO_GPIO2),