Skip to content

L
linux
Commit 42caae0e authored by Geoffrey D. Bennett's avatar Geoffrey D. Bennett Committed by Takashi Iwai
Browse files
Options

ALSA: scarlett2: Refactor common port_count lookups 

Rather than looking up the analogue and mixer I/O counts repeatedly in
info->port_count[SCARLETT2_PORT_TYPE_*][SCARLETT2_PORT_*], save those
numbers in private variables.
Signed-off-by: default avatarGeoffrey D. Bennett <g@b4.vu>
Link: https://lore.kernel.org/r/db0a5b56bdff476e2e31ad8e5ee15008314412b7.1703444932.git.g@b4.vuSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent e5fab78c
Hide whitespace changes
Inline Side-by-side
Showing with 35 additions and 72 deletions
sound/usb/mixer_scarlett2.c
View file @ 42caae0e
...@@ -609,6 +609,9 @@ struct scarlett2_data { ...@@ -609,6 +609,9 @@ struct scarlett2_data {
__u8 bInterval; __u8 bInterval;
u8 num_mux_srcs; u8 num_mux_srcs;
u8 num_mux_dsts; u8 num_mux_dsts;
u8 num_mix_in;
u8 num_mix_out;
u8 num_line_out;
u32 firmware_version; u32 firmware_version;
u8 flash_segment_nums[SCARLETT2_SEGMENT_ID_COUNT]; u8 flash_segment_nums[SCARLETT2_SEGMENT_ID_COUNT];
u8 flash_segment_blocks[SCARLETT2_SEGMENT_ID_COUNT]; u8 flash_segment_blocks[SCARLETT2_SEGMENT_ID_COUNT];
...@@ -1744,10 +1747,8 @@ static int scarlett2_usb_get_mix(struct usb_mixer_interface *mixer, ...@@ -1744,10 +1747,8 @@ static int scarlett2_usb_get_mix(struct usb_mixer_interface *mixer,
int mix_num) int mix_num)
{ {
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info;
int num_mixer_in = int num_mixer_in = private->num_mix_in;
info->port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_OUT];
int err, i, j, k; int err, i, j, k;
struct { struct {
...@@ -1787,7 +1788,6 @@ static int scarlett2_usb_set_mix(struct usb_mixer_interface *mixer, ...@@ -1787,7 +1788,6 @@ static int scarlett2_usb_set_mix(struct usb_mixer_interface *mixer,
int mix_num) int mix_num)
{ {
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info;
struct { struct {
__le16 mix_num; __le16 mix_num;
...@@ -1795,8 +1795,7 @@ static int scarlett2_usb_set_mix(struct usb_mixer_interface *mixer, ...@@ -1795,8 +1795,7 @@ static int scarlett2_usb_set_mix(struct usb_mixer_interface *mixer,
} __packed req; } __packed req;
int i, j; int i, j;
int num_mixer_in = int num_mixer_in = private->num_mix_in;
info->port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_OUT];
req.mix_num = cpu_to_le16(mix_num); req.mix_num = cpu_to_le16(mix_num);
...@@ -1907,9 +1906,6 @@ static void scarlett2_usb_populate_mux(struct scarlett2_data *private, ...@@ -1907,9 +1906,6 @@ static void scarlett2_usb_populate_mux(struct scarlett2_data *private,
static void scarlett2_update_meter_level_map(struct scarlett2_data *private) static void scarlett2_update_meter_level_map(struct scarlett2_data *private)
{ {
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int line_out_count =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
const struct scarlett2_meter_entry *entry; const struct scarlett2_meter_entry *entry;
/* sources already assigned to a destination /* sources already assigned to a destination
...@@ -1938,7 +1934,7 @@ static void scarlett2_update_meter_level_map(struct scarlett2_data *private) ...@@ -1938,7 +1934,7 @@ static void scarlett2_update_meter_level_map(struct scarlett2_data *private)
/* convert mux_idx using line_out_unmap[] */ /* convert mux_idx using line_out_unmap[] */
int map_mux_idx = ( int map_mux_idx = (
info->line_out_remap_enable && info->line_out_remap_enable &&
mux_idx < line_out_count mux_idx < private->num_line_out
) ? info->line_out_unmap[mux_idx] ) ? info->line_out_unmap[mux_idx]
: mux_idx; : mux_idx;
...@@ -2249,10 +2245,7 @@ static int scarlett2_update_volumes(struct usb_mixer_interface *mixer) ...@@ -2249,10 +2245,7 @@ static int scarlett2_update_volumes(struct usb_mixer_interface *mixer)
{ {
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
struct scarlett2_usb_volume_status volume_status; struct scarlett2_usb_volume_status volume_status;
int num_line_out =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
int err, i; int err, i;
int mute; int mute;
...@@ -2272,7 +2265,7 @@ static int scarlett2_update_volumes(struct usb_mixer_interface *mixer) ...@@ -2272,7 +2265,7 @@ static int scarlett2_update_volumes(struct usb_mixer_interface *mixer)
mute = private->dim_mute[SCARLETT2_BUTTON_MUTE]; mute = private->dim_mute[SCARLETT2_BUTTON_MUTE];
for (i = 0; i < num_line_out; i++) for (i = 0; i < private->num_line_out; i++)
if (private->vol_sw_hw_switch[i]) { if (private->vol_sw_hw_switch[i]) {
private->vol[i] = private->master_vol; private->vol[i] = private->master_vol;
private->mute_switch[i] = mute; private->mute_switch[i] = mute;
...@@ -2324,14 +2317,11 @@ static int scarlett2_master_volume_ctl_get(struct snd_kcontrol *kctl, ...@@ -2324,14 +2317,11 @@ static int scarlett2_master_volume_ctl_get(struct snd_kcontrol *kctl,
static int line_out_remap(struct scarlett2_data *private, int index) static int line_out_remap(struct scarlett2_data *private, int index)
{ {
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int line_out_count =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
if (!info->line_out_remap_enable) if (!info->line_out_remap_enable)
return index; return index;
if (index >= line_out_count) if (index >= private->num_line_out)
return index; return index;
return info->line_out_remap[index]; return info->line_out_remap[index];
...@@ -3104,10 +3094,6 @@ static int scarlett2_update_monitor_other(struct usb_mixer_interface *mixer) ...@@ -3104,10 +3094,6 @@ static int scarlett2_update_monitor_other(struct usb_mixer_interface *mixer)
private->speaker_switching_switch = monitor_other_switch[0] + 1; private->speaker_switching_switch = monitor_other_switch[0] + 1;
if (info->has_talkback) { if (info->has_talkback) {
const int (*port_count)[SCARLETT2_PORT_DIRNS] =
info->port_count;
int num_mixes =
port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_IN];
u16 bitmap; u16 bitmap;
int i; int i;
...@@ -3121,7 +3107,7 @@ static int scarlett2_update_monitor_other(struct usb_mixer_interface *mixer) ...@@ -3121,7 +3107,7 @@ static int scarlett2_update_monitor_other(struct usb_mixer_interface *mixer)
1, &bitmap); 1, &bitmap);
if (err < 0) if (err < 0)
return err; return err;
for (i = 0; i < num_mixes; i++, bitmap >>= 1) for (i = 0; i < private->num_mix_out; i++, bitmap >>= 1)
private->talkback_map[i] = bitmap & 1; private->talkback_map[i] = bitmap & 1;
} }
...@@ -3518,10 +3504,6 @@ static int scarlett2_talkback_map_ctl_put( ...@@ -3518,10 +3504,6 @@ static int scarlett2_talkback_map_ctl_put(
struct usb_mixer_elem_info *elem = kctl->private_data; struct usb_mixer_elem_info *elem = kctl->private_data;
struct usb_mixer_interface *mixer = elem->head.mixer; struct usb_mixer_interface *mixer = elem->head.mixer;
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const int (*port_count)[SCARLETT2_PORT_DIRNS] =
private->info->port_count;
int num_mixes = port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_IN];
int index = elem->control; int index = elem->control;
int oval, val, err = 0, i; int oval, val, err = 0, i;
u16 bitmap = 0; u16 bitmap = 0;
...@@ -3541,7 +3523,7 @@ static int scarlett2_talkback_map_ctl_put( ...@@ -3541,7 +3523,7 @@ static int scarlett2_talkback_map_ctl_put(
private->talkback_map[index] = val; private->talkback_map[index] = val;
for (i = 0; i < num_mixes; i++) for (i = 0; i < private->num_mix_out; i++)
bitmap |= private->talkback_map[i] << i; bitmap |= private->talkback_map[i] << i;
/* Send updated bitmap to the device */ /* Send updated bitmap to the device */
...@@ -3567,8 +3549,6 @@ static int scarlett2_add_talkback_ctls(struct usb_mixer_interface *mixer) ...@@ -3567,8 +3549,6 @@ static int scarlett2_add_talkback_ctls(struct usb_mixer_interface *mixer)
{ {
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int num_mixes = port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_IN];
int err, i; int err, i;
char s[SNDRV_CTL_ELEM_ID_NAME_MAXLEN]; char s[SNDRV_CTL_ELEM_ID_NAME_MAXLEN];
...@@ -3582,7 +3562,7 @@ static int scarlett2_add_talkback_ctls(struct usb_mixer_interface *mixer) ...@@ -3582,7 +3562,7 @@ static int scarlett2_add_talkback_ctls(struct usb_mixer_interface *mixer)
if (err < 0) if (err < 0)
return err; return err;
for (i = 0; i < num_mixes; i++) { for (i = 0; i < private->num_mix_out; i++) {
snprintf(s, sizeof(s), snprintf(s, sizeof(s),
"Talkback Mix %c Playback Switch", i + 'A'); "Talkback Mix %c Playback Switch", i + 'A');
err = scarlett2_add_new_ctl(mixer, &scarlett2_talkback_map_ctl, err = scarlett2_add_new_ctl(mixer, &scarlett2_talkback_map_ctl,
...@@ -3629,11 +3609,6 @@ static int scarlett2_dim_mute_ctl_put(struct snd_kcontrol *kctl, ...@@ -3629,11 +3609,6 @@ static int scarlett2_dim_mute_ctl_put(struct snd_kcontrol *kctl,
struct usb_mixer_elem_info *elem = kctl->private_data; struct usb_mixer_elem_info *elem = kctl->private_data;
struct usb_mixer_interface *mixer = elem->head.mixer; struct usb_mixer_interface *mixer = elem->head.mixer;
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int num_line_out =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
int index = elem->control; int index = elem->control;
int oval, val, err = 0, i; int oval, val, err = 0, i;
...@@ -3659,7 +3634,7 @@ static int scarlett2_dim_mute_ctl_put(struct snd_kcontrol *kctl, ...@@ -3659,7 +3634,7 @@ static int scarlett2_dim_mute_ctl_put(struct snd_kcontrol *kctl,
err = 1; err = 1;
if (index == SCARLETT2_BUTTON_MUTE) if (index == SCARLETT2_BUTTON_MUTE)
for (i = 0; i < num_line_out; i++) { for (i = 0; i < private->num_line_out; i++) {
int line_index = line_out_remap(private, i); int line_index = line_out_remap(private, i);
if (private->vol_sw_hw_switch[line_index]) { if (private->vol_sw_hw_switch[line_index]) {
...@@ -3689,9 +3664,6 @@ static int scarlett2_add_line_out_ctls(struct usb_mixer_interface *mixer) ...@@ -3689,9 +3664,6 @@ static int scarlett2_add_line_out_ctls(struct usb_mixer_interface *mixer)
{ {
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int num_line_out =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
int err, i; int err, i;
char s[SNDRV_CTL_ELEM_ID_NAME_MAXLEN]; char s[SNDRV_CTL_ELEM_ID_NAME_MAXLEN];
...@@ -3706,7 +3678,7 @@ static int scarlett2_add_line_out_ctls(struct usb_mixer_interface *mixer) ...@@ -3706,7 +3678,7 @@ static int scarlett2_add_line_out_ctls(struct usb_mixer_interface *mixer)
} }
/* Add volume controls */ /* Add volume controls */
for (i = 0; i < num_line_out; i++) { for (i = 0; i < private->num_line_out; i++) {
int index = line_out_remap(private, i); int index = line_out_remap(private, i);
/* Fader */ /* Fader */
...@@ -3883,9 +3855,7 @@ static int scarlett2_mixer_ctl_put(struct snd_kcontrol *kctl, ...@@ -3883,9 +3855,7 @@ static int scarlett2_mixer_ctl_put(struct snd_kcontrol *kctl,
struct usb_mixer_elem_info *elem = kctl->private_data; struct usb_mixer_elem_info *elem = kctl->private_data;
struct usb_mixer_interface *mixer = elem->head.mixer; struct usb_mixer_interface *mixer = elem->head.mixer;
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info; int oval, val, mix_num, err = 0;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int oval, val, num_mixer_in, mix_num, err = 0;
int index = elem->control; int index = elem->control;
mutex_lock(&private->data_mutex); mutex_lock(&private->data_mutex);
...@@ -3898,8 +3868,7 @@ static int scarlett2_mixer_ctl_put(struct snd_kcontrol *kctl, ...@@ -3898,8 +3868,7 @@ static int scarlett2_mixer_ctl_put(struct snd_kcontrol *kctl,
oval = private->mix[index]; oval = private->mix[index];
val = clamp(ucontrol->value.integer.value[0], val = clamp(ucontrol->value.integer.value[0],
0L, (long)SCARLETT2_MIXER_MAX_VALUE); 0L, (long)SCARLETT2_MIXER_MAX_VALUE);
num_mixer_in = port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_OUT]; mix_num = index / private->num_mix_in;
mix_num = index / num_mixer_in;
if (oval == val) if (oval == val)
goto unlock; goto unlock;
...@@ -3935,19 +3904,12 @@ static const struct snd_kcontrol_new scarlett2_mixer_ctl = { ...@@ -3935,19 +3904,12 @@ static const struct snd_kcontrol_new scarlett2_mixer_ctl = {
static int scarlett2_add_mixer_ctls(struct usb_mixer_interface *mixer) static int scarlett2_add_mixer_ctls(struct usb_mixer_interface *mixer)
{ {
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int err, i, j; int err, i, j;
int index; int index;
char s[SNDRV_CTL_ELEM_ID_NAME_MAXLEN]; char s[SNDRV_CTL_ELEM_ID_NAME_MAXLEN];
int num_inputs = for (i = 0, index = 0; i < private->num_mix_out; i++)
port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_OUT]; for (j = 0; j < private->num_mix_in; j++, index++) {
int num_outputs =
port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_IN];
for (i = 0, index = 0; i < num_outputs; i++)
for (j = 0; j < num_inputs; j++, index++) {
snprintf(s, sizeof(s), snprintf(s, sizeof(s),
"Mix %c Input %02d Playback Volume", "Mix %c Input %02d Playback Volume",
'A' + i, j + 1); 'A' + i, j + 1);
...@@ -4336,12 +4298,13 @@ static void scarlett2_private_suspend(struct usb_mixer_interface *mixer) ...@@ -4336,12 +4298,13 @@ static void scarlett2_private_suspend(struct usb_mixer_interface *mixer)
/*** Initialisation ***/ /*** Initialisation ***/
static void scarlett2_count_mux_io(struct scarlett2_data *private) static void scarlett2_count_io(struct scarlett2_data *private)
{ {
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count; const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int port_type, srcs = 0, dsts = 0; int port_type, srcs = 0, dsts = 0;
/* Count the number of mux sources and destinations */
for (port_type = 0; for (port_type = 0;
port_type < SCARLETT2_PORT_TYPE_COUNT; port_type < SCARLETT2_PORT_TYPE_COUNT;
port_type++) { port_type++) {
...@@ -4351,6 +4314,17 @@ static void scarlett2_count_mux_io(struct scarlett2_data *private) ...@@ -4351,6 +4314,17 @@ static void scarlett2_count_mux_io(struct scarlett2_data *private)
private->num_mux_srcs = srcs; private->num_mux_srcs = srcs;
private->num_mux_dsts = dsts; private->num_mux_dsts = dsts;
/* Mixer inputs are mux outputs and vice versa */
private->num_mix_in =
port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_OUT];
private->num_mix_out =
port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_IN];
/* Number of analogue line outputs */
private->num_line_out =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
} }
/* Look through the interface descriptors for the Focusrite Control /* Look through the interface descriptors for the Focusrite Control
...@@ -4406,7 +4380,7 @@ static int scarlett2_init_private(struct usb_mixer_interface *mixer, ...@@ -4406,7 +4380,7 @@ static int scarlett2_init_private(struct usb_mixer_interface *mixer,
private->info = entry->info; private->info = entry->info;
private->config_set = entry->info->config_set; private->config_set = entry->info->config_set;
private->series_name = entry->series_name; private->series_name = entry->series_name;
scarlett2_count_mux_io(private); scarlett2_count_io(private);
private->scarlett2_seq = 0; private->scarlett2_seq = 0;
private->mixer = mixer; private->mixer = mixer;
...@@ -4544,11 +4518,6 @@ static int scarlett2_read_configs(struct usb_mixer_interface *mixer) ...@@ -4544,11 +4518,6 @@ static int scarlett2_read_configs(struct usb_mixer_interface *mixer)
{ {
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int num_line_out =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
int num_mixer_out =
port_count[SCARLETT2_PORT_TYPE_MIX][SCARLETT2_PORT_IN];
struct scarlett2_usb_volume_status volume_status; struct scarlett2_usb_volume_status volume_status;
int err, i; int err, i;
...@@ -4601,7 +4570,7 @@ static int scarlett2_read_configs(struct usb_mixer_interface *mixer) ...@@ -4601,7 +4570,7 @@ static int scarlett2_read_configs(struct usb_mixer_interface *mixer)
volume_status.master_vol + SCARLETT2_VOLUME_BIAS, volume_status.master_vol + SCARLETT2_VOLUME_BIAS,
0, SCARLETT2_VOLUME_BIAS); 0, SCARLETT2_VOLUME_BIAS);
for (i = 0; i < num_line_out; i++) { for (i = 0; i < private->num_line_out; i++) {
int volume, mute; int volume, mute;
private->vol_sw_hw_switch[i] = private->vol_sw_hw_switch[i] =
...@@ -4621,7 +4590,7 @@ static int scarlett2_read_configs(struct usb_mixer_interface *mixer) ...@@ -4621,7 +4590,7 @@ static int scarlett2_read_configs(struct usb_mixer_interface *mixer)
private->mute_switch[i] = mute; private->mute_switch[i] = mute;
} }
for (i = 0; i < num_mixer_out; i++) { for (i = 0; i < private->num_mix_out; i++) {
err = scarlett2_usb_get_mix(mixer, i); err = scarlett2_usb_get_mix(mixer, i);
if (err < 0) if (err < 0)
return err; return err;
...@@ -4647,9 +4616,6 @@ static void scarlett2_notify_monitor(struct usb_mixer_interface *mixer) ...@@ -4647,9 +4616,6 @@ static void scarlett2_notify_monitor(struct usb_mixer_interface *mixer)
struct snd_card *card = mixer->chip->card; struct snd_card *card = mixer->chip->card;
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int num_line_out =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
int i; int i;
/* if line_out_hw_vol is 0, there are no controls to update */ /* if line_out_hw_vol is 0, there are no controls to update */
...@@ -4661,7 +4627,7 @@ static void scarlett2_notify_monitor(struct usb_mixer_interface *mixer) ...@@ -4661,7 +4627,7 @@ static void scarlett2_notify_monitor(struct usb_mixer_interface *mixer)
snd_ctl_notify(mixer->chip->card, SNDRV_CTL_EVENT_MASK_VALUE, snd_ctl_notify(mixer->chip->card, SNDRV_CTL_EVENT_MASK_VALUE,
&private->master_vol_ctl->id); &private->master_vol_ctl->id);
for (i = 0; i < num_line_out; i++) for (i = 0; i < private->num_line_out; i++)
if (private->vol_sw_hw_switch[line_out_remap(private, i)]) if (private->vol_sw_hw_switch[line_out_remap(private, i)])
snd_ctl_notify(card, SNDRV_CTL_EVENT_MASK_VALUE, snd_ctl_notify(card, SNDRV_CTL_EVENT_MASK_VALUE,
&private->vol_ctls[i]->id); &private->vol_ctls[i]->id);
...@@ -4673,9 +4639,6 @@ static void scarlett2_notify_dim_mute(struct usb_mixer_interface *mixer) ...@@ -4673,9 +4639,6 @@ static void scarlett2_notify_dim_mute(struct usb_mixer_interface *mixer)
struct snd_card *card = mixer->chip->card; struct snd_card *card = mixer->chip->card;
struct scarlett2_data *private = mixer->private_data; struct scarlett2_data *private = mixer->private_data;
const struct scarlett2_device_info *info = private->info; const struct scarlett2_device_info *info = private->info;
const int (*port_count)[SCARLETT2_PORT_DIRNS] = info->port_count;
int num_line_out =
port_count[SCARLETT2_PORT_TYPE_ANALOGUE][SCARLETT2_PORT_OUT];
int i; int i;
private->vol_updated = 1; private->vol_updated = 1;
...@@ -4687,7 +4650,7 @@ static void scarlett2_notify_dim_mute(struct usb_mixer_interface *mixer) ...@@ -4687,7 +4650,7 @@ static void scarlett2_notify_dim_mute(struct usb_mixer_interface *mixer)
snd_ctl_notify(card, SNDRV_CTL_EVENT_MASK_VALUE, snd_ctl_notify(card, SNDRV_CTL_EVENT_MASK_VALUE,
&private->dim_mute_ctls[i]->id); &private->dim_mute_ctls[i]->id);
for (i = 0; i < num_line_out; i++) for (i = 0; i < private->num_line_out; i++)
if (private->vol_sw_hw_switch[line_out_remap(private, i)]) if (private->vol_sw_hw_switch[line_out_remap(private, i)])
snd_ctl_notify(card, SNDRV_CTL_EVENT_MASK_VALUE, snd_ctl_notify(card, SNDRV_CTL_EVENT_MASK_VALUE,
&private->mute_ctls[i]->id); &private->mute_ctls[i]->id);
......
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
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7