To: Alan Kilian <kilian@bobodyne.com>
Cc: Peter Chubb <peterc@gelato.unsw.edu.au>, linux-kernel@vger.kernel.org
Subject: Re: Help enabling PCI interrupts on Dell/SMP and Sun/SMP systems.
References: <1109190273.9116.307.camel@desk>
	<Pine.LNX.4.61.0502231538230.5623@chaos.analogic.com>
	<1109197066.9116.319.camel@desk>
	<16925.2739.232237.418632@wombat.chubb.wattle.id.au>
	<1109201098.9116.330.camel@desk>
From: Roland Dreier <roland@topspin.com>
Date: Wed, 23 Feb 2005 15:46:54 -0800
In-Reply-To: <1109201098.9116.330.camel@desk> (Alan Kilian's message of
 "Wed, 23 Feb 2005 17:24:58 -0600")
Message-ID: <52sm3malg1.fsf@topspin.com>
User-Agent: Gnus/5.1006 (Gnus v5.10.6) XEmacs/21.4 (Jumbo Shrimp, linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 889
Lines: 23

    Alan> I ask the card which interrupt line it was given at
    Alan> boot-time:

    Alan> 	pci_read_config_byte(dev, PCI_INTERRUPT_LINE, &softp->interrupt_line);

    Alan> Then I request an IRQ:

    Alan> 	request_irq(softp->interrupt_line, sseintr, SA_INTERRUPT, "sse", softp);

Don't do that.  The kernel may need you to use a different interrupt
number than you read from the PCI config header.  Use dev->irq, as in

	request_irq(dev->irq, sseintr, SA_SHIRQ | SA_INTERRUPT, "sse", softp);

Also, make sure that you do pci_enable_device(dev) before you look at
dev->irq, since the field will not be initialized until you do that.

 - R.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/