Commit 7cdf8c49 authored by Takashi Iwai's avatar Takashi Iwai

ALSA: hda: generic: Add a helper for mic-mute LED with LED classdev

A new helper, snd_hda_gen_add_micmute_led_cdev(), is introduced here
for creating a LED classdev and setting up the hook to the capture
control for controlling the mic-mute LED to follow the capture switch
change.  This will replace the existing users of
snd_hda_gen_add_micmute_led() in later patches.

Also, introduce a new kconfig CONFIG_SND_HDA_GENERIC_LEDS, to indicate
the usage of mute / mic-mute LED helpers.  It's selected by the codec
drivers (Realtek, Conexant and Sigmatel), while it selects the
necessary LED class dependencies.
Tested-by: default avatarKai-Heng Feng <kai.heng.feng@canonical.com>
Link: https://lore.kernel.org/r/20200618110842.27238-3-tiwai@suse.deSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent bf61c42a
...@@ -8,6 +8,9 @@ config SND_HDA ...@@ -8,6 +8,9 @@ config SND_HDA
select SND_JACK select SND_JACK
select SND_HDA_CORE select SND_HDA_CORE
config SND_HDA_GENERIC_LEDS
bool
config SND_HDA_INTEL config SND_HDA_INTEL
tristate "HD Audio PCI" tristate "HD Audio PCI"
depends on SND_PCI depends on SND_PCI
...@@ -91,6 +94,7 @@ config SND_HDA_PATCH_LOADER ...@@ -91,6 +94,7 @@ config SND_HDA_PATCH_LOADER
config SND_HDA_CODEC_REALTEK config SND_HDA_CODEC_REALTEK
tristate "Build Realtek HD-audio codec support" tristate "Build Realtek HD-audio codec support"
select SND_HDA_GENERIC select SND_HDA_GENERIC
select SND_HDA_GENERIC_LEDS
help help
Say Y or M here to include Realtek HD-audio codec support in Say Y or M here to include Realtek HD-audio codec support in
snd-hda-intel driver, such as ALC880. snd-hda-intel driver, such as ALC880.
...@@ -111,6 +115,7 @@ comment "Set to Y if you want auto-loading the codec driver" ...@@ -111,6 +115,7 @@ comment "Set to Y if you want auto-loading the codec driver"
config SND_HDA_CODEC_SIGMATEL config SND_HDA_CODEC_SIGMATEL
tristate "Build IDT/Sigmatel HD-audio codec support" tristate "Build IDT/Sigmatel HD-audio codec support"
select SND_HDA_GENERIC select SND_HDA_GENERIC
select SND_HDA_GENERIC_LEDS
help help
Say Y or M here to include IDT (Sigmatel) HD-audio codec support in Say Y or M here to include IDT (Sigmatel) HD-audio codec support in
snd-hda-intel driver, such as STAC9200. snd-hda-intel driver, such as STAC9200.
...@@ -155,6 +160,7 @@ comment "Set to Y if you want auto-loading the codec driver" ...@@ -155,6 +160,7 @@ comment "Set to Y if you want auto-loading the codec driver"
config SND_HDA_CODEC_CONEXANT config SND_HDA_CODEC_CONEXANT
tristate "Build Conexant HD-audio codec support" tristate "Build Conexant HD-audio codec support"
select SND_HDA_GENERIC select SND_HDA_GENERIC
select SND_HDA_GENERIC_LEDS
help help
Say Y or M here to include Conexant HD-audio codec support in Say Y or M here to include Conexant HD-audio codec support in
snd-hda-intel driver, such as CX20549. snd-hda-intel driver, such as CX20549.
...@@ -215,6 +221,9 @@ comment "Set to Y if you want auto-loading the codec driver" ...@@ -215,6 +221,9 @@ comment "Set to Y if you want auto-loading the codec driver"
config SND_HDA_GENERIC config SND_HDA_GENERIC
tristate "Enable generic HD-audio codec parser" tristate "Enable generic HD-audio codec parser"
select LEDS_CLASS if SND_HDA_GENERIC_LEDS
select LEDS_TRIGGERS if SND_HDA_GENERIC_LEDS
select LEDS_TRIGGER_AUDIO if SND_HDA_GENERIC_LEDS
help help
Say Y or M here to enable the generic HD-audio codec parser Say Y or M here to enable the generic HD-audio codec parser
in snd-hda-intel driver. in snd-hda-intel driver.
......
...@@ -3887,6 +3887,7 @@ static int parse_mic_boost(struct hda_codec *codec) ...@@ -3887,6 +3887,7 @@ static int parse_mic_boost(struct hda_codec *codec)
return 0; return 0;
} }
#ifdef CONFIG_SND_HDA_GENERIC_LEDS
/* /*
* mic mute LED hook helpers * mic mute LED hook helpers
*/ */
...@@ -3923,10 +3924,8 @@ static void call_micmute_led_update(struct hda_codec *codec) ...@@ -3923,10 +3924,8 @@ static void call_micmute_led_update(struct hda_codec *codec)
spec->micmute_led.led_value = val; spec->micmute_led.led_value = val;
if (spec->micmute_led.update) if (spec->micmute_led.update)
spec->micmute_led.update(codec); spec->micmute_led.update(codec);
#if IS_REACHABLE(CONFIG_LEDS_TRIGGER_AUDIO)
ledtrig_audio_set(LED_AUDIO_MICMUTE, ledtrig_audio_set(LED_AUDIO_MICMUTE,
spec->micmute_led.led_value ? LED_ON : LED_OFF); spec->micmute_led.led_value ? LED_ON : LED_OFF);
#endif
} }
static void update_micmute_led(struct hda_codec *codec, static void update_micmute_led(struct hda_codec *codec,
...@@ -4050,6 +4049,50 @@ void snd_hda_gen_fixup_micmute_led(struct hda_codec *codec, ...@@ -4050,6 +4049,50 @@ void snd_hda_gen_fixup_micmute_led(struct hda_codec *codec,
} }
EXPORT_SYMBOL_GPL(snd_hda_gen_fixup_micmute_led); EXPORT_SYMBOL_GPL(snd_hda_gen_fixup_micmute_led);
/**
* snd_dha_gen_add_micmute_led_cdev - Create a LED classdev and enable as mic-mute LED
* @codec: the HDA codec
* @callback: the callback for LED classdev brightness_set_blocking
*
* Called from the codec drivers for offering the mic mute LED controls.
* This creates a LED classdev and sets up the cap_sync_hook that is called at
* each time when the capture mixer switch changes.
*
* When NULL is passed to @callback, no classdev is created but only the
* LED-trigger is set up.
*
* Returns 0 or a negative error.
*/
int snd_hda_gen_add_micmute_led_cdev(struct hda_codec *codec,
int (*callback)(struct led_classdev *,
enum led_brightness))
{
int err;
struct led_classdev *cdev;
if (callback) {
cdev = devm_kzalloc(&codec->core.dev, sizeof(*cdev), GFP_KERNEL);
if (!cdev)
return -ENOMEM;
cdev->name = "hda::micmute";
cdev->max_brightness = 1;
cdev->default_trigger = "audio-micmute";
cdev->brightness_set_blocking = callback;
cdev->brightness = ledtrig_audio_get(LED_AUDIO_MICMUTE);
err = devm_led_classdev_register(&codec->core.dev, cdev);
if (err) {
codec_warn(codec, "failed to create a mic-mute LED cdev\n");
return err;
}
}
return snd_hda_gen_add_micmute_led(codec, NULL);
}
EXPORT_SYMBOL_GPL(snd_hda_gen_add_micmute_led_cdev);
#endif /* CONFIG_SND_HDA_GENERIC_LEDS */
/* /*
* parse digital I/Os and set up NIDs in BIOS auto-parse mode * parse digital I/Os and set up NIDs in BIOS auto-parse mode
*/ */
......
...@@ -8,6 +8,8 @@ ...@@ -8,6 +8,8 @@
#ifndef __SOUND_HDA_GENERIC_H #ifndef __SOUND_HDA_GENERIC_H
#define __SOUND_HDA_GENERIC_H #define __SOUND_HDA_GENERIC_H
#include <linux/leds.h>
/* table entry for multi-io paths */ /* table entry for multi-io paths */
struct hda_multi_io { struct hda_multi_io {
hda_nid_t pin; /* multi-io widget pin NID */ hda_nid_t pin; /* multi-io widget pin NID */
...@@ -357,5 +359,8 @@ int snd_hda_gen_add_micmute_led(struct hda_codec *codec, ...@@ -357,5 +359,8 @@ int snd_hda_gen_add_micmute_led(struct hda_codec *codec,
void (*hook)(struct hda_codec *)); void (*hook)(struct hda_codec *));
void snd_hda_gen_fixup_micmute_led(struct hda_codec *codec, void snd_hda_gen_fixup_micmute_led(struct hda_codec *codec,
const struct hda_fixup *fix, int action); const struct hda_fixup *fix, int action);
int snd_hda_gen_add_micmute_led_cdev(struct hda_codec *codec,
int (*callback)(struct led_classdev *,
enum led_brightness));
#endif /* __SOUND_HDA_GENERIC_H */ #endif /* __SOUND_HDA_GENERIC_H */
...@@ -4100,16 +4100,6 @@ static void alc_fixup_gpio_mute_hook(void *private_data, int enabled) ...@@ -4100,16 +4100,6 @@ static void alc_fixup_gpio_mute_hook(void *private_data, int enabled)
} }
/* turn on/off mic-mute LED via GPIO per capture hook */ /* turn on/off mic-mute LED via GPIO per capture hook */
static void alc_gpio_micmute_update(struct hda_codec *codec)
{
struct alc_spec *spec = codec->spec;
alc_update_gpio_led(codec, spec->gpio_mic_led_mask,
spec->micmute_led_polarity,
spec->gen.micmute_led.led_value);
}
#if IS_REACHABLE(CONFIG_LEDS_TRIGGER_AUDIO)
static int micmute_led_set(struct led_classdev *led_cdev, static int micmute_led_set(struct led_classdev *led_cdev,
enum led_brightness brightness) enum led_brightness brightness)
{ {
...@@ -4121,14 +4111,6 @@ static int micmute_led_set(struct led_classdev *led_cdev, ...@@ -4121,14 +4111,6 @@ static int micmute_led_set(struct led_classdev *led_cdev,
return 0; return 0;
} }
static struct led_classdev micmute_led_cdev = {
.name = "hda::micmute",
.max_brightness = 1,
.brightness_set_blocking = micmute_led_set,
.default_trigger = "audio-micmute",
};
#endif
/* setup mute and mic-mute GPIO bits, add hooks appropriately */ /* setup mute and mic-mute GPIO bits, add hooks appropriately */
static void alc_fixup_hp_gpio_led(struct hda_codec *codec, static void alc_fixup_hp_gpio_led(struct hda_codec *codec,
int action, int action,
...@@ -4136,9 +4118,6 @@ static void alc_fixup_hp_gpio_led(struct hda_codec *codec, ...@@ -4136,9 +4118,6 @@ static void alc_fixup_hp_gpio_led(struct hda_codec *codec,
unsigned int micmute_mask) unsigned int micmute_mask)
{ {
struct alc_spec *spec = codec->spec; struct alc_spec *spec = codec->spec;
#if IS_REACHABLE(CONFIG_LEDS_TRIGGER_AUDIO)
int err;
#endif
alc_fixup_gpio(codec, action, mute_mask | micmute_mask); alc_fixup_gpio(codec, action, mute_mask | micmute_mask);
...@@ -4150,14 +4129,7 @@ static void alc_fixup_hp_gpio_led(struct hda_codec *codec, ...@@ -4150,14 +4129,7 @@ static void alc_fixup_hp_gpio_led(struct hda_codec *codec,
} }
if (micmute_mask) { if (micmute_mask) {
spec->gpio_mic_led_mask = micmute_mask; spec->gpio_mic_led_mask = micmute_mask;
snd_hda_gen_add_micmute_led(codec, alc_gpio_micmute_update); snd_hda_gen_add_micmute_led_cdev(codec, micmute_led_set);
#if IS_REACHABLE(CONFIG_LEDS_TRIGGER_AUDIO)
micmute_led_cdev.brightness = ledtrig_audio_get(LED_AUDIO_MICMUTE);
err = devm_led_classdev_register(&codec->core.dev, &micmute_led_cdev);
if (err)
codec_warn(codec, "failed to register micmute LED\n");
#endif
} }
} }
......
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