From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alan Cox Subject: Re: 2.6.18-mm2 Date: Sat, 30 Sep 2006 21:54:19 +0100 Message-ID: <1159649659.13029.149.camel@localhost.localdomain> References: <20060928014623.ccc9b885.akpm@osdl.org> <20060929155738.7076f0c8@werewolf> <20060929143949.GL5017@parisc-linux.org> <1159550143.13029.36.camel@localhost.localdomain> <20060929235054.GB2020@slug> <1159629982.14918.4.camel@mulgrave.il.steeleye.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from outpipe-village-512-1.bc.nu ([81.2.110.250]:5538 "EHLO lxorguk.ukuu.org.uk") by vger.kernel.org with ESMTP id S1751935AbWI3U3W (ORCPT ); Sat, 30 Sep 2006 16:29:22 -0400 In-Reply-To: <1159629982.14918.4.camel@mulgrave.il.steeleye.com> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: James Bottomley Cc: Frederik Deweerdt , Matthew Wilcox , "J.A. Magall??n" , Andrew Morton , "Linux-Kernel," , linux-scsi@vger.kernel.org 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