hda: Add 5.1 support for second headphone jack
PATCH 1/1 hda: Add 5.1 support for second headphone jack
Several 92hd7xxx and STAC9228 laptops have multiple headphone jacks, the second headphone jack should be used for the 5.1 surround sound. Add support for 'Headphone as Line Out' switch, which allows it be used in 5.1 surround sound.
---
pci/hda/patch_sigmatel.c +58 -1
@@ -179,6 +179,7 @@ struct sigmatel_spec {/* i/o switches */ unsigned int io_switch[2 ];unsigned int clfe_swap; +unsigned int hp_switch;unsigned int aloopback;struct hda_pcm pcm_rec[2 ];/* PCM information */
@@ -1251,7 +1252,7 @@ static unsigned int ref92hd73xx_pin_confstatic unsigned int dell_m6_pin_configs[13 ] = {0x0321101f ,0x4f00000f ,0x4f0000f0 ,0x90170110 , -0x03a11020 ,0x03011050 ,0x4f0000f0 ,0x4f0000f0 , +0x03a11020 ,0x0321101f ,0x4f0000f0 ,0x4f0000f0 ,0x4f0000f0 ,0x90a60160 ,0x4f0000f0 ,0x4f0000f0 ,0x4f0000f0 , };
@@ -2029,6 +2030,33 @@ static void stac92xx_auto_set_pinctl(str AC_VERB_SET_PIN_WIDGET_CONTROL, pin_type); } +#define stac92xx_hp_switch_info snd_ctl_boolean_mono_info + +static int stac92xx_hp_switch_get (struct snd_kcontrol *kcontrol, +struct snd_ctl_elem_value *ucontrol) +{ +struct hda_codec *codec =snd_kcontrol_chip (kcontrol); +struct sigmatel_spec *spec = codec->spec; + + ucontrol->value.integer.value[0 ] = spec->hp_switch; +return 0 ; +} + +static int stac92xx_hp_switch_put (struct snd_kcontrol *kcontrol, +struct snd_ctl_elem_value *ucontrol) +{ +struct hda_codec *codec =snd_kcontrol_chip (kcontrol); +struct sigmatel_spec *spec = codec->spec; + + spec->hp_switch = ucontrol->value.integer.value[0 ]; + +/* check to be sure that the ports are upto date with +* switch changes */ + codec->patch_ops.unsol_event (codec,STAC_HP_EVENT <<26 ); + +return 1 ; +} +#define stac92xx_io_switch_info snd_ctl_boolean_mono_infostatic int stac92xx_io_switch_get (struct snd_kcontrol *kcontrol,struct snd_ctl_elem_value *ucontrol)
@@ -2099,6 +2127,15 @@ static int stac92xx_clfe_switch_put(strureturn 1 ; } + +#define STAC_CODEC_HP_SWITCH (xname) \ + { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, \ + .name = xname, \ + .index = 0, \ + .info = stac92xx_hp_switch_info, \ + .get = stac92xx_hp_switch_get, \ + .put = stac92xx_hp_switch_put, \ + }#define STAC_CODEC_IO_SWITCH (xname, xpval) \ { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, \
@@ -2124,6 +2161,7 @@ enum {STAC_CTL_WIDGET_VOL ,STAC_CTL_WIDGET_MUTE ,STAC_CTL_WIDGET_MONO_MUX , +STAC_CTL_WIDGET_HP_SWITCH ,STAC_CTL_WIDGET_IO_SWITCH , STAC_CTL_WIDGET_CLFE_SWITCH };
@@ -2132,6 +2170,7 @@ static struct snd_kcontrol_new stac92xx_HDA_CODEC_VOLUME (NULL ,0 ,0 ,0 ),HDA_CODEC_MUTE (NULL ,0 ,0 ,0 ),STAC_MONO_MUX , +STAC_CODEC_HP_SWITCH (NULL ),STAC_CODEC_IO_SWITCH (NULL ,0 ), STAC_CODEC_CLFE_SWITCH(NULL, 0), };
@@ -2396,6 +2435,14 @@ static int stac92xx_auto_create_multi_ou } } +if (cfg->hp_outs >1 ) { + err =stac92xx_add_control (spec, +STAC_CTL_WIDGET_HP_SWITCH , +"Headphone as Line Out Switch" ,0 ); +if (err <0 ) +return err; + } +if (spec->line_switch) { nid = cfg->input_pins[AUTO_PIN_LINE ]; pincap = snd_hda_param_read(codec, nid,
@@ -3144,6 +3191,7 @@ static void stac92xx_hp_detect(struct hd {struct sigmatel_spec *spec = codec->spec;struct auto_pin_cfg *cfg = &spec->autocfg; +int nid = cfg->hp_pins[cfg->hp_outs -1 ];int i, presence; presence =0 ;
@@ -3154,11 +3202,15 @@ static void stac92xx_hp_detect(struct hdfor (i =0 ; i < cfg->hp_outs; i++) {if (presence)break ; +if (spec->hp_switch && cfg->hp_pins[i] == nid) +break ; presence =get_hp_pin_presence (codec, cfg->hp_pins[i]); }if (presence) {/* disable lineouts, enable hp */ +if (spec->hp_switch) +stac92xx_reset_pinctl (codec, nid,AC_PINCTL_OUT_EN );for (i =0 ; i < cfg->line_outs; i++)stac92xx_reset_pinctl (codec, cfg->line_out_pins[i],AC_PINCTL_OUT_EN );
@@ -3167,6 +3219,8 @@ static void stac92xx_hp_detect(struct hd AC_PINCTL_OUT_EN); }else {/* enable lineouts, disable hp */ +if (spec->hp_switch) +stac92xx_set_pinctl (codec, nid,AC_PINCTL_OUT_EN );for (i =0 ; i < cfg->line_outs; i++)stac92xx_set_pinctl (codec, cfg->line_out_pins[i],AC_PINCTL_OUT_EN );
@@ -3174,6 +3228,8 @@ static void stac92xx_hp_detect(struct hdstac92xx_set_pinctl (codec, cfg->speaker_pins[i],AC_PINCTL_OUT_EN ); } +if (!spec->hp_switch && cfg->hp_outs >1 && presence) +stac92xx_set_pinctl (codec, nid,AC_PINCTL_OUT_EN ); }static void stac92xx_pin_sense (struct hda_codec *codec,int idx)
@@ -3427,6 +3483,7 @@ again:switch (spec->multiout.num_dacs) {case 0x3 :/* 6 Channel */ + spec->multiout.hp_nid =0x17 ; spec->mixer = stac92hd73xx_6ch_mixer; spec->init = stac92hd73xx_6ch_core_init;break ;
Matthew,
I just tested this patch. It appears the second headphone jack works, but that switch's functionality doesn't act like "Headphone as Line Out". Instead, it acts as "Turn on speakers when headphones are plugged into the second jack".
Regards,
Quoted 151 lines · Matthew Ranostay
Several 92hd7xxx and STAC9228 laptops have multiple headphone jacks, the second headphone jack should be used for the 5.1 surround sound. Add support for 'Headphone as Line Out' switch, which allows it be used in 5.1 surround sound.
Signed-off-by Matthew Ranostay <[email protected]>---
diff -r547e051a88b5pci/hda/patch_sigmatel.cpci/hda/patch_sigmatel.c +57
-1@@ -179,6 +179,7 @@ struct sigmatel_spec {/* i/o switches */ unsigned int io_switch[2 ];unsigned int clfe_swap; +unsigned int hp_switch;unsigned int aloopback;struct hda_pcm pcm_rec[2 ];/* PCM information */ @@ -1251,7 +1252,7 @@ static unsigned int ref92hd73xx_pin_confstatic unsigned int dell_m6_pin_configs[13 ] = {0x0321101f ,0x4f00000f ,0x4f0000f0 ,0x90170110 , -0x03a11020 ,0x03011050 ,0x4f0000f0 ,0x4f0000f0 , +0x03a11020 ,0x0321101f ,0x4f0000f0 ,0x4f0000f0 ,0x4f0000f0 ,0x90a60160 ,0x4f0000f0 ,0x4f0000f0 ,0x4f0000f0 , };@@ -2029,6 +2030,33 @@ static void stac92xx_auto_set_pinctl(str AC_VERB_SET_PIN_WIDGET_CONTROL, pin_type); } +#define stac92xx_hp_switch_info snd_ctl_boolean_mono_info + +static int stac92xx_hp_switch_get (struct snd_kcontrol *kcontrol, +struct snd_ctl_elem_value *ucontrol) +{ +struct hda_codec *codec =snd_kcontrol_chip (kcontrol); +struct sigmatel_spec *spec = codec->spec; + + ucontrol->value.integer.value[0 ] = spec->hp_switch; +return 0 ; +} + +static int stac92xx_hp_switch_put (struct snd_kcontrol *kcontrol, +struct snd_ctl_elem_value *ucontrol) +{ +struct hda_codec *codec =snd_kcontrol_chip (kcontrol); +struct sigmatel_spec *spec = codec->spec; + + spec->hp_switch = ucontrol->value.integer.value[0 ]; + +/* check to be sure that the ports are upto date with +* switch changes */ + codec->patch_ops.unsol_event (codec,STAC_HP_EVENT <<26 ); + +return 1 ; +} +#define stac92xx_io_switch_info snd_ctl_boolean_mono_infostatic int stac92xx_io_switch_get (struct snd_kcontrol *kcontrol,struct snd_ctl_elem_value *ucontrol)@@ -2099,6 +2127,15 @@ static int stac92xx_clfe_switch_put(strureturn 1 ; } + +#define STAC_CODEC_HP_SWITCH (xname) \ + { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, \ + .name = xname, \ + .index = 0, \ + .info = stac92xx_hp_switch_info, \ + .get = stac92xx_hp_switch_get, \ + .put = stac92xx_hp_switch_put, \ + }#define STAC_CODEC_IO_SWITCH (xname, xpval) \ { .iface = SNDRV_CTL_ELEM_IFACE_MIXER, \@@ -2124,6 +2161,7 @@ enum {STAC_CTL_WIDGET_VOL ,STAC_CTL_WIDGET_MUTE ,STAC_CTL_WIDGET_MONO_MUX , +STAC_CTL_WIDGET_HP_SWITCH ,STAC_CTL_WIDGET_IO_SWITCH , STAC_CTL_WIDGET_CLFE_SWITCH };@@ -2132,6 +2170,7 @@ static struct snd_kcontrol_new stac92xx_HDA_CODEC_VOLUME (NULL ,0 ,0 ,0 ),HDA_CODEC_MUTE (NULL ,0 ,0 ,0 ),STAC_MONO_MUX , +STAC_CODEC_HP_SWITCH (NULL ),STAC_CODEC_IO_SWITCH (NULL ,0 ), STAC_CODEC_CLFE_SWITCH(NULL, 0), };@@ -2396,6 +2435,14 @@ static int stac92xx_auto_create_multi_ou } } +if (cfg->hp_outs >1 ) { + err =stac92xx_add_control (spec, +STAC_CTL_WIDGET_HP_SWITCH , +"Headphone as Line Out Switch" ,0 ); +if (err <0 ) +return err; + } +if (spec->line_switch) { nid = cfg->input_pins[AUTO_PIN_LINE ]; pincap = snd_hda_param_read(codec, nid,@@ -3144,6 +3191,7 @@ static void stac92xx_hp_detect(struct hd {struct sigmatel_spec *spec = codec->spec;struct auto_pin_cfg *cfg = &spec->autocfg; +int nid = cfg->hp_pins[cfg->hp_outs -1 ];int i, presence; presence =0 ;@@ -3154,11 +3202,15 @@ static void stac92xx_hp_detect(struct hdfor (i =0 ; i < cfg->hp_outs; i++) {if (presence)break ; +if (spec->hp_switch && cfg->hp_pins[i] == nid) +break ; presence =get_hp_pin_presence (codec, cfg->hp_pins[i]); }if (presence) {/* disable lineouts, enable hp */ +if (spec->hp_switch) +stac92xx_reset_pinctl (codec, nid,AC_PINCTL_OUT_EN );for (i =0 ; i < cfg->line_outs; i++)stac92xx_reset_pinctl (codec, cfg->line_out_pins[i],AC_PINCTL_OUT_EN );@@ -3167,6 +3219,8 @@ static void stac92xx_hp_detect(struct hd AC_PINCTL_OUT_EN); }else {/* enable lineouts, disable hp */ +if (spec->hp_switch) +stac92xx_set_pinctl (codec, nid,AC_PINCTL_OUT_EN );for (i =0 ; i < cfg->line_outs; i++)stac92xx_set_pinctl (codec, cfg->line_out_pins[i],AC_PINCTL_OUT_EN );@@ -3174,6 +3228,8 @@ static void stac92xx_hp_detect(struct hdstac92xx_set_pinctl (codec, cfg->speaker_pins[i],AC_PINCTL_OUT_EN ); } +if (!spec->hp_switch && cfg->hp_outs >1 && presence) +stac92xx_set_pinctl (codec, nid,AC_PINCTL_OUT_EN ); }static void stac92xx_pin_sense (struct hda_codec *codec,int idx)
pci/hda/patch_sigmatel.c again: · line 3489
Quoted 7 lines
pci/hda/patch_sigmatel.c +1
-0@@ -3427,6 +3483,7 @@ again:switch (spec->multiout.num_dacs) {case 0x3 :/* 6 Channel */ + spec->multiout.hp_nid =0x17 ; spec->mixer = stac92hd73xx_6ch_mixer; spec->init = stac92hd73xx_6ch_core_init;break ;
---
Mario Limonciello
Dell | Linux Engineering
Desk : (512) 723-0582
[email protected]
_______________________________________________
Alsa-devel mailing list
[email protected]
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel