Commit 6447c962 authored by Kailang Yang's avatar Kailang Yang Committed by Takashi Iwai

ALSA: hda/realtek - Support low power consumption for ALC256

Remove hp_pin = 0 return. Add the default pin 0x21 as headphone.
Supported low power consumption, it must do depop procedure when
headset jack was plugged or unplugged.
So, alc256_init() and alc256_shutup() must run delay when headset jack
was plugged or unplugged.
If depop procedure not run with delay, it will have a chance to let
power consumption raise high.

[ NOTE: this is a quite quite similar change for ALC295 by commit
  d3ba58bb ("ALSA: hda/realtek - Support low power consumption for
  ALC295"), but applied to ALC256 codec instead -- tiwai ]

Fixes: 8983eb60 ("ALSA: hda/realtek - Move to ACT_INIT state")
Signed-off-by: default avatarKailang Yang <kailang@realtek.com>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 5095c418
...@@ -3197,7 +3197,7 @@ static void alc256_init(struct hda_codec *codec) ...@@ -3197,7 +3197,7 @@ static void alc256_init(struct hda_codec *codec)
bool hp_pin_sense; bool hp_pin_sense;
if (!hp_pin) if (!hp_pin)
return; hp_pin = 0x21;
msleep(30); msleep(30);
...@@ -3207,17 +3207,25 @@ static void alc256_init(struct hda_codec *codec) ...@@ -3207,17 +3207,25 @@ static void alc256_init(struct hda_codec *codec)
msleep(2); msleep(2);
alc_update_coefex_idx(codec, 0x57, 0x04, 0x0007, 0x1); /* Low power */ alc_update_coefex_idx(codec, 0x57, 0x04, 0x0007, 0x1); /* Low power */
if (spec->ultra_low_power) {
alc_update_coef_idx(codec, 0x03, 1<<1, 1<<1);
alc_update_coef_idx(codec, 0x08, 3<<2, 3<<2);
alc_update_coef_idx(codec, 0x08, 7<<4, 0);
alc_update_coef_idx(codec, 0x3b, 1<<15, 0);
alc_update_coef_idx(codec, 0x0e, 7<<6, 7<<6);
msleep(30);
}
snd_hda_codec_write(codec, hp_pin, 0, snd_hda_codec_write(codec, hp_pin, 0,
AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_MUTE); AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_MUTE);
if (hp_pin_sense) if (hp_pin_sense || spec->ultra_low_power)
msleep(85); msleep(85);
snd_hda_codec_write(codec, hp_pin, 0, snd_hda_codec_write(codec, hp_pin, 0,
AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_OUT); AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_OUT);
if (hp_pin_sense) if (hp_pin_sense || spec->ultra_low_power)
msleep(100); msleep(100);
alc_update_coef_idx(codec, 0x46, 3 << 12, 0); alc_update_coef_idx(codec, 0x46, 3 << 12, 0);
...@@ -3232,10 +3240,8 @@ static void alc256_shutup(struct hda_codec *codec) ...@@ -3232,10 +3240,8 @@ static void alc256_shutup(struct hda_codec *codec)
hda_nid_t hp_pin = alc_get_hp_pin(spec); hda_nid_t hp_pin = alc_get_hp_pin(spec);
bool hp_pin_sense; bool hp_pin_sense;
if (!hp_pin) { if (!hp_pin)
alc269_shutup(codec); hp_pin = 0x21;
return;
}
hp_pin_sense = snd_hda_jack_detect(codec, hp_pin); hp_pin_sense = snd_hda_jack_detect(codec, hp_pin);
...@@ -3245,7 +3251,7 @@ static void alc256_shutup(struct hda_codec *codec) ...@@ -3245,7 +3251,7 @@ static void alc256_shutup(struct hda_codec *codec)
snd_hda_codec_write(codec, hp_pin, 0, snd_hda_codec_write(codec, hp_pin, 0,
AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_MUTE); AC_VERB_SET_AMP_GAIN_MUTE, AMP_OUT_MUTE);
if (hp_pin_sense) if (hp_pin_sense || spec->ultra_low_power)
msleep(85); msleep(85);
/* 3k pull low control for Headset jack. */ /* 3k pull low control for Headset jack. */
...@@ -3256,11 +3262,20 @@ static void alc256_shutup(struct hda_codec *codec) ...@@ -3256,11 +3262,20 @@ static void alc256_shutup(struct hda_codec *codec)
snd_hda_codec_write(codec, hp_pin, 0, snd_hda_codec_write(codec, hp_pin, 0,
AC_VERB_SET_PIN_WIDGET_CONTROL, 0x0); AC_VERB_SET_PIN_WIDGET_CONTROL, 0x0);
if (hp_pin_sense) if (hp_pin_sense || spec->ultra_low_power)
msleep(100); msleep(100);
alc_auto_setup_eapd(codec, false); alc_auto_setup_eapd(codec, false);
alc_shutup_pins(codec); alc_shutup_pins(codec);
if (spec->ultra_low_power) {
msleep(50);
alc_update_coef_idx(codec, 0x03, 1<<1, 0);
alc_update_coef_idx(codec, 0x08, 7<<4, 7<<4);
alc_update_coef_idx(codec, 0x08, 3<<2, 0);
alc_update_coef_idx(codec, 0x3b, 1<<15, 1<<15);
alc_update_coef_idx(codec, 0x0e, 7<<6, 0);
msleep(30);
}
} }
static void alc225_init(struct hda_codec *codec) static void alc225_init(struct hda_codec *codec)
......
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