Commit aeac1a0d authored by Kailang Yang's avatar Kailang Yang Committed by Takashi Iwai

ALSA: hda/realtek - Check headset type by unplug and resume

When system enable HDA power save mode.
This issue will happen on new platform which DMIC connect to PCH.
In Dell headset mode, it will recheck during runtime resume when
headset was plugged.
This patch will move check headset type on unplug and system resume.

[ A few minor code cleanups by tiwai ]
Signed-off-by: default avatarKailang Yang <kailang@realtek.com>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent a188339c
...@@ -834,6 +834,8 @@ static void alc_pre_init(struct hda_codec *codec) ...@@ -834,6 +834,8 @@ static void alc_pre_init(struct hda_codec *codec)
alc_fill_eapd_coef(codec); alc_fill_eapd_coef(codec);
} }
#define is_s3_resume(codec) \
((codec)->core.dev.power.power_state.event == PM_EVENT_RESUME)
#define is_s4_resume(codec) \ #define is_s4_resume(codec) \
((codec)->core.dev.power.power_state.event == PM_EVENT_RESTORE) ((codec)->core.dev.power.power_state.event == PM_EVENT_RESTORE)
...@@ -4901,6 +4903,8 @@ static void alc_update_headset_mode(struct hda_codec *codec) ...@@ -4901,6 +4903,8 @@ static void alc_update_headset_mode(struct hda_codec *codec)
switch (new_headset_mode) { switch (new_headset_mode) {
case ALC_HEADSET_MODE_UNPLUGGED: case ALC_HEADSET_MODE_UNPLUGGED:
alc_headset_mode_unplugged(codec); alc_headset_mode_unplugged(codec);
spec->current_headset_mode = ALC_HEADSET_MODE_UNKNOWN;
spec->current_headset_type = ALC_HEADSET_TYPE_UNKNOWN;
spec->gen.hp_jack_present = false; spec->gen.hp_jack_present = false;
break; break;
case ALC_HEADSET_MODE_HEADSET: case ALC_HEADSET_MODE_HEADSET:
...@@ -4943,8 +4947,6 @@ static void alc_update_headset_mode_hook(struct hda_codec *codec, ...@@ -4943,8 +4947,6 @@ static void alc_update_headset_mode_hook(struct hda_codec *codec,
static void alc_update_headset_jack_cb(struct hda_codec *codec, static void alc_update_headset_jack_cb(struct hda_codec *codec,
struct hda_jack_callback *jack) struct hda_jack_callback *jack)
{ {
struct alc_spec *spec = codec->spec;
spec->current_headset_type = ALC_HEADSET_TYPE_UNKNOWN;
snd_hda_gen_hp_automute(codec, jack); snd_hda_gen_hp_automute(codec, jack);
} }
...@@ -4981,7 +4983,10 @@ static void alc_fixup_headset_mode(struct hda_codec *codec, ...@@ -4981,7 +4983,10 @@ static void alc_fixup_headset_mode(struct hda_codec *codec,
alc_probe_headset_mode(codec); alc_probe_headset_mode(codec);
break; break;
case HDA_FIXUP_ACT_INIT: case HDA_FIXUP_ACT_INIT:
spec->current_headset_mode = 0; if (is_s3_resume(codec) || is_s4_resume(codec)) {
spec->current_headset_mode = ALC_HEADSET_MODE_UNKNOWN;
spec->current_headset_type = ALC_HEADSET_TYPE_UNKNOWN;
}
alc_update_headset_mode(codec); alc_update_headset_mode(codec);
break; break;
} }
......
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