[PATCH] of/i2c: don't pass -1 to irq_dispose_mapping, otherwise kernel will oops

Grant Likely grant.likely at secretlab.ca
Sat Jul 12 07:18:13 EST 2008


On Fri, Jul 11, 2008 at 11:19:30PM +0400, Anton Vorontsov wrote:
> On Fri, Jul 11, 2008 at 09:11:02PM +0200, Jochen Friedrich wrote:
> > 
> > Shouldn't this be:
> > 		if (info.irq != NO_IRQ)
> 
> Maybe, maybe not. See
> 
> http://lkml.org/lkml/2005/11/21/211
> 
> I.e. maybe we should use "if (irq)". Or if we should use NO_IRQ, then we
> need to fixup all the users of .irq.

Yes, these are definitely wrong and need to be fixed.

g.




More information about the Linuxppc-dev mailing list