Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756883Ab1CXNc2 (ORCPT ); Thu, 24 Mar 2011 09:32:28 -0400 Received: from mail-ew0-f46.google.com ([209.85.215.46]:59180 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752717Ab1CXNc1 convert rfc822-to-8bit (ORCPT ); Thu, 24 Mar 2011 09:32:27 -0400 MIME-Version: 1.0 In-Reply-To: References: <1300909189-4135-1-git-send-email-seth.forshee@canonical.com> <20110324131414.GA20612@thinkpad-t410> Date: Thu, 24 Mar 2011 08:32:25 -0500 Message-ID: Subject: Re: [PATCH] eeepc-wmi: Add support for T101MT "Express Gate" key From: Chris Bagwell To: Corentin Chary , Matthew Garrett , acpi4asus-user@lists.sourceforge.net, platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Seth Forshee Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3037 Lines: 71 On Thu, Mar 24, 2011 at 8:27 AM, Chris Bagwell wrote: > On Thu, Mar 24, 2011 at 8:14 AM, Seth Forshee > wrote: >> On Thu, Mar 24, 2011 at 08:33:33AM +0100, Corentin Chary wrote: >>> On Wed, Mar 23, 2011 at 8:39 PM, Seth Forshee >>> wrote: >>> > Signed-off-by: Seth Forshee >>> > --- >>> > >>> > This button is unusual. On press it emits scanocde 0xe4, on release >>> > 0xe5, and when held for a few seconds it begins emitting 0xea about >>> > twice per second. I chose to handle only the press event in order to >>> > make it behave like the other hotkeys. >>> > >>> > Probably the closest thing to the intended function of this button >>> > would be to alternate the display orientation between portrait and >>> > landscape, but since there doesn't appear to be a keycode defined for >>> > that function I went with KEY_PROG2. I'm open to other suggestions >>> > however. >>> >>> Hi, >>> The manual says that under Windows, this is the "Home Key", so >>> KEY_HOME would be more appropriate. >> >> The manual does call it the Home Key (I missed previously that the >> Express Gate description referred only to the quick boot mode), but the >> description of the key operation reads: >> >> ?Shortly press this key to launch the Touch Gate touch screen >> ?interface. Press and hold this key to preview desktop rotation, and >> ?then release the key to finish desktop rotation. >> >> Which doesn't really sound like a home key to me, and the icon above the >> button also suggests display rotation. But then we don't seem to have a >> code for that function, so KEY_PROG2 may be no better than KEY_HOME... >> >>> Also, it would be great to preserve key up and key down informations >>> (sparse keymap have an "autorelease" parameter to control that). >> >> Okay, I'll add some special casing for this key to support that. Do you >> suggest ignoring the 0xea scan codes? >> > > I have one of these in the mail so I'm interested in outcome. > > I suggest implementing as manual describes and can be handled similar > to brightness control logic. > > I would suggest sending BTN_HOME upon button release 0xe5 (not press > 0xe4 and do the autorelease because of it) and BTN_PROG2 on > press-and-hold 0xea. > > If you press-and-hold, does it still send the 0xe5 upon release? ?That > could complicate things so that both button events do not get sent. > > I'm not sure what BTN_HOME is commonly used for. ?This button does > sound like a HOME button in iPad sense but not in web browser sense. > > Chris > When I mentioned similar to brightness control logic, I meant the logic in eeepc-laptop and not eeepc-wmi. It appears to use sparse keys logic but with some event filtering. Chris -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/