[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1159649659.13029.149.camel@localhost.localdomain>
Date: Sat, 30 Sep 2006 21:54:19 +0100
From: Alan Cox <alan@...rguk.ukuu.org.uk>
To: James Bottomley <James.Bottomley@...elEye.com>
Cc: Frederik Deweerdt <deweerdt@...e.fr>,
Matthew Wilcox <matthew@....cx>,
"J.A. Magall??n" <jamagallon@....com>,
Andrew Morton <akpm@...l.org>,
"Linux-Kernel," <linux-kernel@...r.kernel.org>,
linux-scsi@...r.kernel.org
Subject: Re: 2.6.18-mm2
Ar Sad, 2006-09-30 am 10:26 -0500, ysgrifennodd James Bottomley:
> On Fri, 2006-09-29 at 23:50 +0000, Frederik Deweerdt wrote:
> > + if (!pdev->irq)
> > + return -ENODEV;
> > +
>
> Don't I remember that 0 is a valid IRQ on some platforms?
>
> i.e. shouldn't this be
>
> if (pdev->irq == NO_IRQ)
> return -ENODEV;
NO_IRQ is gone. Everyone uses zero and Linus has declared that is how it
shall be.
Alan
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists