Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932737Ab1CXN1o (ORCPT ); Thu, 24 Mar 2011 09:27:44 -0400 Received: from mail-ew0-f46.google.com ([209.85.215.46]:35634 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932354Ab1CXN1n convert rfc822-to-8bit (ORCPT ); Thu, 24 Mar 2011 09:27:43 -0400 MIME-Version: 1.0 In-Reply-To: <20110324131414.GA20612@thinkpad-t410> References: <1300909189-4135-1-git-send-email-seth.forshee@canonical.com> <20110324131414.GA20612@thinkpad-t410> Date: Thu, 24 Mar 2011 08:27:41 -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: 2704 Lines: 63 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 -- 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/