[alsa-devel] hp_pin was NULL value
Kailang
kailang at realtek.com
Wed Jan 9 10:31:33 CET 2019
Hi Takashi,
Could I move the alc294_hp_init(codec) to below line.
Because hp_pin = spec->gen.autocfg.hp_pins[0] was null value when alc294_hp_init(codec) at original line.
Or move alc269_parse_auto_config() upward.
BR,
Kailang
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -7514,7 +7514,6 @@ static int patch_alc269(struct hda_codec *codec)
spec->codec_variant = ALC269_TYPE_ALC294;
spec->gen.mixer_nid = 0; /* ALC2x4 does not have any loopback mixer path */
alc_update_coef_idx(codec, 0x6b, 0x0018, (1<<4) | (1<<3)); /* UAJ MIC Vref control by verb */
- alc294_hp_init(codec);
break;
case 0x10ec0300:
spec->codec_variant = ALC269_TYPE_ALC300;
@@ -7526,7 +7525,6 @@ static int patch_alc269(struct hda_codec *codec)
spec->codec_variant = ALC269_TYPE_ALC700;
spec->gen.mixer_nid = 0; /* ALC700 does not have any loopback mixer path */
alc_update_coef_idx(codec, 0x4a, 1 << 15, 0); /* Combo jack auto trigger control */
- alc294_hp_init(codec);
break;
}
@@ -7553,6 +7551,16 @@ static int patch_alc269(struct hda_codec *codec)
if (err < 0)
goto error;
+ switch (codec->core.vendor_id) {
+ case 0x10ec0234:
+ case 0x10ec0274:
+ case 0x10ec0294:
+ case 0x10ec0700:
+ case 0x10ec0701:
+ case 0x10ec0703:
+ alc294_hp_init(codec);
+ break;
+ }
if (!spec->gen.no_analog && spec->gen.beep_nid && spec->gen.mixer_nid) {
err = set_beep_amp(spec, spec->gen.mixer_nid, 0x04, HDA_INPUT);
if (err < 0)
More information about the Alsa-devel
mailing list