ALSA: hda - Consolidate cap_sync_hook and capture_switch_hook
Two hooks in hda_gen_spec, cap_sync_hook and capture_switch_hook, play
very similar roles. The only differences are that the former is
called more often (e.g. at init or switching capsrc) while the latter
can take an on/off argument.
As a more generic implementation, consolidate these two hooks, and
pass snd_ctl_elem_value pointer as the second argument. If the
secondary argument is non-NULL, it can take the on/off value, so the
caller handles it like the former capture_switch_hook. If it's NULL,
it's called in the init or capsrc switch case.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
diff --git a/sound/pci/hda/hda_generic.c b/sound/pci/hda/hda_generic.c
index e9af9ab..f0d83b2 100644
--- a/sound/pci/hda/hda_generic.c
+++ b/sound/pci/hda/hda_generic.c
@@ -2738,7 +2738,7 @@
mutex_unlock(&codec->control_mutex);
snd_hda_codec_flush_amp_cache(codec); /* flush the updates */
if (err >= 0 && spec->cap_sync_hook)
- spec->cap_sync_hook(codec);
+ spec->cap_sync_hook(codec, ucontrol);
return err;
}
@@ -2774,23 +2774,9 @@
static int cap_sw_put(struct snd_kcontrol *kcontrol,
struct snd_ctl_elem_value *ucontrol)
{
- struct hda_codec *codec = snd_kcontrol_chip(kcontrol);
- struct hda_gen_spec *spec = codec->spec;
- int ret;
-
- ret = cap_put_caller(kcontrol, ucontrol,
+ return cap_put_caller(kcontrol, ucontrol,
snd_hda_mixer_amp_switch_put,
NID_PATH_MUTE_CTL);
- if (ret < 0)
- return ret;
-
- if (spec->capture_switch_hook) {
- bool enable = (ucontrol->value.integer.value[0] ||
- ucontrol->value.integer.value[1]);
- spec->capture_switch_hook(codec, enable);
- }
-
- return ret;
}
static const struct snd_kcontrol_new cap_sw_temp = {
@@ -2860,6 +2846,7 @@
return false;
}
+/* capture switch put callback for a single control with hook call */
static int cap_single_sw_put(struct snd_kcontrol *kcontrol,
struct snd_ctl_elem_value *ucontrol)
{
@@ -2871,11 +2858,8 @@
if (ret < 0)
return ret;
- if (spec->capture_switch_hook) {
- bool enable = (ucontrol->value.integer.value[0] ||
- ucontrol->value.integer.value[1]);
- spec->capture_switch_hook(codec, enable);
- }
+ if (spec->cap_sync_hook)
+ spec->cap_sync_hook(codec, ucontrol);
return ret;
}
@@ -2904,7 +2888,7 @@
amp_val_replace_channels(ctl, chs));
if (!knew)
return -ENOMEM;
- if (is_switch && spec->capture_switch_hook)
+ if (is_switch)
knew->put = cap_single_sw_put;
if (!inv_dmic)
return 0;
@@ -2920,7 +2904,7 @@
amp_val_replace_channels(ctl, 2));
if (!knew)
return -ENOMEM;
- if (is_switch && spec->capture_switch_hook)
+ if (is_switch)
knew->put = cap_single_sw_put;
return 0;
}
@@ -3280,7 +3264,7 @@
return 0;
snd_hda_activate_path(codec, path, true, false);
if (spec->cap_sync_hook)
- spec->cap_sync_hook(codec);
+ spec->cap_sync_hook(codec, NULL);
return 1;
}
@@ -4610,7 +4594,7 @@
update_shared_mic_hp(codec, spec->cur_mux[0]);
if (spec->cap_sync_hook)
- spec->cap_sync_hook(codec);
+ spec->cap_sync_hook(codec, NULL);
}
/* set right pin controls for digital I/O */