From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH v2 1/2] Input: jornada720_kbd - switch to devm_* APIs Date: Fri, 19 Aug 2016 17:45:47 +0100 Message-ID: <20160819164547.GU1041@n2100.armlinux.org.uk> References: <20160819164447.GB20499@dtor-ws> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from pandora.armlinux.org.uk ([78.32.30.218]:35098 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754067AbcHSQpy (ORCPT ); Fri, 19 Aug 2016 12:45:54 -0400 Content-Disposition: inline In-Reply-To: <20160819164447.GB20499@dtor-ws> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Dmitry Torokhov Cc: linux-arm-kernel@lists.infradead.org, linux-input@vger.kernel.org On Fri, Aug 19, 2016 at 09:44:47AM -0700, Dmitry Torokhov wrote: > On Fri, Aug 19, 2016 at 05:30:59PM +0100, Russell King wrote: > > Switch the jornada720_kbd driver to use the devm_* APIs. This is in > > preparation to removing the mach/irqs.h include from this driver. > > > > Signed-off-by: Russell King > > --- > > drivers/input/keyboard/jornada720_kbd.c | 38 ++++++++------------------------- > > 1 file changed, 9 insertions(+), 29 deletions(-) > > > > diff --git a/drivers/input/keyboard/jornada720_kbd.c b/drivers/input/keyboard/jornada720_kbd.c > > index 421d9c55b0e8..4b2e9ff18ba1 100644 > > --- a/drivers/input/keyboard/jornada720_kbd.c > > +++ b/drivers/input/keyboard/jornada720_kbd.c > > @@ -53,6 +53,7 @@ static unsigned short jornada_std_keymap[128] = { /* ROW */ > > struct jornadakbd { > > unsigned short keymap[ARRAY_SIZE(jornada_std_keymap)]; > > struct input_dev *input; > > + int irq; > > I do not think you need this, not in this patch and the next one does > not seem to need this either? > > Can I drop this chunk? Yes, that was accidentally left in while I was considering how to update the driver. It can be safely dropped. Thanks for spotting that. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.