mbox series

[v2,resend,0/9] HID: lenovo: LED fixes and Thinkpad X1 Tablet kbd support

Message ID 20210404080432.4322-1-hdegoede@redhat.com (mailing list archive)
Headers show
Series HID: lenovo: LED fixes and Thinkpad X1 Tablet kbd support | expand

Message

Hans de Goede April 4, 2021, 8:04 a.m. UTC 
Hi Jiri,

This seems to have fallen through the cracks, again (IIRC this is the
second resend) can you please pick this series up?

Patches 1 - 7 have:
Reviewed-by: Marek Behún <kabel@kernel.org>

Patches 1, 4, 5, 7 have:
Acked-by: Pavel Machek <pavel@ucw.cz>

With the ack from Pavel (the LED subsys maintainer) indicating that
there are no problems with the LED classdev API usage in these patches.

Patches 8 - 9 are new in v2 of this series, these add support for the
Thinkpad X1 Tablet Thin keyboard. They have been tested by me on a
Thinkpad 10 ultra keyboard dock (so that that does not regress) and
by Alexander Kobel on a Thinkpad X1 Tablet Thin keyboard.

Regards,

Hans


Hans de Goede (9):
  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_triggers for the mute and micmute LEDs
  HID: lenovo: Rework how the tp10ubkbd code decides which USB interface
    to use
  HID: lenovo: Add support for Thinkpad X1 Tablet Thin keyboard

 drivers/hid/hid-lenovo.c | 147 ++++++++++++++++++++++++++++++---------
 1 file changed, 114 insertions(+), 33 deletions(-)

Comments

Jiri Kosina April 7, 2021, 12:56 p.m. UTC | #1 
On Sun, 4 Apr 2021, Hans de Goede wrote:

> Hi Jiri,
> 
> This seems to have fallen through the cracks, again (IIRC this is the
> second resend) can you please pick this series up?
> 
> Patches 1 - 7 have:
> Reviewed-by: Marek Behún <kabel@kernel.org>
> 
> Patches 1, 4, 5, 7 have:
> Acked-by: Pavel Machek <pavel@ucw.cz>
> 
> With the ack from Pavel (the LED subsys maintainer) indicating that
> there are no problems with the LED classdev API usage in these patches.
> 
> Patches 8 - 9 are new in v2 of this series, these add support for the
> Thinkpad X1 Tablet Thin keyboard. They have been tested by me on a
> Thinkpad 10 ultra keyboard dock (so that that does not regress) and
> by Alexander Kobel on a Thinkpad X1 Tablet Thin keyboard.

Now in for-5.13/lenovo, sorry for the delay and thanks.
Hans de Goede April 7, 2021, 1:42 p.m. UTC | #2 
Hi,

On 4/7/21 2:56 PM, Jiri Kosina wrote:
> On Sun, 4 Apr 2021, Hans de Goede wrote:
> 
>> Hi Jiri,
>>
>> This seems to have fallen through the cracks, again (IIRC this is the
>> second resend) can you please pick this series up?
>>
>> Patches 1 - 7 have:
>> Reviewed-by: Marek Behún <kabel@kernel.org>
>>
>> Patches 1, 4, 5, 7 have:
>> Acked-by: Pavel Machek <pavel@ucw.cz>
>>
>> With the ack from Pavel (the LED subsys maintainer) indicating that
>> there are no problems with the LED classdev API usage in these patches.
>>
>> Patches 8 - 9 are new in v2 of this series, these add support for the
>> Thinkpad X1 Tablet Thin keyboard. They have been tested by me on a
>> Thinkpad 10 ultra keyboard dock (so that that does not regress) and
>> by Alexander Kobel on a Thinkpad X1 Tablet Thin keyboard.
> 
> Now in for-5.13/lenovo,

Great, thank you.

> sorry for the delay

No problem.

Regards,

Hans