Re: [bisected] snd_hda_intel, 2.6.37: automuting on headphone connectbroken; headphone connecter no longer shown in userspace

[Nirbheek Chauhan]

On Mon, Jan 10, 2011 at 8:19 PM, Takashi Iwai <tiwai@xxxxxxx> wrote:
> At Mon, 10 Jan 2011 14:54:25 +0100,
> Takashi Iwai wrote:
>>
>> At Wed, 5 Jan 2011 13:07:41 +0530,
>> Nirbheek Chauhan wrote:
>> >
>> > Hello,
>> >
>> > Upon upgrading to 2.6.37,
>> > auto-muting-laptop-speakers-on-headphone-connect stopped working, and
>> > the headphone connector completely disappeared from alsamixer as well
>> > as the pulseaudio volume control panel. The headphone jack was still
>> > transmitting sound, though.
>> >
>> > I bisected the problem, with 'good' being v2.6.36, and found the
>> > commit below as the first bad commit. Reverting it on v2.6.37 fixes
>> > the problem. I also tried the latest alsa-kernel master git tree,
>> > which had the same problem.
>> >
>> > alsa-info when 'good':
>> >
>> > http://www.alsa-project.org/db/?f=a0b5c95f764a0b01746be46f989685996e1f9210
>> >
>> > alsa-info when 'bad':
>> >
>> > http://www.alsa-project.org/db/?f=30f30ca5461f03b98ab6c7c611cf24ea4526f537
>> >
>> > Note: both were taken without X running, before pulseaudio came up.
>> >
>> > ================
>> > 03642c9a444079aa13f0864383a8f9ca04bfd198 is the first bad commit
>> > commit 03642c9a444079aa13f0864383a8f9ca04bfd198
>> > Author: Takashi Iwai <tiwai@xxxxxxx>
>> > Date: Â Wed Sep 8 15:28:19 2010 +0200
>> >
>> > Â Â ALSA: hda - Clear left-over hp_pins in snd_hda_parse_pin_def_config()
>> >
>> > Â Â In snd_hda_parse_def_config(), some unused values may remain in hp_pins[]
>> > Â Â array during the headphone-reassignment workaround. ÂThis patch clears
>> > Â Â the unused array members.
>> >
>> > Â Â Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
>> >
>> > :040000 040000 34724336a75f74263423f28d64450d65e5ed948d
>> > 8d136fce8661ad2df35ac3c4a9ca1c712c9833ed M Âsound
>> >
>> > ================
>>
>> It seems that the driver worked casually in the earlier versions :)
>> Does the patch below fix your problem?
>>
>> Note that the "Headphone" control will still not appear even with this
>> patch, but these two outputs are controlled by "Front" and "Surround"
>> controls. ÂThe lack of "Headphone" is no regression but simply it
>> shouldn't have been there.
>>
>> I'll work on 2.6.38 tree for better handling of multi headphone
>> cases.
>
> The patch is below. ÂGive it a try.
>

This patch made the jacks and their alsamixer controls function
exactly as I expected them to!

(I have to admit, at first I applied only this patch without the
previous one, and that made things really weird. I had fully
documented the weirdness before I realised that the first patch might
not have been *just* for stable)

alsamixer layout:
[Master] [Headphone] [Headphone 1] [Speaker] [PCM] [Mic 1] [Mic Boost 1]

alsa-info:
http://www.alsa-project.org/db/?f=34790490c2d70f019b14b22f8e94289425b1bcd6

* `Speaker` is only toggle-mute
* `PCM` has no toggle-mute
* `Master` affects audio in all outputs
* Inserting a jack into either headphone socket mutes the laptop speakers
* `Headphone` and `Headphone 1` control their respective headphones
(maybe they should be named Headphone 1, Headphone 2?)
* `Speaker` mutes the laptop speakers
* `PCM` affects audio in all outputs

All this is without pulseaudio running. Pulseaudio doesn't seem to be
able to handle multiple headphones nicely, and I can't control which
connector an audio stream goes to at all (not a regression, I would
note). But I guess this one is for the PA devs :-)

I'll be using both these patches, and will report any bugs I find.

Thanks a lot!

