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