From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH 6/8] minnowboard-keys: Bind MinnowBoard buttons to arrow keys Date: Wed, 26 Jun 2013 11:46:26 +0300 Message-ID: <1372236386.24799.58.camel@smile> References: <1ded9714150de50fac7fb6ee82166fbcb5621f7d.1372211451.git.dvhart@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Return-path: Received: from mga02.intel.com ([134.134.136.20]:62025 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751678Ab3FZIqf convert rfc822-to-8bit (ORCPT ); Wed, 26 Jun 2013 04:46:35 -0400 In-Reply-To: <1ded9714150de50fac7fb6ee82166fbcb5621f7d.1372211451.git.dvhart@linux.intel.com> Sender: platform-driver-x86-owner@vger.kernel.org List-ID: To: Darren Hart Cc: 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 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/... -- Andy Shevchenko Intel Finland Oy