Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752723Ab3FZRQn (ORCPT ); Wed, 26 Jun 2013 13:16:43 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:52907 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751893Ab3FZRQl (ORCPT ); Wed, 26 Jun 2013 13:16:41 -0400 Date: Wed, 26 Jun 2013 10:16:40 -0700 From: Greg Kroah-Hartman To: Darren Hart Cc: Andy Shevchenko , Olof Johansson , Linux Kernel Mailing List , "H. Peter Anvin" , peter.p.waskiewicz.jr@intel.com, danders@circuitco.com, vishal.l.verma@intel.com, Matthew Garrett , Grant Likely , Linus Walleij , platform-driver-x86@vger.kernel.org Subject: Re: [PATCH 6/8] minnowboard-keys: Bind MinnowBoard buttons to arrow keys Message-ID: <20130626171640.GA14661@kroah.com> References: <1ded9714150de50fac7fb6ee82166fbcb5621f7d.1372211451.git.dvhart@linux.intel.com> <1372236386.24799.58.camel@smile> <1372264087.8177.80.camel@envy.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1372264087.8177.80.camel@envy.home> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1921 Lines: 42 On Wed, Jun 26, 2013 at 09:28:07AM -0700, Darren Hart wrote: > On Wed, 2013-06-26 at 11:46 +0300, Andy Shevchenko wrote: > > On Tue, 2013-06-25 at 18:53 -0700, Darren Hart wrote: > > > Configure the four buttons tied to the E6XX GPIO lines on the > > > MinnowBoard as keys using the gpio-keys-polled platform driver. From > > > left to right, bind them to LEFT, DOWN, UP, RIGHT, similar to the VI > > > directional keys. > > > > > > This is separate from the minnowboard driver to provide users with the > > > flexibility to write kernel drivers for their own devices using these GPIO > > > lines. > > > > I'm repeating my comment I did early to you. > > > > I think this driver is a wrong approach, since you just use > > gpio_keys_polling with custom platform data. So, it should go to the > > platform code / board file under arch/x86/platform/minnow/... > > > Olof mentinoed something similar. The reason this is separate is that I > can easily see someone wanting to use these buttons in a different way > when integrating the MinnowBoard into some kind of product. The > minnowboard.c driver sets up the fixed functionality GPIO lines, such as > the LEDs (which can be easily reconfigured via triggers) while this > driver serves as an example of how the GPIO buttons could be used as > keys, but if included in the minnowboard driver, users couldn't get the > fixed functionality without also tying up these GPIO lines. > > I could remedy that with driver command-line options, but I know I've > heard Greg KH discourage their use in the past. Ick, yes, never do that type of thing as a command-line option, that's what device-tree is for :) thanks, greg k-h -- 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/