Commit fe6ce80a authored by Clemens Ladisch's avatar Clemens Ladisch Committed by Takashi Iwai

ALSA: virtuoso: fix setting of Xonar DS line-in/mic-in controls

The Line and Mic inputs cannot be used at the same time, so the driver
has to automatically disable one of them if both are set.  However, it
forgot to notify userspace about this change, so the mixer state would
be inconsistent.  To fix this, check if the other control gets muted,
and send a notification event in this case.
Signed-off-by: default avatarClemens Ladisch <clemens@ladisch.de>
Reported-and-tested-by: Nathan Schagen
Cc: <stable@kernel.org>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 4c25b932
...@@ -53,6 +53,8 @@ struct xonar_wm87x6 { ...@@ -53,6 +53,8 @@ struct xonar_wm87x6 {
struct xonar_generic generic; struct xonar_generic generic;
u16 wm8776_regs[0x17]; u16 wm8776_regs[0x17];
u16 wm8766_regs[0x10]; u16 wm8766_regs[0x10];
struct snd_kcontrol *line_adcmux_control;
struct snd_kcontrol *mic_adcmux_control;
struct snd_kcontrol *lc_controls[13]; struct snd_kcontrol *lc_controls[13];
}; };
...@@ -604,6 +606,7 @@ static int wm8776_input_mux_put(struct snd_kcontrol *ctl, ...@@ -604,6 +606,7 @@ static int wm8776_input_mux_put(struct snd_kcontrol *ctl,
{ {
struct oxygen *chip = ctl->private_data; struct oxygen *chip = ctl->private_data;
struct xonar_wm87x6 *data = chip->model_data; struct xonar_wm87x6 *data = chip->model_data;
struct snd_kcontrol *other_ctl;
unsigned int mux_bit = ctl->private_value; unsigned int mux_bit = ctl->private_value;
u16 reg; u16 reg;
int changed; int changed;
...@@ -611,8 +614,18 @@ static int wm8776_input_mux_put(struct snd_kcontrol *ctl, ...@@ -611,8 +614,18 @@ static int wm8776_input_mux_put(struct snd_kcontrol *ctl,
mutex_lock(&chip->mutex); mutex_lock(&chip->mutex);
reg = data->wm8776_regs[WM8776_ADCMUX]; reg = data->wm8776_regs[WM8776_ADCMUX];
if (value->value.integer.value[0]) { if (value->value.integer.value[0]) {
reg &= ~0x003;
reg |= mux_bit; reg |= mux_bit;
/* line-in and mic-in are exclusive */
mux_bit ^= 3;
if (reg & mux_bit) {
reg &= ~mux_bit;
if (mux_bit == 1)
other_ctl = data->line_adcmux_control;
else
other_ctl = data->mic_adcmux_control;
snd_ctl_notify(chip->card, SNDRV_CTL_EVENT_MASK_VALUE,
&other_ctl->id);
}
} else } else
reg &= ~mux_bit; reg &= ~mux_bit;
changed = reg != data->wm8776_regs[WM8776_ADCMUX]; changed = reg != data->wm8776_regs[WM8776_ADCMUX];
...@@ -964,7 +977,13 @@ static int xonar_ds_mixer_init(struct oxygen *chip) ...@@ -964,7 +977,13 @@ static int xonar_ds_mixer_init(struct oxygen *chip)
err = snd_ctl_add(chip->card, ctl); err = snd_ctl_add(chip->card, ctl);
if (err < 0) if (err < 0)
return err; return err;
if (!strcmp(ctl->id.name, "Line Capture Switch"))
data->line_adcmux_control = ctl;
else if (!strcmp(ctl->id.name, "Mic Capture Switch"))
data->mic_adcmux_control = ctl;
} }
if (!data->line_adcmux_control || !data->mic_adcmux_control)
return -ENXIO;
BUILD_BUG_ON(ARRAY_SIZE(lc_controls) != ARRAY_SIZE(data->lc_controls)); BUILD_BUG_ON(ARRAY_SIZE(lc_controls) != ARRAY_SIZE(data->lc_controls));
for (i = 0; i < ARRAY_SIZE(lc_controls); ++i) { for (i = 0; i < ARRAY_SIZE(lc_controls); ++i) {
ctl = snd_ctl_new1(&lc_controls[i], chip); ctl = snd_ctl_new1(&lc_controls[i], chip);
......
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