Skip to content

Commit 1f55e36

Browse files
committed
ALSA: hda/conexant: Use the new codec SSID matching
Now we can perform the codec ID matching primarily, and reduce the conditional application of the quirk for conflicting PCI SSID between System76 and Tuxedo devices. Signed-off-by: Takashi Iwai <[email protected]> Link: https://patch.msgid.link/[email protected]
1 parent 5b1913a commit 1f55e36

File tree

1 file changed

+7
-21
lines changed

1 file changed

+7
-21
lines changed

sound/pci/hda/patch_conexant.c

Lines changed: 7 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -816,23 +816,6 @@ static const struct hda_pintbl cxt_pincfg_sws_js201d[] = {
816816
{}
817817
};
818818

819-
/* pincfg quirk for Tuxedo Sirius;
820-
* unfortunately the (PCI) SSID conflicts with System76 Pangolin pang14,
821-
* which has incompatible pin setup, so we check the codec SSID (luckily
822-
* different one!) and conditionally apply the quirk here
823-
*/
824-
static void cxt_fixup_sirius_top_speaker(struct hda_codec *codec,
825-
const struct hda_fixup *fix,
826-
int action)
827-
{
828-
/* ignore for incorrectly picked-up pang14 */
829-
if (codec->core.subsystem_id == 0x278212b3)
830-
return;
831-
/* set up the top speaker pin */
832-
if (action == HDA_FIXUP_ACT_PRE_PROBE)
833-
snd_hda_codec_set_pincfg(codec, 0x1d, 0x82170111);
834-
}
835-
836819
static const struct hda_fixup cxt_fixups[] = {
837820
[CXT_PINCFG_LENOVO_X200] = {
838821
.type = HDA_FIXUP_PINS,
@@ -993,8 +976,11 @@ static const struct hda_fixup cxt_fixups[] = {
993976
.v.pins = cxt_pincfg_sws_js201d,
994977
},
995978
[CXT_PINCFG_TOP_SPEAKER] = {
996-
.type = HDA_FIXUP_FUNC,
997-
.v.func = cxt_fixup_sirius_top_speaker,
979+
.type = HDA_FIXUP_PINS,
980+
.v.pins = (const struct hda_pintbl[]) {
981+
{ 0x1d, 0x82170111 },
982+
{ }
983+
},
998984
},
999985
};
1000986

@@ -1092,8 +1078,8 @@ static const struct hda_quirk cxt5066_fixups[] = {
10921078
SND_PCI_QUIRK_VENDOR(0x17aa, "Thinkpad", CXT_FIXUP_THINKPAD_ACPI),
10931079
SND_PCI_QUIRK(0x1c06, 0x2011, "Lemote A1004", CXT_PINCFG_LEMOTE_A1004),
10941080
SND_PCI_QUIRK(0x1c06, 0x2012, "Lemote A1205", CXT_PINCFG_LEMOTE_A1205),
1095-
SND_PCI_QUIRK(0x2782, 0x12c3, "Sirius Gen1", CXT_PINCFG_TOP_SPEAKER),
1096-
SND_PCI_QUIRK(0x2782, 0x12c5, "Sirius Gen2", CXT_PINCFG_TOP_SPEAKER),
1081+
HDA_CODEC_QUIRK(0x2782, 0x12c3, "Sirius Gen1", CXT_PINCFG_TOP_SPEAKER),
1082+
HDA_CODEC_QUIRK(0x2782, 0x12c5, "Sirius Gen2", CXT_PINCFG_TOP_SPEAKER),
10971083
{}
10981084
};
10991085

0 commit comments

Comments
 (0)