ALSA: hda/realtek - Fix missing volume controls with ALC260
ALC260 has multiple mixer widgets connected to the shared DAC, but the driver currently doesn't check this possibility and ignores when the DAC is shared with others. This resulted in the silent output from some routes because of lack of the amp setup. This patch adds the workaround for it by checking the route even with the shared DAC, but also checking the conflict with the existing control for the very same widget NID. Reference: https://bugzilla.novell.com/show_bug.cgi?id=726812 Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
parent
5cdf745eba
commit
527e4d73af
|
@ -600,7 +600,8 @@ int snd_hda_check_amp_list_power(struct hda_codec *codec,
|
||||||
#define get_amp_nid_(pv) ((pv) & 0xffff)
|
#define get_amp_nid_(pv) ((pv) & 0xffff)
|
||||||
#define get_amp_nid(kc) get_amp_nid_((kc)->private_value)
|
#define get_amp_nid(kc) get_amp_nid_((kc)->private_value)
|
||||||
#define get_amp_channels(kc) (((kc)->private_value >> 16) & 0x3)
|
#define get_amp_channels(kc) (((kc)->private_value >> 16) & 0x3)
|
||||||
#define get_amp_direction(kc) (((kc)->private_value >> 18) & 0x1)
|
#define get_amp_direction_(pv) (((pv) >> 18) & 0x1)
|
||||||
|
#define get_amp_direction(kc) get_amp_direction_((kc)->private_value)
|
||||||
#define get_amp_index(kc) (((kc)->private_value >> 19) & 0xf)
|
#define get_amp_index(kc) (((kc)->private_value >> 19) & 0xf)
|
||||||
#define get_amp_offset(kc) (((kc)->private_value >> 23) & 0x3f)
|
#define get_amp_offset(kc) (((kc)->private_value >> 23) & 0x3f)
|
||||||
#define get_amp_min_mute(kc) (((kc)->private_value >> 29) & 0x1)
|
#define get_amp_min_mute(kc) (((kc)->private_value >> 29) & 0x1)
|
||||||
|
|
|
@ -116,6 +116,8 @@ struct alc_spec {
|
||||||
const hda_nid_t *capsrc_nids;
|
const hda_nid_t *capsrc_nids;
|
||||||
hda_nid_t dig_in_nid; /* digital-in NID; optional */
|
hda_nid_t dig_in_nid; /* digital-in NID; optional */
|
||||||
hda_nid_t mixer_nid; /* analog-mixer NID */
|
hda_nid_t mixer_nid; /* analog-mixer NID */
|
||||||
|
DECLARE_BITMAP(vol_ctls, 0x20 << 1);
|
||||||
|
DECLARE_BITMAP(sw_ctls, 0x20 << 1);
|
||||||
|
|
||||||
/* capture setup for dynamic dual-adc switch */
|
/* capture setup for dynamic dual-adc switch */
|
||||||
hda_nid_t cur_adc;
|
hda_nid_t cur_adc;
|
||||||
|
@ -3006,14 +3008,32 @@ static int alc_auto_fill_dac_nids(struct hda_codec *codec)
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static inline unsigned int get_ctl_pos(unsigned int data)
|
||||||
|
{
|
||||||
|
hda_nid_t nid = get_amp_nid_(data);
|
||||||
|
unsigned int dir = get_amp_direction_(data);
|
||||||
|
return (nid << 1) | dir;
|
||||||
|
}
|
||||||
|
|
||||||
|
#define is_ctl_used(bits, data) \
|
||||||
|
test_bit(get_ctl_pos(data), bits)
|
||||||
|
#define mark_ctl_usage(bits, data) \
|
||||||
|
set_bit(get_ctl_pos(data), bits)
|
||||||
|
|
||||||
static int alc_auto_add_vol_ctl(struct hda_codec *codec,
|
static int alc_auto_add_vol_ctl(struct hda_codec *codec,
|
||||||
const char *pfx, int cidx,
|
const char *pfx, int cidx,
|
||||||
hda_nid_t nid, unsigned int chs)
|
hda_nid_t nid, unsigned int chs)
|
||||||
{
|
{
|
||||||
|
struct alc_spec *spec = codec->spec;
|
||||||
|
unsigned int val;
|
||||||
if (!nid)
|
if (!nid)
|
||||||
return 0;
|
return 0;
|
||||||
|
val = HDA_COMPOSE_AMP_VAL(nid, chs, 0, HDA_OUTPUT);
|
||||||
|
if (is_ctl_used(spec->vol_ctls, val) && chs != 2) /* exclude LFE */
|
||||||
|
return 0;
|
||||||
|
mark_ctl_usage(spec->vol_ctls, val);
|
||||||
return __add_pb_vol_ctrl(codec->spec, ALC_CTL_WIDGET_VOL, pfx, cidx,
|
return __add_pb_vol_ctrl(codec->spec, ALC_CTL_WIDGET_VOL, pfx, cidx,
|
||||||
HDA_COMPOSE_AMP_VAL(nid, chs, 0, HDA_OUTPUT));
|
val);
|
||||||
}
|
}
|
||||||
|
|
||||||
#define alc_auto_add_stereo_vol(codec, pfx, cidx, nid) \
|
#define alc_auto_add_stereo_vol(codec, pfx, cidx, nid) \
|
||||||
|
@ -3026,6 +3046,7 @@ static int alc_auto_add_sw_ctl(struct hda_codec *codec,
|
||||||
const char *pfx, int cidx,
|
const char *pfx, int cidx,
|
||||||
hda_nid_t nid, unsigned int chs)
|
hda_nid_t nid, unsigned int chs)
|
||||||
{
|
{
|
||||||
|
struct alc_spec *spec = codec->spec;
|
||||||
int wid_type;
|
int wid_type;
|
||||||
int type;
|
int type;
|
||||||
unsigned long val;
|
unsigned long val;
|
||||||
|
@ -3042,6 +3063,9 @@ static int alc_auto_add_sw_ctl(struct hda_codec *codec,
|
||||||
type = ALC_CTL_BIND_MUTE;
|
type = ALC_CTL_BIND_MUTE;
|
||||||
val = HDA_COMPOSE_AMP_VAL(nid, chs, 2, HDA_INPUT);
|
val = HDA_COMPOSE_AMP_VAL(nid, chs, 2, HDA_INPUT);
|
||||||
}
|
}
|
||||||
|
if (is_ctl_used(spec->sw_ctls, val) && chs != 2) /* exclude LFE */
|
||||||
|
return 0;
|
||||||
|
mark_ctl_usage(spec->sw_ctls, val);
|
||||||
return __add_pb_sw_ctrl(codec->spec, type, pfx, cidx, val);
|
return __add_pb_sw_ctrl(codec->spec, type, pfx, cidx, val);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -3136,12 +3160,16 @@ static int alc_auto_create_extra_out(struct hda_codec *codec, hda_nid_t pin,
|
||||||
int err;
|
int err;
|
||||||
|
|
||||||
if (!dac) {
|
if (!dac) {
|
||||||
|
unsigned int val;
|
||||||
/* the corresponding DAC is already occupied */
|
/* the corresponding DAC is already occupied */
|
||||||
if (!(get_wcaps(codec, pin) & AC_WCAP_OUT_AMP))
|
if (!(get_wcaps(codec, pin) & AC_WCAP_OUT_AMP))
|
||||||
return 0; /* no way */
|
return 0; /* no way */
|
||||||
/* create a switch only */
|
/* create a switch only */
|
||||||
return add_pb_sw_ctrl(spec, ALC_CTL_WIDGET_MUTE, pfx,
|
val = HDA_COMPOSE_AMP_VAL(pin, 3, 0, HDA_OUTPUT);
|
||||||
HDA_COMPOSE_AMP_VAL(pin, 3, 0, HDA_OUTPUT));
|
if (is_ctl_used(spec->sw_ctls, val))
|
||||||
|
return 0; /* already created */
|
||||||
|
mark_ctl_usage(spec->sw_ctls, val);
|
||||||
|
return add_pb_sw_ctrl(spec, ALC_CTL_WIDGET_MUTE, pfx, val);
|
||||||
}
|
}
|
||||||
|
|
||||||
sw = alc_look_for_out_mute_nid(codec, pin, dac);
|
sw = alc_look_for_out_mute_nid(codec, pin, dac);
|
||||||
|
@ -3186,8 +3214,12 @@ static int alc_auto_create_extra_outs(struct hda_codec *codec, int num_pins,
|
||||||
if (!num_pins || !pins[0])
|
if (!num_pins || !pins[0])
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
if (num_pins == 1)
|
if (num_pins == 1) {
|
||||||
return alc_auto_create_extra_out(codec, *pins, *dacs, pfx);
|
hda_nid_t dac = *dacs;
|
||||||
|
if (!dac)
|
||||||
|
dac = spec->multiout.dac_nids[0];
|
||||||
|
return alc_auto_create_extra_out(codec, *pins, dac, pfx);
|
||||||
|
}
|
||||||
|
|
||||||
if (dacs[num_pins - 1]) {
|
if (dacs[num_pins - 1]) {
|
||||||
/* OK, we have a multi-output system with individual volumes */
|
/* OK, we have a multi-output system with individual volumes */
|
||||||
|
|
Loading…
Reference in New Issue