From mboxrd@z Thu Jan 1 00:00:00 1970 From: w.sang@pengutronix.de (Wolfram Sang) Date: Fri, 4 Feb 2011 15:54:38 +0100 Subject: [PATCH v3 2/2] Input: ads7846: use gpio_request_one to configure pendown_gpio In-Reply-To: <20110204141618.GE2070@m-desktop> References: <1296746506-12221-1-git-send-email-sourav.poddar@ti.com> <20110203165405.GB12802@core.coreip.homeip.net> <20110203171953.GA13997@core.coreip.homeip.net> <20110204133250.GB2070@m-desktop> <20110204140847.GA3178@pengutronix.de> <20110204141618.GE2070@m-desktop> Message-ID: <20110204145438.GB3178@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Feb 04, 2011 at 07:46:18PM +0530, G, Manjunath Kondaiah wrote: > On Fri, Feb 04, 2011 at 03:08:47PM +0100, Wolfram Sang wrote: > > On Fri, Feb 04, 2011 at 07:02:50PM +0530, G, Manjunath Kondaiah wrote: > > > On Thu, Feb 03, 2011 at 09:19:53AM -0800, Dmitry Torokhov wrote: > > > > On Thu, Feb 03, 2011 at 08:54:05AM -0800, Dmitry Torokhov wrote: > > > > > On Thu, Feb 03, 2011 at 08:51:46PM +0530, Sourav Poddar wrote: > > > > > > The ads7846 driver requests a gpio but does not currently > > > > > > configure it explicitly as an input. Use gpio_request_one > > > > > > to request and configure it at one shot. > > > > > > > > > > > > Signed-off-by: Sourav Poddar > > > > > > Cc: Dmitry Torokhov > > > > > > > > > > Will apply this one, the other one is a bit messy IMO, will have to > > > > > think about it. > > > > > > > > > > > > > Something like below should do I think. > > > Patch looks good but it applies only on top of previous patch: > > > https://patchwork.kernel.org/patch/529941/ > > > > > > Why to have two patches for this fix? > > > > http://www.spinics.net/lists/linux-omap/msg45167.html > My point here is: > 1. The first patch only replaces gpio_request with gpio_request_one > 2. Rest of the things are handled in 2nd patch posted by dmitry I understood. Exactly this is discussed in the first paragraph of that link. -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: