* [PATCH v2] HID: input: map System Microphone Mute LED [not found] <20260904022903.607880-1-ait@ait.place> @ 2026-09-04 4:47 ` Jonathan Jansen 2026-09-11 15:58 ` Jiri Kosina 0 siblings, 1 reply; 3+ messages in thread From: Jonathan Jansen @ 2026-09-04 4:47 UTC (permalink / raw) To: linux-input Cc: linux-kernel, jikos, bentiss, dmitry.torokhov, linux-api, jonathanjansen HUTRR110 defines LED page usage 0x57 as System Microphone Mute. The usage is currently ignored, so the indicator stays dark and userspace falls back to per-vendor code in hid-lenovo, hid-playstation, hid-plantronics. Map it to a new LED_MICMUTE. It appears as /sys/class/leds/inputN::micmute and uses the "audio-micmute" trigger from snd-ctl-led, so the LED follows the system-wide mic mute state the same way LED_MUTE (page 0x09) already does for speaker mute. LED_MICMUTE takes the next free code (0x0b). Link: https://www.usb.org/sites/default/files/hutrr110-systemmicrophonemute.pdf Signed-off-by: Jonathan Jansen <jonathanjansen@ait.place> --- Notes: Changes since v1: rewrote the commit message; no functional changes. drivers/hid/hid-debug.c | 2 +- drivers/hid/hid-input.c | 1 + drivers/input/input-leds.c | 1 + include/uapi/linux/input-event-codes.h | 1 + 4 files changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/hid/hid-debug.c b/drivers/hid/hid-debug.c index f44e6e708..92a950105 100644 --- a/drivers/hid/hid-debug.c +++ b/drivers/hid/hid-debug.c @@ -3550,7 +3550,7 @@ static const char *leds[LED_MAX + 1] = { [LED_KANA] = "Kana", [LED_SLEEP] = "Sleep", [LED_SUSPEND] = "Suspend", [LED_MUTE] = "Mute", [LED_MISC] = "Misc", [LED_MAIL] = "Mail", - [LED_CHARGING] = "Charging", + [LED_CHARGING] = "Charging", [LED_MICMUTE] = "Mic Mute", }; static const char *repeats[REP_MAX + 1] = { diff --git a/drivers/hid/hid-input.c b/drivers/hid/hid-input.c index edd6bbf83..727374dd5 100644 --- a/drivers/hid/hid-input.c +++ b/drivers/hid/hid-input.c @@ -990,6 +990,7 @@ static void hidinput_configure_usage(struct hid_input *hidinput, struct hid_fiel case 0x27: map_led (LED_SLEEP); break; /* "Stand-By" */ case 0x4c: map_led (LED_SUSPEND); break; /* "System Suspend" */ case 0x09: map_led (LED_MUTE); break; /* "Mute" */ + case 0x57: map_led (LED_MICMUTE); break; /* "System Microphone Mute" */ case 0x4b: map_led (LED_MISC); break; /* "Generic Indicator" */ case 0x19: map_led (LED_MAIL); break; /* "Message Waiting" */ case 0x4d: map_led (LED_CHARGING); break; /* "External Power Connected" */ diff --git a/drivers/input/input-leds.c b/drivers/input/input-leds.c index b08d1d08d..6522079a4 100644 --- a/drivers/input/input-leds.c +++ b/drivers/input/input-leds.c @@ -36,6 +36,7 @@ static const struct { [LED_SLEEP] = { "sleep" } , [LED_SUSPEND] = { "suspend" }, [LED_MUTE] = { "mute", AUDIO_TRIGGER("audio-mute") }, + [LED_MICMUTE] = { "micmute", AUDIO_TRIGGER("audio-micmute") }, [LED_MISC] = { "misc" }, [LED_MAIL] = { "mail" }, [LED_CHARGING] = { "charging" }, diff --git a/include/uapi/linux/input-event-codes.h b/include/uapi/linux/input-event-codes.h index 3528168f7..bc374821c 100644 --- a/include/uapi/linux/input-event-codes.h +++ b/include/uapi/linux/input-event-codes.h @@ -983,6 +983,7 @@ #define LED_MISC 0x08 #define LED_MAIL 0x09 #define LED_CHARGING 0x0a +#define LED_MICMUTE 0x0b #define LED_MAX 0x0f #define LED_CNT (LED_MAX+1) -- 2.55.0 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] HID: input: map System Microphone Mute LED 2026-09-04 4:47 ` [PATCH v2] HID: input: map System Microphone Mute LED Jonathan Jansen @ 2026-09-11 15:58 ` Jiri Kosina 2026-09-11 16:36 ` Dmitry Torokhov 0 siblings, 1 reply; 3+ messages in thread From: Jiri Kosina @ 2026-09-11 15:58 UTC (permalink / raw) To: Jonathan Jansen Cc: linux-input, linux-kernel, bentiss, dmitry.torokhov, linux-api On Fri, 4 Sep 2026, Jonathan Jansen wrote: > HUTRR110 defines LED page usage 0x57 as System Microphone Mute. > > The usage is currently ignored, so the indicator stays dark and > userspace falls back to per-vendor code in hid-lenovo, > hid-playstation, hid-plantronics. > > Map it to a new LED_MICMUTE. It appears as > /sys/class/leds/inputN::micmute and uses the "audio-micmute" > trigger from snd-ctl-led, so the LED follows the system-wide > mic mute state the same way LED_MUTE (page 0x09) already does > for speaker mute. > > LED_MICMUTE takes the next free code (0x0b). > > Link: https://www.usb.org/sites/default/files/hutrr110-systemmicrophonemute.pdf > > Signed-off-by: Jonathan Jansen <jonathanjansen@ait.place> > --- > > Notes: > Changes since v1: rewrote the commit message; no functional changes. > > drivers/hid/hid-debug.c | 2 +- > drivers/hid/hid-input.c | 1 + Acked-by: Jiri Kosina <jkosina@suse.com> for the drivers/hid changes. Dmitry, will you take this through your tree, please? Thanks, -- Jiri Kosina SUSE Labs ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] HID: input: map System Microphone Mute LED 2026-09-11 15:58 ` Jiri Kosina @ 2026-09-11 16:36 ` Dmitry Torokhov 0 siblings, 0 replies; 3+ messages in thread From: Dmitry Torokhov @ 2026-09-11 16:36 UTC (permalink / raw) To: Jiri Kosina Cc: Jonathan Jansen, linux-input, linux-kernel, bentiss, linux-api On Fri, Sep 11, 2026 at 05:58:24PM +0200, Jiri Kosina wrote: > On Fri, 4 Sep 2026, Jonathan Jansen wrote: > > > HUTRR110 defines LED page usage 0x57 as System Microphone Mute. > > > > The usage is currently ignored, so the indicator stays dark and > > userspace falls back to per-vendor code in hid-lenovo, > > hid-playstation, hid-plantronics. > > > > Map it to a new LED_MICMUTE. It appears as > > /sys/class/leds/inputN::micmute and uses the "audio-micmute" > > trigger from snd-ctl-led, so the LED follows the system-wide > > mic mute state the same way LED_MUTE (page 0x09) already does > > for speaker mute. > > > > LED_MICMUTE takes the next free code (0x0b). > > > > Link: https://www.usb.org/sites/default/files/hutrr110-systemmicrophonemute.pdf > > > > Signed-off-by: Jonathan Jansen <jonathanjansen@ait.place> > > --- > > > > Notes: > > Changes since v1: rewrote the commit message; no functional changes. > > > > drivers/hid/hid-debug.c | 2 +- > > drivers/hid/hid-input.c | 1 + > > Acked-by: Jiri Kosina <jkosina@suse.com> > > for the drivers/hid changes. > > Dmitry, will you take this through your tree, please? Hi Jiri, Jonathan, Sorry but we should not add any new LED definitions to input since we have proper LED subsystem now. The existing LED definitions are strictly legacy now and input core has a bridge to LEDs (in input-leds.c). I see we had similar submissions in '23 and '24 at the very least. I should add a comment to that effect to input-event-codes.h. Thanks. -- Dmitry ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-09-11 16:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20260904022903.607880-1-ait@ait.place>
2026-09-04 4:47 ` [PATCH v2] HID: input: map System Microphone Mute LED Jonathan Jansen
2026-09-11 15:58 ` Jiri Kosina
2026-09-11 16:36 ` Dmitry Torokhov
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®