* [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop
@ 2016-12-12 12:42 Chris Chiu
2016-12-13 8:16 ` Benjamin Tissoires
2016-12-14 10:20 ` Jiri Kosina
0 siblings, 2 replies; 6+ messages in thread
From: Chris Chiu @ 2016-12-12 12:42 UTC (permalink / raw)
To: Jiri Kosina, Benjamin Tissoires, linux-kernel, linux-input
Cc: linux, Chris Chiu
ROG means ASUS "Republic of Gamers" laptops. The input device info
also represents itself as "ASASTeK COMPUTER INC. ROG MacroKey". It
uses special HID_USAGE code for function keys. This commit remap the
special code to standard keycode for function keys handling. It's
verified on GL553VD/VE, GL753VD/VE.
Signed-off-by: Chris Chiu <chiu@endlessm.com>
Reported-by: Yukai Li <yukaili@gmail.com>
---
drivers/hid/Kconfig | 11 +++++++
drivers/hid/Makefile | 1 +
drivers/hid/hid-asus-rog.c | 80 ++++++++++++++++++++++++++++++++++++++++++++++
drivers/hid/hid-ids.h | 2 ++
include/linux/hid.h | 1 +
5 files changed, 95 insertions(+)
create mode 100644 drivers/hid/hid-asus-rog.c
diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
index 78ac481..925aa7b 100644
--- a/drivers/hid/Kconfig
+++ b/drivers/hid/Kconfig
@@ -144,6 +144,17 @@ config HID_ASUS
- EeeBook X205TA
- VivoBook E200HA
+config HID_ASUS_GAMING_NB_KBD
+ tristate "ASUS special macrokey device for Republic of Gamers laptop"
+ depends on HID
+ ---help---
+ Support for ASUS special macrokey devices on Republic of Gamers latops
+ that are not fully compliant with HID standard
+
+ Say Y if you want support for the non-compliant features of the Asus
+ Republic of Gamers laptop keyboards, e.g:
+ - Asus Notebook GL553VD/GL553VE/GL753VD/GL753VE
+
config HID_AUREAL
tristate "Aureal"
depends on HID
diff --git a/drivers/hid/Makefile b/drivers/hid/Makefile
index fc4b2aa..f15479a 100644
--- a/drivers/hid/Makefile
+++ b/drivers/hid/Makefile
@@ -26,6 +26,7 @@ obj-$(CONFIG_HID_ACRUX) += hid-axff.o
obj-$(CONFIG_HID_APPLE) += hid-apple.o
obj-$(CONFIG_HID_APPLEIR) += hid-appleir.o
obj-$(CONFIG_HID_ASUS) += hid-asus.o
+obj-$(CONFIG_HID_ASUS_GAMING_NB_KBD) += hid-asus-rog.o
obj-$(CONFIG_HID_AUREAL) += hid-aureal.o
obj-$(CONFIG_HID_BELKIN) += hid-belkin.o
obj-$(CONFIG_HID_BETOP_FF) += hid-betopff.o
diff --git a/drivers/hid/hid-asus-rog.c b/drivers/hid/hid-asus-rog.c
new file mode 100644
index 0000000..a33a693
--- /dev/null
+++ b/drivers/hid/hid-asus-rog.c
@@ -0,0 +1,80 @@
+/*
+ * HID driver for some Asus Gaming model equipped with "special" macrokey
+ * devices for hotkey handling
+ *
+ * Currently supported devices are:
+ * ASUS laptops for "Republic of Gamers"
+ * GL553VD/GL553VE
+ * GL753VD/GL753VE
+ *
+ * Copyright (c) 2016 Chris Chiu <chiu@endlessm.com>
+ *
+ * This module based on hid-gyration
+ *
+ */
+
+/*
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the Free
+ * Software Foundation; either version 2 of the License, or (at your option)
+ * any later version.
+ */
+
+#include <linux/device.h>
+#include <linux/input.h>
+#include <linux/hid.h>
+#include <linux/module.h>
+
+#include "hid-ids.h"
+
+#define asus_rog_map_key_clear(c) hid_map_usage_clear(hi, usage, bit, \
+ max, EV_KEY, (c))
+static int asus_rog_input_mapping(struct hid_device *hdev, struct hid_input *hi,
+ struct hid_field *field, struct hid_usage *usage,
+ unsigned long **bit, int *max)
+{
+ if ((usage->hid & HID_USAGE_PAGE) != HID_UP_ASUS_ROG_HOTKEY)
+ return 0;
+
+ set_bit(EV_REP, hi->input->evbit);
+ switch (usage->hid & HID_USAGE) {
+ /* Reported on ASUS Gaming model (Republic of Gamers) keyboards */
+ case 0x6c: asus_rog_map_key_clear(KEY_SLEEP); break;
+ case 0x88: asus_rog_map_key_clear(KEY_WLAN); break;
+ case 0xc5: asus_rog_map_key_clear(KEY_KBDILLUMDOWN); break;
+ case 0xc4: asus_rog_map_key_clear(KEY_KBDILLUMUP); break;
+ case 0x10: asus_rog_map_key_clear(KEY_BRIGHTNESSDOWN); break;
+ case 0x20: asus_rog_map_key_clear(KEY_BRIGHTNESSUP); break;
+ case 0x35: asus_rog_map_key_clear(KEY_DISPLAY_OFF); break;
+ // KEY_F21 is for ASUS touchpad toggle
+ case 0x6b: asus_rog_map_key_clear(KEY_F21); break;
+ case 0x82: asus_rog_map_key_clear(KEY_CAMERA); break;
+ case 0xb5: asus_rog_map_key_clear(KEY_CALC); break;
+ // KEY_PROG1 for ROG key
+ case 0x38: asus_rog_map_key_clear(KEY_PROG1); break;
+ // KEY_PROG2 for Fn+C ASUS Splendid
+ case 0xba: asus_rog_map_key_clear(KEY_PROG2); break;
+ // KEY_PROG3 for Fn+Space Power4Gear Hybrid, may not be present
+ case 0x5c: asus_rog_map_key_clear(KEY_PROG3); break;
+
+ default:
+ return 0;
+ }
+ return 1;
+}
+
+static const struct hid_device_id asus_rog_devices[] = {
+ { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY1) },
+ { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY2) },
+ { }
+};
+MODULE_DEVICE_TABLE(hid, asus_rog_devices);
+
+static struct hid_driver asus_rog_driver = {
+ .name = "asus-rog",
+ .id_table = asus_rog_devices,
+ .input_mapping = asus_rog_input_mapping,
+};
+module_hid_driver(asus_rog_driver);
+
+MODULE_LICENSE("GPL");
diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
index 4ed9a4f..5c60aee 100644
--- a/drivers/hid/hid-ids.h
+++ b/drivers/hid/hid-ids.h
@@ -168,6 +168,8 @@
#define USB_DEVICE_ID_ASUSTEK_LCM 0x1726
#define USB_DEVICE_ID_ASUSTEK_LCM2 0x175b
#define USB_DEVICE_ID_ASUSTEK_NOTEBOOK_KEYBOARD 0x8585
+#define USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY1 0x1854
+#define USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY2 0x1837
#define USB_VENDOR_ID_ATEN 0x0557
#define USB_DEVICE_ID_ATEN_UC100KM 0x2004
diff --git a/include/linux/hid.h b/include/linux/hid.h
index 75b66ec..a6ccadb 100644
--- a/include/linux/hid.h
+++ b/include/linux/hid.h
@@ -170,6 +170,7 @@ struct hid_item {
#define HID_UP_LOGIVENDOR 0xffbc0000
#define HID_UP_LOGIVENDOR2 0xff090000
#define HID_UP_LOGIVENDOR3 0xff430000
+#define HID_UP_ASUS_ROG_HOTKEY 0xff310000
#define HID_UP_LNVENDOR 0xffa00000
#define HID_UP_SENSOR 0x00200000
--
2.1.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop
2016-12-12 12:42 [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop Chris Chiu
@ 2016-12-13 8:16 ` Benjamin Tissoires
2017-01-31 19:42 ` Daniel Drake
2016-12-14 10:20 ` Jiri Kosina
1 sibling, 1 reply; 6+ messages in thread
From: Benjamin Tissoires @ 2016-12-13 8:16 UTC (permalink / raw)
To: Chris Chiu; +Cc: Jiri Kosina, linux-kernel, linux-input, linux
On Dec 12 2016 or thereabouts, Chris Chiu wrote:
> ROG means ASUS "Republic of Gamers" laptops. The input device info
> also represents itself as "ASASTeK COMPUTER INC. ROG MacroKey". It
> uses special HID_USAGE code for function keys. This commit remap the
> special code to standard keycode for function keys handling. It's
> verified on GL553VD/VE, GL753VD/VE.
>
> Signed-off-by: Chris Chiu <chiu@endlessm.com>
> Reported-by: Yukai Li <yukaili@gmail.com>
> ---
Hi Chris,
there is no real point of having a separate driver from hid-asus. The
HID tree prefer to group drivers by vendors, so these keyboards should
be supported through hid-asus.
Please fold the patch in hid-asus and probably add a special quirk for
these keyboards in input_mapping. With a little luck, the quirk won't
even be required and we can apply these mapping to all Asus keyboards.
> drivers/hid/Kconfig | 11 +++++++
> drivers/hid/Makefile | 1 +
> drivers/hid/hid-asus-rog.c | 80 ++++++++++++++++++++++++++++++++++++++++++++++
> drivers/hid/hid-ids.h | 2 ++
> include/linux/hid.h | 1 +
> 5 files changed, 95 insertions(+)
> create mode 100644 drivers/hid/hid-asus-rog.c
>
> diff --git a/drivers/hid/Kconfig b/drivers/hid/Kconfig
> index 78ac481..925aa7b 100644
> --- a/drivers/hid/Kconfig
> +++ b/drivers/hid/Kconfig
> @@ -144,6 +144,17 @@ config HID_ASUS
> - EeeBook X205TA
> - VivoBook E200HA
>
> +config HID_ASUS_GAMING_NB_KBD
> + tristate "ASUS special macrokey device for Republic of Gamers laptop"
> + depends on HID
> + ---help---
> + Support for ASUS special macrokey devices on Republic of Gamers latops
> + that are not fully compliant with HID standard
> +
> + Say Y if you want support for the non-compliant features of the Asus
> + Republic of Gamers laptop keyboards, e.g:
> + - Asus Notebook GL553VD/GL553VE/GL753VD/GL753VE
> +
> config HID_AUREAL
> tristate "Aureal"
> depends on HID
> diff --git a/drivers/hid/Makefile b/drivers/hid/Makefile
> index fc4b2aa..f15479a 100644
> --- a/drivers/hid/Makefile
> +++ b/drivers/hid/Makefile
> @@ -26,6 +26,7 @@ obj-$(CONFIG_HID_ACRUX) += hid-axff.o
> obj-$(CONFIG_HID_APPLE) += hid-apple.o
> obj-$(CONFIG_HID_APPLEIR) += hid-appleir.o
> obj-$(CONFIG_HID_ASUS) += hid-asus.o
> +obj-$(CONFIG_HID_ASUS_GAMING_NB_KBD) += hid-asus-rog.o
> obj-$(CONFIG_HID_AUREAL) += hid-aureal.o
> obj-$(CONFIG_HID_BELKIN) += hid-belkin.o
> obj-$(CONFIG_HID_BETOP_FF) += hid-betopff.o
> diff --git a/drivers/hid/hid-asus-rog.c b/drivers/hid/hid-asus-rog.c
> new file mode 100644
> index 0000000..a33a693
> --- /dev/null
> +++ b/drivers/hid/hid-asus-rog.c
> @@ -0,0 +1,80 @@
> +/*
> + * HID driver for some Asus Gaming model equipped with "special" macrokey
> + * devices for hotkey handling
> + *
> + * Currently supported devices are:
> + * ASUS laptops for "Republic of Gamers"
> + * GL553VD/GL553VE
> + * GL753VD/GL753VE
> + *
> + * Copyright (c) 2016 Chris Chiu <chiu@endlessm.com>
> + *
> + * This module based on hid-gyration
> + *
> + */
> +
> +/*
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License as published by the Free
> + * Software Foundation; either version 2 of the License, or (at your option)
> + * any later version.
> + */
> +
> +#include <linux/device.h>
> +#include <linux/input.h>
> +#include <linux/hid.h>
> +#include <linux/module.h>
> +
> +#include "hid-ids.h"
> +
> +#define asus_rog_map_key_clear(c) hid_map_usage_clear(hi, usage, bit, \
> + max, EV_KEY, (c))
> +static int asus_rog_input_mapping(struct hid_device *hdev, struct hid_input *hi,
> + struct hid_field *field, struct hid_usage *usage,
> + unsigned long **bit, int *max)
> +{
> + if ((usage->hid & HID_USAGE_PAGE) != HID_UP_ASUS_ROG_HOTKEY)
> + return 0;
> +
> + set_bit(EV_REP, hi->input->evbit);
> + switch (usage->hid & HID_USAGE) {
> + /* Reported on ASUS Gaming model (Republic of Gamers) keyboards */
> + case 0x6c: asus_rog_map_key_clear(KEY_SLEEP); break;
> + case 0x88: asus_rog_map_key_clear(KEY_WLAN); break;
> + case 0xc5: asus_rog_map_key_clear(KEY_KBDILLUMDOWN); break;
> + case 0xc4: asus_rog_map_key_clear(KEY_KBDILLUMUP); break;
> + case 0x10: asus_rog_map_key_clear(KEY_BRIGHTNESSDOWN); break;
> + case 0x20: asus_rog_map_key_clear(KEY_BRIGHTNESSUP); break;
> + case 0x35: asus_rog_map_key_clear(KEY_DISPLAY_OFF); break;
> + // KEY_F21 is for ASUS touchpad toggle
> + case 0x6b: asus_rog_map_key_clear(KEY_F21); break;
> + case 0x82: asus_rog_map_key_clear(KEY_CAMERA); break;
> + case 0xb5: asus_rog_map_key_clear(KEY_CALC); break;
> + // KEY_PROG1 for ROG key
> + case 0x38: asus_rog_map_key_clear(KEY_PROG1); break;
> + // KEY_PROG2 for Fn+C ASUS Splendid
> + case 0xba: asus_rog_map_key_clear(KEY_PROG2); break;
> + // KEY_PROG3 for Fn+Space Power4Gear Hybrid, may not be present
> + case 0x5c: asus_rog_map_key_clear(KEY_PROG3); break;
> +
> + default:
> + return 0;
> + }
> + return 1;
> +}
> +
> +static const struct hid_device_id asus_rog_devices[] = {
> + { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY1) },
> + { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY2) },
> + { }
> +};
> +MODULE_DEVICE_TABLE(hid, asus_rog_devices);
> +
> +static struct hid_driver asus_rog_driver = {
> + .name = "asus-rog",
> + .id_table = asus_rog_devices,
> + .input_mapping = asus_rog_input_mapping,
> +};
> +module_hid_driver(asus_rog_driver);
> +
> +MODULE_LICENSE("GPL");
> diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
> index 4ed9a4f..5c60aee 100644
> --- a/drivers/hid/hid-ids.h
> +++ b/drivers/hid/hid-ids.h
> @@ -168,6 +168,8 @@
> #define USB_DEVICE_ID_ASUSTEK_LCM 0x1726
> #define USB_DEVICE_ID_ASUSTEK_LCM2 0x175b
> #define USB_DEVICE_ID_ASUSTEK_NOTEBOOK_KEYBOARD 0x8585
> +#define USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY1 0x1854
> +#define USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY2 0x1837
>
> #define USB_VENDOR_ID_ATEN 0x0557
> #define USB_DEVICE_ID_ATEN_UC100KM 0x2004
> diff --git a/include/linux/hid.h b/include/linux/hid.h
> index 75b66ec..a6ccadb 100644
> --- a/include/linux/hid.h
> +++ b/include/linux/hid.h
> @@ -170,6 +170,7 @@ struct hid_item {
> #define HID_UP_LOGIVENDOR 0xffbc0000
> #define HID_UP_LOGIVENDOR2 0xff090000
> #define HID_UP_LOGIVENDOR3 0xff430000
> +#define HID_UP_ASUS_ROG_HOTKEY 0xff310000
Not sure this belongs here. You are checking this in a local driver, not
in the generic hid-input path, so there is no point in telling everybody
that ASUS ROG keyboards are using this usage page.
Cheers,
Benjamin
> #define HID_UP_LNVENDOR 0xffa00000
> #define HID_UP_SENSOR 0x00200000
>
> --
> 2.1.4
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop
2016-12-12 12:42 [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop Chris Chiu
2016-12-13 8:16 ` Benjamin Tissoires
@ 2016-12-14 10:20 ` Jiri Kosina
2016-12-14 16:50 ` Chris Chiu
1 sibling, 1 reply; 6+ messages in thread
From: Jiri Kosina @ 2016-12-14 10:20 UTC (permalink / raw)
To: Chris Chiu; +Cc: Benjamin Tissoires, linux-kernel, linux-input, linux
On Mon, 12 Dec 2016, Chris Chiu wrote:
[ ... snip ... ]
> +static const struct hid_device_id asus_rog_devices[] = {
> + { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY1) },
> + { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY2) },
> + { }
> +};
> +MODULE_DEVICE_TABLE(hid, asus_rog_devices);
[ ... snip ... ]
In addition to what Benjamin said already, you also seem to be missing
hid_have_special_driver[] entry for these VID/PIDs
Thanks,
--
Jiri Kosina
SUSE Labs
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop
2016-12-14 10:20 ` Jiri Kosina
@ 2016-12-14 16:50 ` Chris Chiu
0 siblings, 0 replies; 6+ messages in thread
From: Chris Chiu @ 2016-12-14 16:50 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Benjamin Tissoires, linux-kernel, linux-input, linux
On Wed, Dec 14, 2016 at 6:20 PM, Jiri Kosina <jikos@kernel.org> wrote:
> On Mon, 12 Dec 2016, Chris Chiu wrote:
>
> [ ... snip ... ]
>> +static const struct hid_device_id asus_rog_devices[] = {
>> + { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY1) },
>> + { HID_USB_DEVICE(USB_VENDOR_ID_ASUSTEK, USB_DEVICE_ID_ASUSTEK_ROG_MACROKEY2) },
>> + { }
>> +};
>> +MODULE_DEVICE_TABLE(hid, asus_rog_devices);
>
> [ ... snip ... ]
>
> In addition to what Benjamin said already, you also seem to be missing
> hid_have_special_driver[] entry for these VID/PIDs
>
> Thanks,
>
> --
> Jiri Kosina
> SUSE Labs
>
You're right. I forgot to add VID/PID in hid_have_special_driver in
this patch. I'll modify per Benjamin's suggestion and re-commit.
Thanks.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop
2016-12-13 8:16 ` Benjamin Tissoires
@ 2017-01-31 19:42 ` Daniel Drake
2017-02-01 8:18 ` Benjamin Tissoires
0 siblings, 1 reply; 6+ messages in thread
From: Daniel Drake @ 2017-01-31 19:42 UTC (permalink / raw)
To: Benjamin Tissoires
Cc: Chris Chiu, Jiri Kosina, Linux Kernel, linux-input,
Linux Upstreaming Team
Hi Benjamin,
On Tue, Dec 13, 2016 at 2:16 AM, Benjamin Tissoires
<benjamin.tissoires@redhat.com> wrote:
> On Dec 12 2016 or thereabouts, Chris Chiu wrote:
>> ROG means ASUS "Republic of Gamers" laptops. The input device info
>> also represents itself as "ASASTeK COMPUTER INC. ROG MacroKey". It
>> uses special HID_USAGE code for function keys. This commit remap the
>> special code to standard keycode for function keys handling. It's
>> verified on GL553VD/VE, GL753VD/VE.
>>
>> Signed-off-by: Chris Chiu <chiu@endlessm.com>
>> Reported-by: Yukai Li <yukaili@gmail.com>
>> ---
>
> Hi Chris,
>
> there is no real point of having a separate driver from hid-asus. The
> HID tree prefer to group drivers by vendors, so these keyboards should
> be supported through hid-asus.
Thanks for the feedback. We will submit a new version soon.
Just to confirm, hid-asus currently depends on I2C_HID. The new
devices being quirked now are USB devices. With this patch should we
now update it to depend on I2C_HID && USB_HID?
Daniel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop
2017-01-31 19:42 ` Daniel Drake
@ 2017-02-01 8:18 ` Benjamin Tissoires
0 siblings, 0 replies; 6+ messages in thread
From: Benjamin Tissoires @ 2017-02-01 8:18 UTC (permalink / raw)
To: Daniel Drake
Cc: Chris Chiu, Jiri Kosina, Linux Kernel, linux-input,
Linux Upstreaming Team
On Jan 31 2017 or thereabouts, Daniel Drake wrote:
> Hi Benjamin,
>
> On Tue, Dec 13, 2016 at 2:16 AM, Benjamin Tissoires
> <benjamin.tissoires@redhat.com> wrote:
> > On Dec 12 2016 or thereabouts, Chris Chiu wrote:
> >> ROG means ASUS "Republic of Gamers" laptops. The input device info
> >> also represents itself as "ASASTeK COMPUTER INC. ROG MacroKey". It
> >> uses special HID_USAGE code for function keys. This commit remap the
> >> special code to standard keycode for function keys handling. It's
> >> verified on GL553VD/VE, GL753VD/VE.
> >>
> >> Signed-off-by: Chris Chiu <chiu@endlessm.com>
> >> Reported-by: Yukai Li <yukaili@gmail.com>
> >> ---
> >
> > Hi Chris,
> >
> > there is no real point of having a separate driver from hid-asus. The
> > HID tree prefer to group drivers by vendors, so these keyboards should
> > be supported through hid-asus.
>
> Thanks for the feedback. We will submit a new version soon.
>
> Just to confirm, hid-asus currently depends on I2C_HID. The new
> devices being quirked now are USB devices. With this patch should we
> now update it to depend on I2C_HID && USB_HID?
>
Actually you can remove the I2C_HID dependency entirely (and not add one
for USB_HID). In the Kconfig, the modules are already guarded by a
"if HID", and there is nothing transport specific in hid-asus as far as
I can tell.
Note: we should also probably clean up the Kconfig once again and remove
all the "depends on HID" for those that are guarded by the if statement.
Cheers,
Benjamin
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-02-01 8:19 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-12 12:42 [PATCH] HID: add Asus macrokey support for Asus "Republic of Gamers" laptop Chris Chiu
2016-12-13 8:16 ` Benjamin Tissoires
2017-01-31 19:42 ` Daniel Drake
2017-02-01 8:18 ` Benjamin Tissoires
2016-12-14 10:20 ` Jiri Kosina
2016-12-14 16:50 ` Chris Chiu
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).