Commit c84bfedc authored by Takashi Iwai's avatar Takashi Iwai

ALSA: hda/realtek: Fix bass speaker DAC assignment on Asus Zephyrus G14

ASUS Zephyrus G14 has two speaker pins, and the auto-parser tries to
assign an individual DAC to each pin as much as possible.
Unfortunately the third DAC has no volume control unlike the two DACs,
and this resulted in the inconsistent speaker volumes.

As a workaround, wire both speaker pins to the same DAC by modifying
the existing quirk (ALC289_FIXUP_ASUS_GA401) applied to this device.
Since this quirk entry is chained by another, we need to avoid
applying the DAC assignment change for it.  Luckily, there is another
quirk entry (ALC289_FIXUP_ASUS_GA502) doing the very same thing, so we
can chain to the GA502 quirk instead.

Note that this patch uses a new flag of the generic parser,
obey_preferred_dacs, for enforcing the DACs.

BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=210359
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/20201127141104.11041-2-tiwai@suse.deSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 242d990c
...@@ -6027,6 +6027,21 @@ static void alc274_fixup_bind_dacs(struct hda_codec *codec, ...@@ -6027,6 +6027,21 @@ static void alc274_fixup_bind_dacs(struct hda_codec *codec,
codec->power_save_node = 0; codec->power_save_node = 0;
} }
/* avoid DAC 0x06 for bass speaker 0x17; it has no volume control */
static void alc289_fixup_asus_ga401(struct hda_codec *codec,
const struct hda_fixup *fix, int action)
{
static const hda_nid_t preferred_pairs[] = {
0x14, 0x02, 0x17, 0x02, 0x21, 0x03, 0
};
struct alc_spec *spec = codec->spec;
if (action == HDA_FIXUP_ACT_PRE_PROBE) {
spec->gen.preferred_dacs = preferred_pairs;
spec->gen.obey_preferred_dacs = 1;
}
}
/* The DAC of NID 0x3 will introduce click/pop noise on headphones, so invalidate it */ /* The DAC of NID 0x3 will introduce click/pop noise on headphones, so invalidate it */
static void alc285_fixup_invalidate_dacs(struct hda_codec *codec, static void alc285_fixup_invalidate_dacs(struct hda_codec *codec,
const struct hda_fixup *fix, int action) const struct hda_fixup *fix, int action)
...@@ -7600,11 +7615,10 @@ static const struct hda_fixup alc269_fixups[] = { ...@@ -7600,11 +7615,10 @@ static const struct hda_fixup alc269_fixups[] = {
.chain_id = ALC269_FIXUP_HEADSET_MIC .chain_id = ALC269_FIXUP_HEADSET_MIC
}, },
[ALC289_FIXUP_ASUS_GA401] = { [ALC289_FIXUP_ASUS_GA401] = {
.type = HDA_FIXUP_PINS, .type = HDA_FIXUP_FUNC,
.v.pins = (const struct hda_pintbl[]) { .v.func = alc289_fixup_asus_ga401,
{ 0x19, 0x03a11020 }, /* headset mic with jack detect */ .chained = true,
{ } .chain_id = ALC289_FIXUP_ASUS_GA502,
},
}, },
[ALC289_FIXUP_ASUS_GA502] = { [ALC289_FIXUP_ASUS_GA502] = {
.type = HDA_FIXUP_PINS, .type = HDA_FIXUP_PINS,
...@@ -7728,7 +7742,7 @@ static const struct hda_fixup alc269_fixups[] = { ...@@ -7728,7 +7742,7 @@ static const struct hda_fixup alc269_fixups[] = {
{ } { }
}, },
.chained = true, .chained = true,
.chain_id = ALC289_FIXUP_ASUS_GA401 .chain_id = ALC289_FIXUP_ASUS_GA502
}, },
[ALC274_FIXUP_HP_MIC] = { [ALC274_FIXUP_HP_MIC] = {
.type = HDA_FIXUP_VERBS, .type = HDA_FIXUP_VERBS,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment