linux-kernel - Re: [PATCH] Patch enabling Fn function keys for Laptop Asus G730 (keyboard 0x1869)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <nycvar.YFH.7.76.1806251519190.6203@cbobk.fhfr.pm>
Date:   Mon, 25 Jun 2018 15:20:48 +0200 (CEST)
From:   Jiri Kosina <jikos@...nel.org>
To:     vereecke.jan@...net.be
cc:     linux-kernel@...r.kernel.org, linux-input@...r.kernel.org,
        benjamin.tissoires@...hat.com
Subject: Re: [PATCH] Patch enabling Fn function keys for Laptop Asus G730
 (keyboard 0x1869)

On Thu, 14 Jun 2018, vereecke.jan@...net.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