| Message ID | 20210220122438.21857-1-hdegoede@redhat.com |
|---|---|
| Headers | show |
| Series | HID: lenovo: Mute LED handling fixes and improvements | expand |
On Sat, 20 Feb 2021 17:34:12 +0100 Hans de Goede <hdegoede@redhat.com> wrote: > Hi, > > On 2/20/21 4:16 PM, Marek Behun wrote: > > On Sat, 20 Feb 2021 13:24:36 +0100 > > Hans de Goede <hdegoede@redhat.com> wrote: > > > >> + data->led_mute.max_brightness = LED_ON; > > > > These constants are obsolete since we now support variable maximum > > brightness. Just put 1 there. > > Ok, any other remarks wrt the LED bits before I send out a v3 ? I will review the remaining patches tonight.
Reviewed-by: Marek Behún <kabel@kernel.org>
Hi All, Here is v2 of my series with mute LED handling fixes and improvements for the hid-lenovo driver. This time I've added the LED folks to the Cc in case they have any input, but there is nothing controversial in here wrt use of the LED API. The following patches were changed or are new in version 2 of the series, see the individual patches for detaisl: [PATCH v2 2/7] HID: lenovo: Fix lenovo_led_set_tp10ubkbd() error handling [PATCH v2 4/7] HID: lenovo: Remove lenovo_led_brightness_get() [PATCH v2 5/7] HID: lenovo: Set LEDs max_brightness value Regards, Hans Hans de Goede (7): HID: lenovo: Use brightness_set_blocking callback for setting LEDs brightness HID: lenovo: Fix lenovo_led_set_tp10ubkbd() error handling HID: lenovo: Check hid_get_drvdata() returns non NULL in lenovo_event() HID: lenovo: Remove lenovo_led_brightness_get() HID: lenovo: Set LEDs max_brightness value HID: lenovo: Map mic-mute button to KEY_F20 instead of KEY_MICMUTE HID: lenovo: Set default_trigger-s for the mute and micmute LEDs drivers/hid/hid-lenovo.c | 61 ++++++++++++++++++++-------------------- 1 file changed, 31 insertions(+), 30 deletions(-)