Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754530Ab1C1OOn (ORCPT ); Mon, 28 Mar 2011 10:14:43 -0400 Received: from mail-ew0-f46.google.com ([209.85.215.46]:34936 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753981Ab1C1OOe convert rfc822-to-8bit (ORCPT ); Mon, 28 Mar 2011 10:14:34 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=M5USyfC6IyYxF6ZiHT9T3e6XcbblG7I9tnJ0tYPNSwezz82NyhKf5vUqOWS3NFGrKZ XdiGhHP/fabnzHqsU9ap06uRJH0wXImlnvxzPNEILvOUaCk+SoeJ2ogBCs9QAuns3RbF LJsHkLnaW5slWmq1OgMSJJd6drgUW32aAxevg= MIME-Version: 1.0 In-Reply-To: <20110328134606.GA16374@thinkpad-t410> References: <20110325135311.GA14328@thinkpad-t410> <20110325161405.GC5099@core.coreip.homeip.net> <20110325162850.GD14328@thinkpad-t410> <20110325170333.GD5099@core.coreip.homeip.net> <20110325185824.GE14328@thinkpad-t410> <20110327171304.GD30244@core.coreip.homeip.net> <20110327191116.GB31692@core.coreip.homeip.net> <20110328134606.GA16374@thinkpad-t410> Date: Mon, 28 Mar 2011 14:14:33 +0000 Message-ID: Subject: Re: [PATCH 2/2] eeepc-wmi: Add support for T101MT Home/Express Gate key From: Corentin Chary To: Dmitry Torokhov , Corentin Chary , Chris Bagwell , Matthew Garrett , acpi4asus-user@lists.sourceforge.net, platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, vojtech@suze.cz Cc: Seth Forshee Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2508 Lines: 56 On Mon, Mar 28, 2011 at 1:46 PM, Seth Forshee wrote: > On Sun, Mar 27, 2011 at 12:11:16PM -0700, Dmitry Torokhov wrote: >> > If we do set up auto-repeat and increase REP_DELAY, I'm guessing this >> > would enable auto-repeat for all other keys defined in driver?  That >> > needs to have some thought on if could have negative impact (any other >> > keys not using auto-release?). >> >> Right. Right now there are 4 autoreprat options (in general): >> >> - hardware autorepeat (if hardware supports it); >> - input core software autorepeat (one delay and rate per input device); >> - driver-implemented software autorepeat - in cases when different >>   repeat rate is needed; >> - userspace autorepeat (like X does nowadays); >> >> Well, 4th option is not mutually exclusive with the other 3... > > Currently all the other keys are using autorelease, so the autorepeat > setting shouldn't be affecting them at all. So the concern is whether > enabling autorepeat could become a problem the next time some oddball > key shows up on a machine. As far as I know, the only key that could be like that (not yet mapped, but I've seen it somewhere) is a "Camera Autofocus Key". For this key we don't really care about "repeating", we just need to send "press" and "release" events correctly, and userspace will keep adjusting the autofocus while the key is pressed. > Corentin, you were concerned about the loss of information to userspace, > but it seems the only way to maintain the hardware events is to drop the > use of sparse-keymap altogether. Do you have an opinion on how to > proceed? I really don't think dropping sparse-keymap for a single key on a single model is a good idea. On my side I'd have used another keycode for 0xea events, and reported press and release events as they come, since it's more a "two-in-one" key than a key with hardware autorepeat. But since Dmitry and Chris seems to prefer the autorepeat approach with a single key code, if it works then it's ok, we just need to set REP_DELAY, report press and release events, and drop 0xea events. > I'm leaning towards using the input core autorepeat, since it seems to > get the closest to typical key behavior. -- Corentin Chary http://xf.iksaif.net -- 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/