linux-kernel - Re: [PATCH 5/6] input: Add keycodes used by Lenovo Carbon X1 2014

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 for Android: free password hash cracker in your pocket

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

Message-Id: <1424700613.3136058.231192293.03A600A7@webmail.messagingengine.com>
Date:	Mon, 23 Feb 2015 11:10:13 -0300
From:	Henrique de Moraes Holschuh <hmh@....eng.br>
To:	Bastien Nocera <hadess@...ess.net>
Cc:	Darren Hart <dvhart@...radead.org>,
	ibm-acpi-devel@...ts.sourceforge.net,
	platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-input@...r.kernel.org
Subject: Re: [PATCH 5/6] input: Add keycodes used by Lenovo Carbon X1 2014

On Mon, Feb 23, 2015, at 08:41, Bastien Nocera wrote:
> > You could opt to change some of those keypresses dinamically into
> > driver-specific
> > ACPI events, by setting the keymap entry to KEY_RESERVED.  The driver
> > will
> > issue HKEY events in that case. That will be an viable alternative for
> > some of the new
> > keys.
> 
> I don't understand what you mean here.

I mean please set anything that you don't have a keycode for in the new
keymaps to either KEY_RESERVED (will issue an HKEY ACPI event) or
KEY_UNKNOWN (will issue an input EV_KEY event for KEY_UNKNOWN grouped
with an EV_MSC EV MSC_SCAN event that allows userspace to know which key
was pressed/released even if it has no specific keycode).

> > The beauty of it is that keymaps are configurable in run-time, so
> > userspace can
> > just set any key entries it wants to change to something else, and ACPI
> > events will be
> > suppressed if the entry is not set to KEY_RESERVED anymore.
> 
> Keymaps are already configurable at run-time, that's how udev changes
> keymaps. Or did you mean something else?

I mean if anyone wants to change something that is KEY_RESERVED or
KEY_UNKNOWN they can configure the driver to do so at runtime through
the input device interface, i.e. get udev to do it.

Nothing new, there.

> > We need to be careful with tpacpi_input_send_key_masked(). If any of the
> > codepaths
> > could call it from the adaptative keyboard handling code,
> 
> But we're not calling it from there, are we?

Probably not, but it would be a good idea to double-check.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/