Messages in this thread | | | Date | Mon, 25 Jun 2018 15:20:48 +0200 (CEST) | From | Jiri Kosina <> | Subject | Re: [PATCH] Patch enabling Fn function keys for Laptop Asus G730 (keyboard 0x1869) |
| |
On Thu, 14 Jun 2018, vereecke.jan@skynet.be wrote:
> Attached is a patch enabling (most of) the Fn function keys for the Asus ROG > Strix GL730 Laptop. The keyboard identifies as a USB device 0x1869. > > This patch enables: > volume/mute keys, ROG key, rew/play/ff, > monitor and keyboard brightness keys, > Touchpad toggle key, > Calculator key > > 'Touchpad on/off' Key returns keycode 33, which is mapped to the 'p' key. > 'Display on/off' returns keycode 253, which is mapped in X11 to NoSymbol > 'Airplane Key' returns keycode 255, which is mapped in X11 to NoSymbol > > The following keys are not activated yet: > Microphone Mute key, Fan key, Display Switch key
Thanks a lot for the patch.
> This patch is against the 4.17-rc7 kernel, for the x86 architecture
This usually doesn't go to the changelog, it'd be unnecessary polution of the set-in-stone git changelog.
> diffstat for this patch is: > hid-asus.c | 2 ++ > hid-ids.h | 1 + > hid-quirks.c | 1 + > 3 files changed, 4 insertions(+) > > To apply the patch, in the root of the kernel tree use: > patch -p1 < AsusGL703-Keyboard-Fn-keys.patch
Neither does any of this.
What is OTOH missing is your signoff.
Could you please fix these up and resubmit (ideally with Subject: line also tailored to fit the subsystem customs, IOW something like "HID: asus: add support for ....").
Thanks,
-- Jiri Kosina SUSE Labs
| |