From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Dooks Subject: Re: [PATCH] I2C: ISP1301_OMAP: New-style i2c driver updates, part 2 Date: Mon, 31 Mar 2008 18:37:49 +0100 Message-ID: <20080331173749.GA10234@fluff.org.uk> References: <1205585237-21492-1-git-send-email-me@felipebalbi.com> <1205585237-21492-2-git-send-email-me@felipebalbi.com> <1205585237-21492-3-git-send-email-me@felipebalbi.com> <1205585237-21492-4-git-send-email-me@felipebalbi.com> <20080315124918.GA21547@kedavra.cpe.vivax.com.br> <20080315125458.GB21547@kedavra.cpe.vivax.com.br> <20080315131309.GA24990@kedavra.cpe.vivax.com.br> <20080316105617.GA4503@kedavra.cpe.vivax.com.br> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20080316105617.GA4503-4vvIQG7NF+ITKvXZea5imILpUPVTGn5w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org Errors-To: i2c-bounces-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org To: Felipe Balbi Cc: David Brownell , i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org List-Id: linux-i2c@vger.kernel.org On Sun, Mar 16, 2008 at 12:56:18PM +0200, Felipe Balbi wrote: > This patch uses set_irq_type as suggested by Dave. > > ============= CUT HERE ========= > > >From 293a73277794a1ca0f2307e7fe9d5ecd76d3ad06 Mon Sep 17 00:00:00 2001 > From: Felipe Balbi > Date: Tue, 1 Jan 2008 23:00:18 -0500 > Subject: [PATCH] I2C: ISP1301_OMAP: New-style i2c driver updates, part 2 > > Based on David Brownell's patch for tps65010, this patch > finish conversting isp1301_omap.c to new-style i2c driver. > > Signed-off-by: Felipe Balbi > --- > arch/arm/mach-omap1/board-h2.c | 4 + > arch/arm/mach-omap1/board-h3.c | 8 ++ > arch/arm/mach-omap2/board-h4.c | 14 ++++ > drivers/i2c/chips/isp1301_omap.c | 134 ++++++++++--------------------------- > 4 files changed, 62 insertions(+), 98 deletions(-) > > diff --git a/arch/arm/mach-omap1/board-h2.c b/arch/arm/mach-omap1/board-h2.c > index 5079877..fd9567c 100644 > --- a/arch/arm/mach-omap1/board-h2.c > +++ b/arch/arm/mach-omap1/board-h2.c > @@ -22,6 +22,8 @@ > #include > #include > #include > +#include > +#include > #include > #include > #include > @@ -366,6 +368,8 @@ static void __init h2_init_irq(void) > omap_init_irq(); > omap_gpio_init(); > h2_init_smc91x(); > + set_irq_type(OMAP_GPIO_IRQ(2), > + IRQF_SAMPLE_RANDOM | IRQF_TRIGGER_FALLING); Calling set_irq_type directly is not the recommended way of doing this. Use the correct trigger flags to request_irq(). -- Ben (ben-elnMNo+KYs3YtjvyW6yDsg@public.gmane.org, http://www.fluff.org/) 'a smiley only costs 4 bytes' _______________________________________________ i2c mailing list i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org http://lists.lm-sensors.org/mailman/listinfo/i2c