From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mika Westerberg Subject: Re: [PATCH] gpio: better lookup method for platform GPIOs Date: Thu, 28 Nov 2013 18:59:22 +0200 Message-ID: <20131128165922.GC2281@intel.com> References: <1385628388-23827-1-git-send-email-acourbot@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mga02.intel.com ([134.134.136.20]:38542 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751551Ab3K1Qwo (ORCPT ); Thu, 28 Nov 2013 11:52:44 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-gpio-owner@vger.kernel.org List-Id: linux-gpio@vger.kernel.org To: Linus Walleij Cc: Alexandre Courbot , Rhyland Klein , Heikki Krogerus , "linux-gpio@vger.kernel.org" , "linux-kernel@vger.kernel.org" On Thu, Nov 28, 2013 at 03:45:37PM +0100, Linus Walleij wrote: > On Thu, Nov 28, 2013 at 9:46 AM, Alexandre Courbot wrote: > > > Change the format of the platform GPIO lookup tables to make them less > > confusing and improve lookup efficiency. > > > > The previous format was a single linked-list that required to compare > > the device name and function ID of every single GPIO defined for each > > lookup. Switch that to a list of per-device tables, so that the lookup > > can be done in two steps, omitting the GPIOs that are not relevant for a > > particular device. > > > > The matching rules are now defined as follows: > > - The device name must match *exactly*, and can be NULL for GPIOs not > > assigned to a particular device, > > - If the function ID in the lookup table is NULL, the con_id argument of > > gpiod_get() will not be used for lookup. However, if it is defined, it > > must match exactly. > > - The index must always match. > > > > Signed-off-by: Alexandre Courbot > > --- > > Let's change this bad design before more users start using it. ;) > > OK given that Mika has based his patches on this I'll request > that he ACK this and then I'll merge this so that he can put > this patch at the bottom of his patch stack. > > Mika: requesting ACK. Alexandre, please check Andy's comments. Apart from that this looks good to me. Acked-by: Mika Westerberg