* [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
@ 2020-07-17 11:41 Hans de Goede
2020-07-17 11:41 ` [PATCH 1/3] Input: allocate keycodes for notification-center, pickup-phone and hangup-phone Hans de Goede
` (3 more replies)
0 siblings, 4 replies; 13+ messages in thread
From: Hans de Goede @ 2020-07-17 11:41 UTC (permalink / raw)
To: Dmitry Torokhov, Darren Hart, Andy Shevchenko,
Henrique de Moraes Holschuh
Cc: Hans de Goede, linux-input, Marco Trevisan, Mark Pearson,
Christian Kellner, Benjamin Berg, ibm-acpi-devel,
platform-driver-x86, linux-kernel
Hi All,
This is a simple patch-series adding support for 3 new hotkeys found
on various new Lenovo Thinkpad models.
There is one small complication, these 3 new hotkeys also require
the addition of new key-codes to include/uapi/linux/input-event-codes.h.
And those changes have to land first. It is probably easiest to just
merge everything through the platform/x86 tree, or alternatively we
would need an immutable branch from the input-subsys with the first
patch on there which the platform/x86 maintainers can then merge.
Dmitry, can you please let us know how you want to proceed with these
changes; And if you are ok with merging the input-event-codes.h changes
through the platform/x86 tree, may we have your Acked-by for that ?
Regards,
Hans
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 1/3] Input: allocate keycodes for notification-center, pickup-phone and hangup-phone
2020-07-17 11:41 [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Hans de Goede
@ 2020-07-17 11:41 ` Hans de Goede
2020-07-17 11:41 ` [PATCH 2/3] platform/x86: thinkpad_acpi: Add support for new hotkeys found on X1C8 / T14 Hans de Goede
` (2 subsequent siblings)
3 siblings, 0 replies; 13+ messages in thread
From: Hans de Goede @ 2020-07-17 11:41 UTC (permalink / raw)
To: Dmitry Torokhov, Darren Hart, Andy Shevchenko,
Henrique de Moraes Holschuh
Cc: Hans de Goede, linux-input, Marco Trevisan, Mark Pearson,
Christian Kellner, Benjamin Berg, ibm-acpi-devel,
platform-driver-x86, linux-kernel
New Lenovo Thinkpad models, e.g. the X1 Carbon 8th gen and the new T14 gen1
models have 3 new symbols / shortcuts on their F9-F11 keys (and the
thinkpad_acpi driver receives 3 new "scancodes" for these):
F9: Has a symbol resembling a rectangular speech baloon, the manual says
the hotkey functions shows or hides the notification center
F10: Has a symbol of a telephone horn which has been picked up from the
receiver, the manual says: "Answer incoming calls"
F11: Has a symbol of a telephone horn which is resting on the receiver,
the manual says: "Decline incoming calls"
We have no existing keycodes which are a good match for these, so
add 3 new keycodes for these.
I noticed that we have a hole in our keycodes between 0x1ba and 0x1c0
which does not seem to be reserved for any specific purpose, so these
new 3 codes use 0x1bc - 0x1be, instead of starting at 0x27b.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
include/uapi/linux/input-event-codes.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/include/uapi/linux/input-event-codes.h b/include/uapi/linux/input-event-codes.h
index b6a835d37826..8d605a6df502 100644
--- a/include/uapi/linux/input-event-codes.h
+++ b/include/uapi/linux/input-event-codes.h
@@ -515,6 +515,9 @@
#define KEY_10CHANNELSUP 0x1b8 /* 10 channels up (10+) */
#define KEY_10CHANNELSDOWN 0x1b9 /* 10 channels down (10-) */
#define KEY_IMAGES 0x1ba /* AL Image Browser */
+#define KEY_NOTIFICATION_CENTER 0x1bc /* Show/hide the notification center */
+#define KEY_PICKUP_PHONE 0x1bd /* Answer incoming call */
+#define KEY_HANGUP_PHONE 0x1be /* Decline incoming call */
#define KEY_DEL_EOL 0x1c0
#define KEY_DEL_EOS 0x1c1
--
2.26.2
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 2/3] platform/x86: thinkpad_acpi: Add support for new hotkeys found on X1C8 / T14
2020-07-17 11:41 [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Hans de Goede
2020-07-17 11:41 ` [PATCH 1/3] Input: allocate keycodes for notification-center, pickup-phone and hangup-phone Hans de Goede
@ 2020-07-17 11:41 ` Hans de Goede
2020-07-17 11:41 ` [PATCH 3/3] platform/x86: thinkpad_acpi: Map Clipping tool hotkey to KEY_SELECTIVE_SCREENSHOT Hans de Goede
2020-07-19 22:56 ` [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Henrique de Moraes Holschuh
3 siblings, 0 replies; 13+ messages in thread
From: Hans de Goede @ 2020-07-17 11:41 UTC (permalink / raw)
To: Dmitry Torokhov, Darren Hart, Andy Shevchenko,
Henrique de Moraes Holschuh
Cc: Hans de Goede, linux-input, Marco Trevisan, Mark Pearson,
Christian Kellner, Benjamin Berg, ibm-acpi-devel,
platform-driver-x86, linux-kernel
New Lenovo Thinkpad models, e.g. the X1 Carbon 8th gen and the new T14 gen1
models have 3 new symbols / shortcuts on their F9-F11 keys (and the
thinkpad_acpi driver receives 3 new hkey events for these):
F9: Has a symbol resembling a rectangular speech baloon, the manual says
the hotkey functions shows or hides the notification center
F10: Has a symbol of a telephone horn which has been picked up from the
receiver, the manual says: "Answer incoming calls"
F11: Has a symbol of a telephone horn which is resting on the receiver,
the manual says: "Decline incoming calls"
This commit adds support for these 3 new hotkeys.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/platform/x86/thinkpad_acpi.c | 18 +++++++++++++-----
1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
index 7dd84cb4fcef..7fc44b6f8370 100644
--- a/drivers/platform/x86/thinkpad_acpi.c
+++ b/drivers/platform/x86/thinkpad_acpi.c
@@ -1915,6 +1915,10 @@ enum { /* hot key scan codes (derived from ACPI DSDT) */
TP_ACPI_HOTKEYSCAN_CALCULATOR,
TP_ACPI_HOTKEYSCAN_BLUETOOTH,
TP_ACPI_HOTKEYSCAN_KEYBOARD,
+ TP_ACPI_HOTKEYSCAN_1316, /* 0x1316 scancode is unknown */
+ TP_ACPI_HOTKEYSCAN_NOTIFICATION_CENTER,
+ TP_ACPI_HOTKEYSCAN_PICKUP_PHONE,
+ TP_ACPI_HOTKEYSCAN_HANGUP_PHONE,
/* Hotkey keymap size */
TPACPI_HOTKEY_MAP_LEN
@@ -3431,11 +3435,15 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
KEY_UNKNOWN, KEY_UNKNOWN, KEY_UNKNOWN, KEY_UNKNOWN,
KEY_UNKNOWN,
- KEY_BOOKMARKS, /* Favorite app, 0x311 */
- KEY_RESERVED, /* Clipping tool */
- KEY_CALC, /* Calculator (above numpad, P52) */
- KEY_BLUETOOTH, /* Bluetooth */
- KEY_KEYBOARD /* Keyboard, 0x315 */
+ KEY_BOOKMARKS, /* Favorite app, 0x311 */
+ KEY_RESERVED, /* Clipping tool */
+ KEY_CALC, /* Calculator (above numpad, P52) */
+ KEY_BLUETOOTH, /* Bluetooth */
+ KEY_KEYBOARD, /* Keyboard, 0x315 */
+ KEY_UNKNOWN, /* 0x316, unknown */
+ KEY_NOTIFICATION_CENTER, /* Notification Center */
+ KEY_PICKUP_PHONE, /* Answer incoming call */
+ KEY_HANGUP_PHONE, /* Decline incoming call */
},
};
--
2.26.2
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 3/3] platform/x86: thinkpad_acpi: Map Clipping tool hotkey to KEY_SELECTIVE_SCREENSHOT
2020-07-17 11:41 [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Hans de Goede
2020-07-17 11:41 ` [PATCH 1/3] Input: allocate keycodes for notification-center, pickup-phone and hangup-phone Hans de Goede
2020-07-17 11:41 ` [PATCH 2/3] platform/x86: thinkpad_acpi: Add support for new hotkeys found on X1C8 / T14 Hans de Goede
@ 2020-07-17 11:41 ` Hans de Goede
2020-07-17 12:21 ` Bastien Nocera
2020-07-19 22:56 ` [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Henrique de Moraes Holschuh
3 siblings, 1 reply; 13+ messages in thread
From: Hans de Goede @ 2020-07-17 11:41 UTC (permalink / raw)
To: Dmitry Torokhov, Darren Hart, Andy Shevchenko,
Henrique de Moraes Holschuh
Cc: Hans de Goede, linux-input, Marco Trevisan, Mark Pearson,
Christian Kellner, Benjamin Berg, ibm-acpi-devel,
platform-driver-x86, linux-kernel
Commit 696c6523ec8f ("platform/x86: thinkpad_acpi: add mapping for new
hotkeys") added support for a bunch of new hotkeys, but the
clipping/snipping tool hotkey got ignored because there was no good
key-code to map it to.
Recently a new KEY_SELECTIVE_SCREENSHOT keycode was added by commit
3b059da9835c ("Input: allocate keycode for "Selective Screenshot" key")
quoting from the commit message:
"New Chrome OS keyboards have a "snip" key that is basically a selective
screenshot (allows a user to select an area of screen to be copied).
Allocate a keycode for it."
Support for this "snip" key seems like it is also a good match for the
clipping/snipping tool hotkey, so map this hotkey to the new
KEY_SELECTIVE_SCREENSHOT key-code.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/platform/x86/thinkpad_acpi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
index 7fc44b6f8370..70d533b0c907 100644
--- a/drivers/platform/x86/thinkpad_acpi.c
+++ b/drivers/platform/x86/thinkpad_acpi.c
@@ -3436,7 +3436,7 @@ static int __init hotkey_init(struct ibm_init_struct *iibm)
KEY_UNKNOWN,
KEY_BOOKMARKS, /* Favorite app, 0x311 */
- KEY_RESERVED, /* Clipping tool */
+ KEY_SELECTIVE_SCREENSHOT, /* Clipping tool */
KEY_CALC, /* Calculator (above numpad, P52) */
KEY_BLUETOOTH, /* Bluetooth */
KEY_KEYBOARD, /* Keyboard, 0x315 */
--
2.26.2
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 3/3] platform/x86: thinkpad_acpi: Map Clipping tool hotkey to KEY_SELECTIVE_SCREENSHOT
2020-07-17 11:41 ` [PATCH 3/3] platform/x86: thinkpad_acpi: Map Clipping tool hotkey to KEY_SELECTIVE_SCREENSHOT Hans de Goede
@ 2020-07-17 12:21 ` Bastien Nocera
0 siblings, 0 replies; 13+ messages in thread
From: Bastien Nocera @ 2020-07-17 12:21 UTC (permalink / raw)
To: Hans de Goede, Dmitry Torokhov, Darren Hart, Andy Shevchenko,
Henrique de Moraes Holschuh
Cc: linux-input, Marco Trevisan, Mark Pearson, Christian Kellner,
Benjamin Berg, ibm-acpi-devel, platform-driver-x86, linux-kernel
On Fri, 2020-07-17 at 13:41 +0200, Hans de Goede wrote:
> Commit 696c6523ec8f ("platform/x86: thinkpad_acpi: add mapping for
> new
> hotkeys") added support for a bunch of new hotkeys, but the
> clipping/snipping tool hotkey got ignored because there was no good
> key-code to map it to.
>
> Recently a new KEY_SELECTIVE_SCREENSHOT keycode was added by commit
> 3b059da9835c ("Input: allocate keycode for "Selective Screenshot"
> key")
> quoting from the commit message:
>
> "New Chrome OS keyboards have a "snip" key that is basically a
> selective
> screenshot (allows a user to select an area of screen to be copied).
> Allocate a keycode for it."
>
> Support for this "snip" key seems like it is also a good match for
> the
> clipping/snipping tool hotkey, so map this hotkey to the new
> KEY_SELECTIVE_SCREENSHOT key-code.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Added 5 years ago for the Carbon X1 2014, and finally getting
a keycode ;)
Reviewed-by: Bastien Nocera <hadess@hadess.net>
> ---
> drivers/platform/x86/thinkpad_acpi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/thinkpad_acpi.c
> b/drivers/platform/x86/thinkpad_acpi.c
> index 7fc44b6f8370..70d533b0c907 100644
> --- a/drivers/platform/x86/thinkpad_acpi.c
> +++ b/drivers/platform/x86/thinkpad_acpi.c
> @@ -3436,7 +3436,7 @@ static int __init hotkey_init(struct
> ibm_init_struct *iibm)
> KEY_UNKNOWN,
>
> KEY_BOOKMARKS, /* Favorite app,
> 0x311 */
> - KEY_RESERVED, /* Clipping tool */
> + KEY_SELECTIVE_SCREENSHOT, /* Clipping tool */
> KEY_CALC, /* Calculator (above numpad,
> P52) */
> KEY_BLUETOOTH, /* Bluetooth */
> KEY_KEYBOARD, /* Keyboard, 0x315 */
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-17 11:41 [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Hans de Goede
` (2 preceding siblings ...)
2020-07-17 11:41 ` [PATCH 3/3] platform/x86: thinkpad_acpi: Map Clipping tool hotkey to KEY_SELECTIVE_SCREENSHOT Hans de Goede
@ 2020-07-19 22:56 ` Henrique de Moraes Holschuh
2020-07-22 5:41 ` Dmitry Torokhov
3 siblings, 1 reply; 13+ messages in thread
From: Henrique de Moraes Holschuh @ 2020-07-19 22:56 UTC (permalink / raw)
To: Hans de Goede
Cc: Dmitry Torokhov, Darren Hart, Andy Shevchenko,
Henrique de Moraes Holschuh, linux-input, Marco Trevisan,
Mark Pearson, Christian Kellner, Benjamin Berg, ibm-acpi-devel,
platform-driver-x86, linux-kernel
On Fri, 17 Jul 2020, Hans de Goede wrote:
> This is a simple patch-series adding support for 3 new hotkeys found
> on various new Lenovo Thinkpad models.
For all three patches, pending an ack for the new keycodes by the input
maintainers:
Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
--
Henrique Holschuh
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-19 22:56 ` [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Henrique de Moraes Holschuh
@ 2020-07-22 5:41 ` Dmitry Torokhov
2020-07-27 0:50 ` Henrique de Moraes Holschuh
2020-08-26 15:12 ` Hans de Goede
0 siblings, 2 replies; 13+ messages in thread
From: Dmitry Torokhov @ 2020-07-22 5:41 UTC (permalink / raw)
To: Henrique de Moraes Holschuh
Cc: Hans de Goede, Darren Hart, Andy Shevchenko,
Henrique de Moraes Holschuh, linux-input, Marco Trevisan,
Mark Pearson, Christian Kellner, Benjamin Berg, ibm-acpi-devel,
platform-driver-x86, linux-kernel
On Sun, Jul 19, 2020 at 07:56:49PM -0300, Henrique de Moraes Holschuh wrote:
> On Fri, 17 Jul 2020, Hans de Goede wrote:
> > This is a simple patch-series adding support for 3 new hotkeys found
> > on various new Lenovo Thinkpad models.
>
> For all three patches, pending an ack for the new keycodes by the input
> maintainers:
>
> Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Do you want me to merge all 3 through input tree?
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-22 5:41 ` Dmitry Torokhov
@ 2020-07-27 0:50 ` Henrique de Moraes Holschuh
2020-07-27 7:45 ` Hans de Goede
2020-08-26 15:12 ` Hans de Goede
1 sibling, 1 reply; 13+ messages in thread
From: Henrique de Moraes Holschuh @ 2020-07-27 0:50 UTC (permalink / raw)
To: Hans de Goede
Cc: Darren Hart, Dmitry Torokhov, Andy Shevchenko, linux-input,
Marco Trevisan, Mark Pearson, Christian Kellner, Benjamin Berg,
ibm-acpi-devel, platform-driver-x86, linux-kernel
On Tue, 21 Jul 2020, Dmitry Torokhov wrote:
> On Sun, Jul 19, 2020 at 07:56:49PM -0300, Henrique de Moraes Holschuh wrote:
> > On Fri, 17 Jul 2020, Hans de Goede wrote:
> > > This is a simple patch-series adding support for 3 new hotkeys found
> > > on various new Lenovo Thinkpad models.
> >
> > For all three patches, pending an ack for the new keycodes by the input
> > maintainers:
> >
> > Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
>
> Do you want me to merge all 3 through input tree?
Hans, Daren, Andy, what do you prefer?
--
Henrique Holschuh
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-27 0:50 ` Henrique de Moraes Holschuh
@ 2020-07-27 7:45 ` Hans de Goede
2020-07-27 7:49 ` Andy Shevchenko
0 siblings, 1 reply; 13+ messages in thread
From: Hans de Goede @ 2020-07-27 7:45 UTC (permalink / raw)
To: Henrique de Moraes Holschuh
Cc: Darren Hart, Dmitry Torokhov, Andy Shevchenko, linux-input,
Marco Trevisan, Mark Pearson, Christian Kellner, Benjamin Berg,
ibm-acpi-devel, platform-driver-x86, linux-kernel
Hi,
On 7/27/20 2:50 AM, Henrique de Moraes Holschuh wrote:
> On Tue, 21 Jul 2020, Dmitry Torokhov wrote:
>> On Sun, Jul 19, 2020 at 07:56:49PM -0300, Henrique de Moraes Holschuh wrote:
>>> On Fri, 17 Jul 2020, Hans de Goede wrote:
>>>> This is a simple patch-series adding support for 3 new hotkeys found
>>>> on various new Lenovo Thinkpad models.
>>>
>>> For all three patches, pending an ack for the new keycodes by the input
>>> maintainers:
>>>
>>> Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
>>
>> Do you want me to merge all 3 through input tree?
>
> Hans, Daren, Andy, what do you prefer?
Taking all this upstream through Dmitry's input tree is fine with
me, but this really is up to Andy and/or Daren.
Regards,
Hans
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-27 7:45 ` Hans de Goede
@ 2020-07-27 7:49 ` Andy Shevchenko
2020-07-27 10:24 ` Andy Shevchenko
0 siblings, 1 reply; 13+ messages in thread
From: Andy Shevchenko @ 2020-07-27 7:49 UTC (permalink / raw)
To: Hans de Goede
Cc: Henrique de Moraes Holschuh, Darren Hart, Dmitry Torokhov,
Andy Shevchenko, linux-input, Marco Trevisan, Mark Pearson,
Christian Kellner, Benjamin Berg, Thinkpad-acpi devel ML,
Platform Driver, Linux Kernel Mailing List
On Mon, Jul 27, 2020 at 10:45 AM Hans de Goede <hdegoede@redhat.com> wrote:
>
> Hi,
>
> On 7/27/20 2:50 AM, Henrique de Moraes Holschuh wrote:
> > On Tue, 21 Jul 2020, Dmitry Torokhov wrote:
> >> On Sun, Jul 19, 2020 at 07:56:49PM -0300, Henrique de Moraes Holschuh wrote:
> >>> On Fri, 17 Jul 2020, Hans de Goede wrote:
> >>>> This is a simple patch-series adding support for 3 new hotkeys found
> >>>> on various new Lenovo Thinkpad models.
> >>>
> >>> For all three patches, pending an ack for the new keycodes by the input
> >>> maintainers:
> >>>
> >>> Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
> >>
> >> Do you want me to merge all 3 through input tree?
> >
> > Hans, Daren, Andy, what do you prefer?
>
> Taking all this upstream through Dmitry's input tree is fine with
> me, but this really is up to Andy and/or Daren.
Fine with me.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-27 7:49 ` Andy Shevchenko
@ 2020-07-27 10:24 ` Andy Shevchenko
2020-08-01 19:43 ` Henrique de Moraes Holschuh
0 siblings, 1 reply; 13+ messages in thread
From: Andy Shevchenko @ 2020-07-27 10:24 UTC (permalink / raw)
To: Hans de Goede
Cc: Henrique de Moraes Holschuh, Darren Hart, Dmitry Torokhov,
Andy Shevchenko, linux-input, Marco Trevisan, Mark Pearson,
Christian Kellner, Benjamin Berg, Thinkpad-acpi devel ML,
Platform Driver, Linux Kernel Mailing List
On Mon, Jul 27, 2020 at 10:49 AM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
> On Mon, Jul 27, 2020 at 10:45 AM Hans de Goede <hdegoede@redhat.com> wrote:
> >
> > Hi,
> >
> > On 7/27/20 2:50 AM, Henrique de Moraes Holschuh wrote:
> > > On Tue, 21 Jul 2020, Dmitry Torokhov wrote:
> > >> On Sun, Jul 19, 2020 at 07:56:49PM -0300, Henrique de Moraes Holschuh wrote:
> > >>> On Fri, 17 Jul 2020, Hans de Goede wrote:
> > >>>> This is a simple patch-series adding support for 3 new hotkeys found
> > >>>> on various new Lenovo Thinkpad models.
> > >>>
> > >>> For all three patches, pending an ack for the new keycodes by the input
> > >>> maintainers:
> > >>>
> > >>> Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
> > >>
> > >> Do you want me to merge all 3 through input tree?
> > >
> > > Hans, Daren, Andy, what do you prefer?
> >
> > Taking all this upstream through Dmitry's input tree is fine with
> > me, but this really is up to Andy and/or Daren.
>
> Fine with me.
To be clear, I assume it will go thru input tree.
If my formal tag needed, use
Acked-by: Andy Shevchenko <andy.shevchenko@gmail.com>
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-27 10:24 ` Andy Shevchenko
@ 2020-08-01 19:43 ` Henrique de Moraes Holschuh
0 siblings, 0 replies; 13+ messages in thread
From: Henrique de Moraes Holschuh @ 2020-08-01 19:43 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Hans de Goede, Darren Hart, Dmitry Torokhov, Andy Shevchenko,
linux-input, Marco Trevisan, Mark Pearson, Christian Kellner,
Benjamin Berg, Thinkpad-acpi devel ML, Platform Driver,
Linux Kernel Mailing List
On Mon, 27 Jul 2020, Andy Shevchenko wrote:
> On Mon, Jul 27, 2020 at 10:49 AM Andy Shevchenko
> <andy.shevchenko@gmail.com> wrote:
> > On Mon, Jul 27, 2020 at 10:45 AM Hans de Goede <hdegoede@redhat.com> wrote:
> > >
> > > Hi,
> > >
> > > On 7/27/20 2:50 AM, Henrique de Moraes Holschuh wrote:
> > > > On Tue, 21 Jul 2020, Dmitry Torokhov wrote:
> > > >> On Sun, Jul 19, 2020 at 07:56:49PM -0300, Henrique de Moraes Holschuh wrote:
> > > >>> On Fri, 17 Jul 2020, Hans de Goede wrote:
> > > >>>> This is a simple patch-series adding support for 3 new hotkeys found
> > > >>>> on various new Lenovo Thinkpad models.
> > > >>>
> > > >>> For all three patches, pending an ack for the new keycodes by the input
> > > >>> maintainers:
> > > >>>
> > > >>> Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
> > > >>
> > > >> Do you want me to merge all 3 through input tree?
> > > >
> > > > Hans, Daren, Andy, what do you prefer?
> > >
> > > Taking all this upstream through Dmitry's input tree is fine with
> > > me, but this really is up to Andy and/or Daren.
> >
> > Fine with me.
>
> To be clear, I assume it will go thru input tree.
> If my formal tag needed, use
> Acked-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Also,
Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
--
Henrique Holschuh
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads
2020-07-22 5:41 ` Dmitry Torokhov
2020-07-27 0:50 ` Henrique de Moraes Holschuh
@ 2020-08-26 15:12 ` Hans de Goede
1 sibling, 0 replies; 13+ messages in thread
From: Hans de Goede @ 2020-08-26 15:12 UTC (permalink / raw)
To: Dmitry Torokhov, Henrique de Moraes Holschuh
Cc: Darren Hart, Andy Shevchenko, Henrique de Moraes Holschuh,
linux-input, Marco Trevisan, Mark Pearson, Christian Kellner,
Benjamin Berg, ibm-acpi-devel, platform-driver-x86, linux-kernel
Hi Dmitry,
On 7/22/20 7:41 AM, Dmitry Torokhov wrote:
> On Sun, Jul 19, 2020 at 07:56:49PM -0300, Henrique de Moraes Holschuh wrote:
>> On Fri, 17 Jul 2020, Hans de Goede wrote:
>>> This is a simple patch-series adding support for 3 new hotkeys found
>>> on various new Lenovo Thinkpad models.
>>
>> For all three patches, pending an ack for the new keycodes by the input
>> maintainers:
>>
>> Acked-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
>
> Do you want me to merge all 3 through input tree?
Despite Andy's acked-by:
Acked-by: Andy Shevchenko <andy.shevchenko@gmail.com>
For merging these through the input tree I'm not seeing these in 5.9-rc2 ?
Regards,
Hans
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2020-08-26 15:12 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-17 11:41 [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Hans de Goede
2020-07-17 11:41 ` [PATCH 1/3] Input: allocate keycodes for notification-center, pickup-phone and hangup-phone Hans de Goede
2020-07-17 11:41 ` [PATCH 2/3] platform/x86: thinkpad_acpi: Add support for new hotkeys found on X1C8 / T14 Hans de Goede
2020-07-17 11:41 ` [PATCH 3/3] platform/x86: thinkpad_acpi: Map Clipping tool hotkey to KEY_SELECTIVE_SCREENSHOT Hans de Goede
2020-07-17 12:21 ` Bastien Nocera
2020-07-19 22:56 ` [PATCH 0/3] Add 3 new keycodes and use them for 3 new hotkeys on new Lenovo Thinkpads Henrique de Moraes Holschuh
2020-07-22 5:41 ` Dmitry Torokhov
2020-07-27 0:50 ` Henrique de Moraes Holschuh
2020-07-27 7:45 ` Hans de Goede
2020-07-27 7:49 ` Andy Shevchenko
2020-07-27 10:24 ` Andy Shevchenko
2020-08-01 19:43 ` Henrique de Moraes Holschuh
2020-08-26 15:12 ` Hans de Goede
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).