linux-kernel - Re: [PATCH] dell-wmi: Add KEY_SWITCHVIDEOMODE for Dell N4110.

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>] [day] [month] [year] [list]

Message-ID: <52DD4991.4050308@gmail.com>
Date:	Mon, 20 Jan 2014 10:06:41 -0600
From:	Souvik Banerjee <souvik1997@...il.com>
To:	linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] dell-wmi: Add KEY_SWITCHVIDEOMODE for Dell N4110.

Forgot to cc the LKML.

-------- Original Message --------
Subject: Re: [PATCH] dell-wmi: Add KEY_SWITCHVIDEOMODE for Dell N4110.
Date: Mon, 20 Jan 2014 10:05:03 -0600
From: Souvik Banerjee <souvik1997@...il.com>
To: Matthew Garrett <mjg59@...f.ucam.org>

No, I don't think so. My testing shows that it is now just reported by
the keyboard controller. My commit is similar to
'f1566f0dc07ec9b5409b348070f5a700032d7881' in which keys for the Dell
XPS L502X are added to the legacy keymap to reduce the noise. Without
this patch dell-wmi reports an unknown key, but with it those messages
are silenced.

--
Souvik Banerjee

On 01/20/2014 09:31 AM, Matthew Garrett wrote:
> On Mon, Jan 20, 2014 at 09:28:22AM -0600, Souvik Banerjee wrote:
>> This key is reported by the keyboard controller but also generates WMI
>> events. By adding it to the legacy keymap the WMI messages can be
>> silenced.
> 
> Wouldn't this result in it being reported twice?
> 

Download attachment "signature.asc" of type "application/pgp-signature" (902 bytes)