Re: suspicious code in linux/arch/arm/common/sa1111.c

From: Pavel Machek
Date: Sun Jan 09 2011 - 02:15:28 EST


On Tue 2010-11-16 17:09:59, Eric Miao wrote:
> On Tue, Nov 16, 2010 at 5:07 PM, Pavel Machek <pma@xxxxxxxxx> wrote:
> > Hi!
> >
> > Is this good idea?
> >
> > Signed-off-by: Pavel Machek <pma@xxxxxxxxx>
>
> Yep, obviously was a typo.

Can you apply the patch? Or should I re--generate one with better changelog?

> > diff -ur linux.ofic//arch/arm/common/sa1111.c linux/arch/arm/common/sa1111.c
> > --- linux.ofic//arch/arm/common/sa1111.c        2010-10-17 20:31:13.000000000 +0200
> > +++ linux/arch/arm/common/sa1111.c      2010-11-05 10:44:17.000000000 +0100
> > @@ -282,7 +312,7 @@
> >        for (i = 0; i < 8; i++) {
> >                sa1111_writel(ip0 ^ mask, mapbase + SA1111_INTPOL0);
> >                sa1111_writel(ip0, mapbase + SA1111_INTPOL0);
> > -               if (sa1111_readl(mapbase + SA1111_INTSTATCLR1) & mask)
> > +               if (sa1111_readl(mapbase + SA1111_INTSTATCLR0) & mask)
> >                        break;
> >        }
> >
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/