[PATCH 1/6] arm/cpuimx51: gpio_to_irq() should be used to get irq from gpio number

Russell King - ARM Linux linux at arm.linux.org.uk
Sat Aug 13 05:35:29 EDT 2011


On Fri, Aug 12, 2011 at 07:13:07PM +0800, Shawn Guo wrote:
> On Fri, Aug 12, 2011 at 05:36:32PM +0800, Jason Liu wrote:
> > 2011/8/12 Shawn Guo <shawn.guo at linaro.org>:
> > > Instead of irq_to_gpio(), gpio_to_irq() should be used to get irq
> > > from gpio number.
> > >
> > > Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
> > > Cc: Sascha Hauer <s.hauer at pengutronix.de>
> > > ---
> > >  arch/arm/mach-mx5/board-cpuimx51.c |    2 +-
> > >  1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-mx5/board-cpuimx51.c b/arch/arm/mach-mx5/board-cpuimx51.c
> > > index 7c893fa..68934ea 100644
> > > --- a/arch/arm/mach-mx5/board-cpuimx51.c
> > > +++ b/arch/arm/mach-mx5/board-cpuimx51.c
> > > @@ -81,7 +81,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
> > >                .flags = UPF_BOOT_AUTOCONF | UPF_SKIP_TEST | UPF_IOREMAP,
> > >        }, {
> > >                .mapbase = (unsigned long)(MX51_CS1_BASE_ADDR + 0x2000000),
> > > -               .irq = irq_to_gpio(CPUIMX51_QUARTD_GPIO),
> > > +               .irq = gpio_to_irq(CPUIMX51_QUARTD_GPIO),
> > >                .irqflags = IRQF_TRIGGER_HIGH,
> > >                .uartclk = CPUIMX51_QUART_XTAL,
> > >                .regshift = CPUIMX51_QUART_REGSHIFT,
> > > --
> > 
> > The fix has already in Sascah's tree:
> > 
> > Author: Ben Dooks <ben-linux at fluff.org>
> > Date:   Thu Aug 4 16:47:35 2011 +0100
> > 
> >     ARM: mx5: board-cpuimx51.c fixup irq_to_gpio() usage
> > 
> >     irq_to_gpio() is being called on a GPIO so change to using
> >     gpio_to_irq() instead.
> > 
> >     Signed-off-by: Ben Dooks <ben-linux at fluff.org>
> >     Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
> > 
> > Jason
> > 
> Sorry, I missed that.  Thanks for pointing it out, Jason.
> 
> I will fold Ben's patch here, as I see Russell would probably pick up
> the series.  Otherwise, Russell, please let me know if I should base
> it on Sascha's tree.

Shouldn't this patch be targetted for -rc and maybe stable?  It's
difficult to tell from the commit description whether it really does
cause a runtime problem or not.

If it does go in to a -rc, then I can easily move my gpio patch set
forward.



More information about the linux-arm-kernel mailing list