From 119b75c150773425a89033215eab4d15d4198f8b Mon Sep 17 00:00:00 2001

From: Hui Wang <hui.wang@canonical.com>

Date: Sat, 22 May 2021 11:47:41 +0800

Subject: [PATCH] ALSA: hda/realtek: Headphone volume is controlled by Front mixer

Git-commit: 119b75c150773425a89033215eab4d15d4198f8b

Patch-mainline: v5.13-rc4

References: git-fixes

On some ASUS and MSI machines, the audio codec is alc1220 and the

Headphone is connected to audio mixer 0xf and DAC 0x5, in theory

the Headphone volume is controlled by DAC 0x5 (Heapdhone Playback

Volume), but somehow it is controlled by DAC 0x2 (Front Playback

Volume), maybe this is a defect on the codec alc1220.

Because of this issue, the PA couldn't switch the headphone and

Lineout correctly, If we apply the quirk CLEVO_P950 to those machines,

the Lineout and Headphone will share the audio mixer 0xc and DAC 0x2,

and generate Headphone+LO mixer, then PA could handle them when

switching between them.

Buglink: https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/1206

Cc: <stable@vger.kernel.org>

Signed-off-by: Hui Wang <hui.wang@canonical.com>

Link: https://lore.kernel.org/r/20210522034741.13415-1-hui.wang@canonical.com

Signed-off-by: Takashi Iwai <tiwai@suse.de>

---

 sound/pci/hda/patch_realtek.c | 23 +++++++++++++++++++++++

 1 file changed, 23 insertions(+)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c

index 552e2cb73291..ffaeb8d3c316 100644

--- a/sound/pci/hda/patch_realtek.c

+++ b/sound/pci/hda/patch_realtek.c

@@ -2603,6 +2603,28 @@ static const struct hda_model_fixup alc882_fixup_models[] = {

 	{}

 };

+static const struct snd_hda_pin_quirk alc882_pin_fixup_tbl[] = {

+	SND_HDA_PIN_QUIRK(0x10ec1220, 0x1043, "ASUS", ALC1220_FIXUP_CLEVO_P950,

+		{0x14, 0x01014010},

+		{0x15, 0x01011012},

+		{0x16, 0x01016011},

+		{0x18, 0x01a19040},

+		{0x19, 0x02a19050},

+		{0x1a, 0x0181304f},

+		{0x1b, 0x0221401f},

+		{0x1e, 0x01456130}),

+	SND_HDA_PIN_QUIRK(0x10ec1220, 0x1462, "MS-7C35", ALC1220_FIXUP_CLEVO_P950,

+		{0x14, 0x01015010},

+		{0x15, 0x01011012},

+		{0x16, 0x01011011},

+		{0x18, 0x01a11040},

+		{0x19, 0x02a19050},

+		{0x1a, 0x0181104f},

+		{0x1b, 0x0221401f},

+		{0x1e, 0x01451130}),

+	{}

+};

+

 /*

  * BIOS auto configuration

  */

@@ -2644,6 +2666,7 @@ static int patch_alc882(struct hda_codec *codec)

 	snd_hda_pick_fixup(codec, alc882_fixup_models, alc882_fixup_tbl,

 		       alc882_fixups);

+	snd_hda_pick_pin_fixup(codec, alc882_pin_fixup_tbl, alc882_fixups, true);

 	snd_hda_apply_fixup(codec, HDA_FIXUP_ACT_PRE_PROBE);

 	alc_auto_parse_customize_define(codec);

-- 

2.26.2