From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755613AbdKJB6s (ORCPT ); Thu, 9 Nov 2017 20:58:48 -0500 Received: from mail-out-2.itc.rwth-aachen.de ([134.130.5.47]:17935 "EHLO mail-out-2.itc.rwth-aachen.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755440AbdKJB6r (ORCPT ); Thu, 9 Nov 2017 20:58:47 -0500 X-IronPort-AV: E=Sophos;i="5.44,371,1505772000"; d="asc'?scan'208";a="22996315" From: Stefan =?ISO-8859-1?Q?Br=FCns?= To: Darren Hart CC: , , AceLan Kao , Andy Shevchenko , Subject: Re: [PATCH v2 2/5] platform/x86: intel-vbtn: Support separate press/release events Date: Fri, 10 Nov 2017 02:58:36 +0100 Message-ID: <1637751.MqEGyCB5Qi@pebbles> In-Reply-To: <20171110013417.GB9783@fury> References: <20171109224436.16472-1-stefan.bruens@rwth-aachen.de> <024c7574-12f2-4708-9f90-1192856df401@rwthex-w2-a.rwth-ad.de> <20171110013417.GB9783@fury> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart2629176.D3vXaZ9z9b"; micalg=pgp-sha1; protocol="application/pgp-signature" X-Originating-IP: [78.49.52.200] X-ClientProxiedBy: rwthex-w3-b.rwth-ad.de (2002:8682:1aa3::8682:1aa3) To rwthex-w2-a.rwth-ad.de (2002:8682:1a9e::8682:1a9e) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --nextPart2629176.D3vXaZ9z9b Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" On Friday, November 10, 2017 2:34:17 AM CET Darren Hart wrote: > On Thu, Nov 09, 2017 at 11:44:33PM +0100, Stefan Br=FCns wrote: > > Currently all key events use autorelease, but this forbids use as a > > modifier key. > >=20 > > As all event codes come in even/odd pairs, we can lookup the key type > > (KE_KEY/KE_IGNORE) for the key up event corresponding to the currently > > handled key down event. If the key up is ignored, we keep setting the > > autorelease flag for the key down. >=20 > What is the use-case for using these buttons as modifiers? I'm picturing = one > of these devices in tablet mode, with a physical Windows button. What oth= er > action does a user want to modify by holding the Windows button down? Or = is > there another scenario we're trying to support here? Windows/KEY_LEFTMETA can be used as a modifier key, e.g. in combination wit= h=20 the Volume Up/Down keys. On Windows, the default for Win + VolumeUp creates= a=20 screenshot. You can also use this in combination with an onscreen keyboard. Pressing th= e=20 hardware button with the hand holding the tablet and typing with the other= =20 hand on the OSK is probably easier than hitting both keys on the OSK. Additionally, the Volume Up/Down currently do not autorepeat, as the key is autoreleased on the press event. The XPS 12 does issue distinct press/relea= se=20 events, so this could be done properly. The same apparently holds for some= =20 other convertibles, see the links in Patch 1/5. Kind regards, Stefan =2D-=20 Stefan Br=FCns / Bergstra=DFe 21 / 52062 Aachen home: +49 241 53809034 mobile: +49 151 50412019 --nextPart2629176.D3vXaZ9z9b Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit -----BEGIN PGP SIGNATURE----- iF0EABECAB0WIQSwWRWIpJbl0W4DemNvf0o9jP6qUwUCWgUHzAAKCRBvf0o9jP6q U/f8AJ9cavM7MbqNUsuGSK5BNh83ThmpAQCfezT+RVVwnqTHlmPobleeGTIBdZo= =g7/J -----END PGP SIGNATURE----- --nextPart2629176.D3vXaZ9z9b--