From a90229e0517938a5c5d9d682708b6ee7cd0e19b9 Mon Sep 17 00:00:00 2001 From: Takashi Iwai Date: Fri, 18 Jan 2013 14:10:00 +0100 Subject: [PATCH] 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 --- sound/pci/hda/hda_generic.c | 34 +++++++++------------------------- sound/pci/hda/hda_generic.h | 6 ++---- sound/pci/hda/patch_realtek.c | 3 ++- sound/pci/hda/patch_sigmatel.c | 12 +++++++++--- 4 files changed, 22 insertions(+), 33 deletions(-) diff --git a/sound/pci/hda/hda_generic.c b/sound/pci/hda/hda_generic.c index e9af9ab4fac5..f0d83b2f42ac 100644 --- a/sound/pci/hda/hda_generic.c +++ b/sound/pci/hda/hda_generic.c @@ -2738,7 +2738,7 @@ static int cap_put_caller(struct snd_kcontrol *kcontrol, 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 const struct snd_kcontrol_new cap_vol_temp = { 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 @@ static bool is_inv_dmic_pin(struct hda_codec *codec, hda_nid_t nid) 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 @@ static int cap_single_sw_put(struct snd_kcontrol *kcontrol, 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 @@ static int add_single_cap_ctl(struct hda_codec *codec, const char *label, 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 @@ static int add_single_cap_ctl(struct hda_codec *codec, const char *label, 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 @@ static int mux_select(struct hda_codec *codec, unsigned int adc_idx, 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 @@ static void init_input_src(struct hda_codec *codec) 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 */ diff --git a/sound/pci/hda/hda_generic.h b/sound/pci/hda/hda_generic.h index f6b88cd4584f..594a9cc4ecca 100644 --- a/sound/pci/hda/hda_generic.h +++ b/sound/pci/hda/hda_generic.h @@ -233,7 +233,8 @@ struct hda_gen_spec { /* hooks */ void (*init_hook)(struct hda_codec *codec); void (*automute_hook)(struct hda_codec *codec); - void (*cap_sync_hook)(struct hda_codec *codec); + void (*cap_sync_hook)(struct hda_codec *codec, + struct snd_ctl_elem_value *ucontrol); /* PCM hooks */ void (*pcm_playback_hook)(struct hda_pcm_stream *hinfo, @@ -252,9 +253,6 @@ struct hda_gen_spec { struct hda_jack_tbl *tbl); void (*mic_autoswitch_hook)(struct hda_codec *codec, struct hda_jack_tbl *tbl); - - /* capture switch hook (for mic-mute LED) */ - void (*capture_switch_hook)(struct hda_codec *codec, bool enable); }; int snd_hda_gen_spec_init(struct hda_gen_spec *spec); diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 71a8894438ab..70b0e323f5af 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -670,7 +670,8 @@ static void alc_inv_dmic_sync(struct hda_codec *codec, bool force) } } -static void alc_inv_dmic_hook(struct hda_codec *codec) +static void alc_inv_dmic_hook(struct hda_codec *codec, + struct snd_ctl_elem_value *ucontrol) { alc_inv_dmic_sync(codec, false); } diff --git a/sound/pci/hda/patch_sigmatel.c b/sound/pci/hda/patch_sigmatel.c index 9d2dfad7f0bc..456ebc771313 100644 --- a/sound/pci/hda/patch_sigmatel.c +++ b/sound/pci/hda/patch_sigmatel.c @@ -316,11 +316,17 @@ static void stac_gpio_set(struct hda_codec *codec, unsigned int mask, } /* hook for controlling mic-mute LED GPIO */ -static void stac_capture_led_hook(struct hda_codec *codec, bool enable) +static void stac_capture_led_hook(struct hda_codec *codec, + struct snd_ctl_elem_value *ucontrol) { struct sigmatel_spec *spec = codec->spec; - bool mute = !enable; + bool mute; + if (!ucontrol) + return; + + mute = !(ucontrol->value.integer.value[0] || + ucontrol->value.integer.value[1]); if (spec->mic_mute_led_on != mute) { spec->mic_mute_led_on = mute; if (mute) @@ -3806,7 +3812,7 @@ static void stac_setup_gpio(struct hda_codec *codec) spec->mic_mute_led_on = true; spec->gpio_data |= spec->mic_mute_led_gpio; - spec->gen.capture_switch_hook = stac_capture_led_hook; + spec->gen.cap_sync_hook = stac_capture_led_hook; } } -- 2.30.2