On Mon, Nov 3, 2014 at 5:50 PM, Lee Jones <[email protected]> wrote: > On Mon, 03 Nov 2014, Fabio Estevam wrote: > >> On Mon, Nov 3, 2014 at 2:15 PM, Lee Jones <[email protected]> wrote: >> >> > /* Clean up the mailbox interrupts after pre-kernel code. */ >> > @@ -3179,15 +3179,14 @@ static int db8500_prcmu_probe(struct >> > platform_device *pdev) >> > irq = platform_get_irq(pdev, 0); >> > if (irq <= 0) { >> >> Shouldn't this be if (irq < 0) then? > > Perhaps Linus can answer that. > > Linus, > > Is 0 a valid IRQ on the u8500 platform or not?
It is not. 0 is NO_IRQ and should be so for every system... just the lazy and ancient ones lag behind. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

