From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Dmitry Torokhov" Subject: Re: [PATCH] Tosa keyboard support Date: Fri, 11 Jan 2008 15:52:04 -0500 Message-ID: References: <20071223232914.GA28932@doriath.ww600.siemens.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from fk-out-0910.google.com ([209.85.128.191]:9521 "EHLO fk-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750761AbYAKUwG (ORCPT ); Fri, 11 Jan 2008 15:52:06 -0500 Received: by fk-out-0910.google.com with SMTP id z23so979369fkz.5 for ; Fri, 11 Jan 2008 12:52:05 -0800 (PST) In-Reply-To: Content-Disposition: inline Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Dmitry Baryshkov Cc: linux-input@vger.kernel.org, linux-arm-kernel@lists.arm.linux.org.uk On Jan 11, 2008 11:38 AM, Dmitry Baryshkov wrote: > Hi, > > > Dmitry Torokhov wrote: > > > Hi, > > > > On Jan 3, 2008 6:01 PM, Dmitry Baryshkov wrote: > >> > >> > >> > Also, I'd put these defines rigth in tosakbd.c. > >> > >> The TOSA_KEY_RECORD and SYNC are used inside arch/arm/mach-pxa/tosa.c > >> (for gpio-keys declaration). So I would prefer to place all such > >> defines here. When the Kdrive would be fixed, I would drop KEY_RECORD > >> and happily move that defines to tosakbd.c > >> > >> > > Ok, fair enough. Because driver touches arch-specific arm parts it does > > not apply cleanly to my tree. I think the best way would be to merge it > > through arm side, this will minimize merge efforts Feel free to add: > > Strange. The patch is against 2.6.24-rc7 and applies cleanly to both > for-linus and master branches of .../dtor/input.git Hmm, I think the version I originally tried did not... Anyway, my concern is that when flood gates for .25 are opened there will be alot of changes in PXA land. However changes to input side are trivial (Kconfig and Makefile) and are resolved much easier so if you coordinate with Russel it will work much better. > > Acked-by: Dmitry Torokhov > > Russell, do I need to resent patch? > -- Dmitry