>
> Takashi
>
> ===
> From bcb2f0f517ebae7350526bbde8912ad187147e2d Mon Sep 17 00:00:00 2001
> From: Takashi Iwai <tiwai@xxxxxxx>
> Date: Mon, 10 Jan 2011 15:45:23 +0100
> Subject: [PATCH] ALSA: hda - Add support for multiple headphone/speaker controls for Realtek
>
> So far, Realtek auto-parser assumed that the multiple pins are only for
> line-outs, and assigned the channel names like Front, Surround, etc for
> the multiple outputs. ÂBut, there are devices that have multiple
> headphones, and these can be better controlled with the corresponding
> control-name like "Headphone" with indicies.
>
> Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
> ---
> Âsound/pci/hda/patch_realtek.c | Â137 ++++++++++++++++++++---------------------
> Â1 files changed, 68 insertions(+), 69 deletions(-)
>
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index b4f7895..0ecd75e 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -5068,6 +5068,25 @@ static int alc880_auto_fill_dac_nids(struct alc_spec *spec,
> Â Â Â Âreturn 0;
> Â}
>
> +static const char *alc_get_line_out_pfx(const struct auto_pin_cfg *cfg,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â bool can_be_master)
> +{
> + Â Â Â if (!cfg->hp_outs && !cfg->speaker_outs && can_be_master)
> + Â Â Â Â Â Â Â return "Master";
> +
> + Â Â Â switch (cfg->line_out_type) {
> + Â Â Â case AUTO_PIN_SPEAKER_OUT:
> + Â Â Â Â Â Â Â return "Speaker";
> + Â Â Â case AUTO_PIN_HP_OUT:
> + Â Â Â Â Â Â Â return "Headphone";
> + Â Â Â default:
> + Â Â Â Â Â Â Â if (cfg->line_outs == 1)
> + Â Â Â Â Â Â Â Â Â Â Â return "PCM";
> + Â Â Â Â Â Â Â break;
> + Â Â Â }
> + Â Â Â return NULL;
> +}
> +
> Â/* add playback controls from the parsed DAC table */
> Âstatic int alc880_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â const struct auto_pin_cfg *cfg)
> @@ -5075,6 +5094,7 @@ static int alc880_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Âstatic const char *chname[4] = {
> Â Â Â Â Â Â Â Â"Front", "Surround", NULL /*CLFE*/, "Side"
> Â Â Â Â};
> + Â Â Â const char *pfx = alc_get_line_out_pfx(cfg, false);
> Â Â Â Âhda_nid_t nid;
> Â Â Â Âint i, err;
>
> @@ -5082,7 +5102,7 @@ static int alc880_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Â Â Â Â Âif (!spec->multiout.dac_nids[i])
> Â Â Â Â Â Â Â Â Â Â Â Âcontinue;
> Â Â Â Â Â Â Â Ânid = alc880_idx_to_mixer(alc880_dac_to_idx(spec->multiout.dac_nids[i]));
> - Â Â Â Â Â Â Â if (i == 2) {
> + Â Â Â Â Â Â Â if (!pfx && i == 2) {
> Â Â Â Â Â Â Â Â Â Â Â Â/* Center/LFE */
> Â Â Â Â Â Â Â Â Â Â Â Âerr = add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"Center",
> @@ -5109,18 +5129,17 @@ static int alc880_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> Â Â Â Â Â Â Â Â} else {
> - Â Â Â Â Â Â Â Â Â Â Â const char *pfx;
> - Â Â Â Â Â Â Â Â Â Â Â if (cfg->line_outs == 1 &&
> - Â Â Â Â Â Â Â Â Â Â Â Â Â cfg->line_out_type == AUTO_PIN_SPEAKER_OUT)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = "Speaker";
> - Â Â Â Â Â Â Â Â Â Â Â else
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = chname[i];
> - Â Â Â Â Â Â Â Â Â Â Â err = add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL, pfx,
> + Â Â Â Â Â Â Â Â Â Â Â const char *name = pfx;
> + Â Â Â Â Â Â Â Â Â Â Â if (!name)
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â name = chname[i];
> + Â Â Â Â Â Â Â Â Â Â Â err = __add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â name, i,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_COMPOSE_AMP_VAL(nid, 3, 0,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_OUTPUT));
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> - Â Â Â Â Â Â Â Â Â Â Â err = add_pb_sw_ctrl(spec, ALC_CTL_BIND_MUTE, pfx,
> + Â Â Â Â Â Â Â Â Â Â Â err = __add_pb_sw_ctrl(spec, ALC_CTL_BIND_MUTE,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âname, i,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_COMPOSE_AMP_VAL(nid, 3, 2,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_INPUT));
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> @@ -12085,13 +12104,8 @@ static int alc262_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Âspec->multiout.dac_nids = spec->private_dac_nids;
> Â Â Â Âspec->multiout.dac_nids[0] = 2;
>
> - Â Â Â if (!cfg->speaker_pins[0] && !cfg->hp_pins[0])
> - Â Â Â Â Â Â Â pfx = "Master";
> - Â Â Â else if (cfg->line_out_type == AUTO_PIN_SPEAKER_OUT)
> - Â Â Â Â Â Â Â pfx = "Speaker";
> - Â Â Â else if (cfg->line_out_type == AUTO_PIN_HP_OUT)
> - Â Â Â Â Â Â Â pfx = "Headphone";
> - Â Â Â else
> + Â Â Â pfx = alc_get_line_out_pfx(cfg, true);
> + Â Â Â if (!pfx)
> Â Â Â Â Â Â Â Âpfx = "Front";
> Â Â Â Âfor (i = 0; i < 2; i++) {
> Â Â Â Â Â Â Â Âerr = alc262_add_out_sw_ctl(spec, cfg->line_out_pins[i], pfx, i);
> @@ -15885,13 +15899,16 @@ static int alc861_auto_fill_dac_nids(struct hda_codec *codec,
> Â Â Â Âreturn 0;
> Â}
>
> -static int alc861_create_out_sw(struct hda_codec *codec, const char *pfx,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â hda_nid_t nid, unsigned int chs)
> +static int __alc861_create_out_sw(struct hda_codec *codec, const char *pfx,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â hda_nid_t nid, int idx, unsigned int chs)
> Â{
> - Â Â Â return add_pb_sw_ctrl(codec->spec, ALC_CTL_WIDGET_MUTE, pfx,
> + Â Â Â return __add_pb_sw_ctrl(codec->spec, ALC_CTL_WIDGET_MUTE, pfx, idx,
> Â Â Â Â Â Â Â Â Â Â Â Â Â HDA_COMPOSE_AMP_VAL(nid, chs, 0, HDA_OUTPUT));
> Â}
>
> +#define alc861_create_out_sw(codec, pfx, nid, chs) \
> + Â Â Â __alc861_create_out_sw(codec, pfx, nid, 0, chs)
> +
> Â/* add playback controls from the parsed DAC table */
> Âstatic int alc861_auto_create_multi_out_ctls(struct hda_codec *codec,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â const struct auto_pin_cfg *cfg)
> @@ -15900,26 +15917,15 @@ static int alc861_auto_create_multi_out_ctls(struct hda_codec *codec,
> Â Â Â Âstatic const char *chname[4] = {
> Â Â Â Â Â Â Â Â"Front", "Surround", NULL /*CLFE*/, "Side"
> Â Â Â Â};
> + Â Â Â const char *pfx = alc_get_line_out_pfx(cfg, true);
> Â Â Â Âhda_nid_t nid;
> Â Â Â Âint i, err;
>
> - Â Â Â if (cfg->line_outs == 1) {
> - Â Â Â Â Â Â Â const char *pfx = NULL;
> - Â Â Â Â Â Â Â if (!cfg->hp_outs)
> - Â Â Â Â Â Â Â Â Â Â Â pfx = "Master";
> - Â Â Â Â Â Â Â else if (cfg->line_out_type == AUTO_PIN_SPEAKER_OUT)
> - Â Â Â Â Â Â Â Â Â Â Â pfx = "Speaker";
> - Â Â Â Â Â Â Â if (pfx) {
> - Â Â Â Â Â Â Â Â Â Â Â nid = spec->multiout.dac_nids[0];
> - Â Â Â Â Â Â Â Â Â Â Â return alc861_create_out_sw(codec, pfx, nid, 3);
> - Â Â Â Â Â Â Â }
> - Â Â Â }
> -
> Â Â Â Âfor (i = 0; i < cfg->line_outs; i++) {
> Â Â Â Â Â Â Â Ânid = spec->multiout.dac_nids[i];
> Â Â Â Â Â Â Â Âif (!nid)
> Â Â Â Â Â Â Â Â Â Â Â Âcontinue;
> - Â Â Â Â Â Â Â if (i == 2) {
> + Â Â Â Â Â Â Â if (!pfx && i == 2) {
> Â Â Â Â Â Â Â Â Â Â Â Â/* Center/LFE */
> Â Â Â Â Â Â Â Â Â Â Â Âerr = alc861_create_out_sw(codec, "Center", nid, 1);
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> @@ -15928,7 +15934,10 @@ static int alc861_auto_create_multi_out_ctls(struct hda_codec *codec,
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> Â Â Â Â Â Â Â Â} else {
> - Â Â Â Â Â Â Â Â Â Â Â err = alc861_create_out_sw(codec, chname[i], nid, 3);
> + Â Â Â Â Â Â Â Â Â Â Â const char *name = pfx;
> + Â Â Â Â Â Â Â Â Â Â Â if (!name)
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â name = chname[i];
> + Â Â Â Â Â Â Â Â Â Â Â err = __alc861_create_out_sw(codec, name, nid, i, 3);
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> Â Â Â Â Â Â Â Â}
> @@ -17033,6 +17042,7 @@ static int alc861vd_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â const struct auto_pin_cfg *cfg)
> Â{
> Â Â Â Âstatic const char *chname[4] = {"Front", "Surround", "CLFE", "Side"};
> + Â Â Â const char *pfx = alc_get_line_out_pfx(cfg, true);
> Â Â Â Âhda_nid_t nid_v, nid_s;
> Â Â Â Âint i, err;
>
> @@ -17046,7 +17056,7 @@ static int alc861vd_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âalc880_dac_to_idx(
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âspec->multiout.dac_nids[i]));
>
> - Â Â Â Â Â Â Â if (i == 2) {
> + Â Â Â Â Â Â Â if (!pfx && i == 2) {
> Â Â Â Â Â Â Â Â Â Â Â Â/* Center/LFE */
> Â Â Â Â Â Â Â Â Â Â Â Âerr = add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"Center",
> @@ -17073,24 +17083,17 @@ static int alc861vd_auto_create_multi_out_ctls(struct alc_spec *spec,
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> Â Â Â Â Â Â Â Â} else {
> - Â Â Â Â Â Â Â Â Â Â Â const char *pfx;
> - Â Â Â Â Â Â Â Â Â Â Â if (cfg->line_outs == 1 &&
> - Â Â Â Â Â Â Â Â Â Â Â Â Â cfg->line_out_type == AUTO_PIN_SPEAKER_OUT) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!cfg->hp_pins)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = "Speaker";
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â else
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = "PCM";
> - Â Â Â Â Â Â Â Â Â Â Â } else
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = chname[i];
> - Â Â Â Â Â Â Â Â Â Â Â err = add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL, pfx,
> + Â Â Â Â Â Â Â Â Â Â Â const char *name = pfx;
> + Â Â Â Â Â Â Â Â Â Â Â if (!name)
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â name = chname[i];
> + Â Â Â Â Â Â Â Â Â Â Â err = __add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â name, i,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_COMPOSE_AMP_VAL(nid_v, 3, 0,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_OUTPUT));
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> - Â Â Â Â Â Â Â Â Â Â Â if (cfg->line_outs == 1 &&
> - Â Â Â Â Â Â Â Â Â Â Â Â Â cfg->line_out_type == AUTO_PIN_SPEAKER_OUT)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = "Speaker";
> - Â Â Â Â Â Â Â Â Â Â Â err = add_pb_sw_ctrl(spec, ALC_CTL_BIND_MUTE, pfx,
> + Â Â Â Â Â Â Â Â Â Â Â err = __add_pb_sw_ctrl(spec, ALC_CTL_BIND_MUTE,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âname, i,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_COMPOSE_AMP_VAL(nid_s, 3, 2,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂHDA_INPUT));
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> @@ -19078,20 +19081,24 @@ static int alc662_auto_fill_dac_nids(struct hda_codec *codec,
> Â Â Â Âreturn 0;
> Â}
>
> -static inline int alc662_add_vol_ctl(struct alc_spec *spec, const char *pfx,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â hda_nid_t nid, unsigned int chs)
> +static inline int __alc662_add_vol_ctl(struct alc_spec *spec, const char *pfx,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhda_nid_t nid, int idx, unsigned int chs)
> Â{
> - Â Â Â return add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL, pfx,
> + Â Â Â return __add_pb_vol_ctrl(spec, ALC_CTL_WIDGET_VOL, pfx, idx,
> Â Â Â Â Â Â Â Â Â Â Â Â Â HDA_COMPOSE_AMP_VAL(nid, chs, 0, HDA_OUTPUT));
> Â}
>
> -static inline int alc662_add_sw_ctl(struct alc_spec *spec, const char *pfx,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Âhda_nid_t nid, unsigned int chs)
> +static inline int __alc662_add_sw_ctl(struct alc_spec *spec, const char *pfx,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â hda_nid_t nid, int idx, unsigned int chs)
> Â{
> - Â Â Â return add_pb_sw_ctrl(spec, ALC_CTL_WIDGET_MUTE, pfx,
> + Â Â Â return __add_pb_sw_ctrl(spec, ALC_CTL_WIDGET_MUTE, pfx, idx,
> Â Â Â Â Â Â Â Â Â Â Â Â Â HDA_COMPOSE_AMP_VAL(nid, chs, 0, HDA_INPUT));
> Â}
>
> +#define alc662_add_vol_ctl(spec, pfx, nid, chs) \
> + Â Â Â __alc662_add_vol_ctl(spec, pfx, nid, 0, chs)
> +#define alc662_add_sw_ctl(spec, pfx, nid, chs) \
> + Â Â Â __alc662_add_sw_ctl(spec, pfx, nid, 0, chs)
> Â#define alc662_add_stereo_vol(spec, pfx, nid) \
> Â Â Â Âalc662_add_vol_ctl(spec, pfx, nid, 3)
> Â#define alc662_add_stereo_sw(spec, pfx, nid) \
> @@ -19105,6 +19112,7 @@ static int alc662_auto_create_multi_out_ctls(struct hda_codec *codec,
> Â Â Â Âstatic const char *chname[4] = {
> Â Â Â Â Â Â Â Â"Front", "Surround", NULL /*CLFE*/, "Side"
> Â Â Â Â};
> + Â Â Â const char *pfx = alc_get_line_out_pfx(cfg, true);
> Â Â Â Âhda_nid_t nid, mix;
> Â Â Â Âint i, err;
>
> @@ -19115,7 +19123,7 @@ static int alc662_auto_create_multi_out_ctls(struct hda_codec *codec,
> Â Â Â Â Â Â Â Âmix = alc662_dac_to_mix(codec, cfg->line_out_pins[i], nid);
> Â Â Â Â Â Â Â Âif (!mix)
> Â Â Â Â Â Â Â Â Â Â Â Âcontinue;
> - Â Â Â Â Â Â Â if (i == 2) {
> + Â Â Â Â Â Â Â if (!pfx && i == 2) {
> Â Â Â Â Â Â Â Â Â Â Â Â/* Center/LFE */
> Â Â Â Â Â Â Â Â Â Â Â Âerr = alc662_add_vol_ctl(spec, "Center", nid, 1);
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> @@ -19130,22 +19138,13 @@ static int alc662_auto_create_multi_out_ctls(struct hda_codec *codec,
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> Â Â Â Â Â Â Â Â} else {
> - Â Â Â Â Â Â Â Â Â Â Â const char *pfx;
> - Â Â Â Â Â Â Â Â Â Â Â if (cfg->line_outs == 1 &&
> - Â Â Â Â Â Â Â Â Â Â Â Â Â cfg->line_out_type == AUTO_PIN_SPEAKER_OUT) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (cfg->hp_outs)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = "Speaker";
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â else
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = "PCM";
> - Â Â Â Â Â Â Â Â Â Â Â } else
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = chname[i];
> - Â Â Â Â Â Â Â Â Â Â Â err = alc662_add_vol_ctl(spec, pfx, nid, 3);
> + Â Â Â Â Â Â Â Â Â Â Â const char *name = pfx;
> + Â Â Â Â Â Â Â Â Â Â Â if (!name)
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â name = chname[i];
> + Â Â Â Â Â Â Â Â Â Â Â err = __alc662_add_vol_ctl(spec, name, nid, i, 3);
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> - Â Â Â Â Â Â Â Â Â Â Â if (cfg->line_outs == 1 &&
> - Â Â Â Â Â Â Â Â Â Â Â Â Â cfg->line_out_type == AUTO_PIN_SPEAKER_OUT)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pfx = "Speaker";
> - Â Â Â Â Â Â Â Â Â Â Â err = alc662_add_sw_ctl(spec, pfx, mix, 3);
> + Â Â Â Â Â Â Â Â Â Â Â err = __alc662_add_sw_ctl(spec, name, mix, i, 3);
> Â Â Â Â Â Â Â Â Â Â Â Âif (err < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn err;
> Â Â Â Â Â Â Â Â}
> --
> 1.7.3.4
>
>



-- 
~Nirbheek Chauhan

Gentoo GNOME+Mozilla Team
èº{.nÇ+‰·Ÿ®‰­†+%ŠËlzwm…ébëæìr¸›zX§»®w¥Š{ayºÊÚë,j­¢f£¢·hš‹àz¹®w¥¢¸¢·¦j:+v‰¨ŠwèjØm¶Ÿÿ¾«‘êçzZ+ƒùšŽŠÝj"ú!¶iO•æ¬z·švØ^¶m§ÿðÃnÆàþY&—