Re: [PATCH v2 5/5] platform/x86: intel-vbtn: support panel front button

From: Bastien Nocera
Date: Thu Nov 09 2017 - 18:40:44 EST


On Thu, 2017-11-09 at 23:44 +0100, Stefan BrÃns wrote:
> The Lenovo Helix 2 and Dell XPS 12 (9Q33) have an extra button on the
> front showing a 'Windows' logo, both reporting event codes 0xC2/0xC3
> on press/release. On the Dell, both press/release are distinct events
> while on the Helix 2 both events are generated on release.
>
> Tested on XPS 12, for info on the Helix 2 see:
> https://www.spinics.net/lists/ibm-acpi-devel/msg03982.html
>
> Signed-off-by: Stefan BrÃns <stefan.bruens@xxxxxxxxxxxxxx>

Same problem as mentioned in patch 3. Pretty sure you need to set the
Windows key release to KEY_IGNORE.

Or better, teach the intel-vbtn driver which buttons should be
autoreleased, and which ones should send key presses and key releases
separately. This would allow handling long presses in the upper layers.

> ---
>
> Changes in v2:
> - Emit KEY_LEFTMETA instead of KEY_MENU
>
> drivers/platform/x86/intel-vbtn.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/platform/x86/intel-vbtn.c
> b/drivers/platform/x86/intel-vbtn.c
> index a484bcc6393b..0861efe490d0 100644
> --- a/drivers/platform/x86/intel-vbtn.c
> +++ b/drivers/platform/x86/intel-vbtn.c
> @@ -38,6 +38,8 @@ static const struct acpi_device_id intel_vbtn_ids[]
> = {
> static const struct key_entry intel_vbtn_keymap[] = {
> { KE_KEY, 0xC0, { KEY_POWER } }, /* power key press
> */
> { KE_IGNORE, 0xC1, { KEY_POWER } }, /* power key
> release */
> + { KE_KEY, 0xC2, { KEY_LEFTMETA } }, /*
> 'Windows' key press */
> + { KE_KEY, 0xC3, { KEY_LEFTMETA } }, /*
> 'Windows' key release */
> { KE_KEY, 0xC4, { KEY_VOLUMEUP } }, /*
> volume-up key press */
> { KE_IGNORE, 0xC5, { KEY_VOLUMEUP } }, /*
> volume-up key release */
> { KE_KEY, 0xC6, { KEY_VOLUMEDOWN } }, /*
> volume-down key press */