Jeremy Higdon wrote:
On Tue, Aug 15, 2006 at 05:25:14PM -0700, Wolstenholme, Daniel E wrote:
In the course of some work I'm doing with MSI interrupts, I had an
Intel GD81344 card I wanted to try out with MSI interrupts, and I
found that it does indeed support MSI. Here's a patch to add support.
You sent the patch as html. That's not going to work. Even the text/plain
part of it is quoted/printable with lots of extra blank lines, etc. If
you're using Outlook, the only way to send a patch is to first turn off
html and then send the patch as a text/plain attachment (as far as I know).
I implemented this rather simple patch using #ifdef CONFIG_PCI_MSI. I
noticed some other drivers, such as sata_mv and LSI's fusion drivers,
use a kernel command-line parameter to enable MSI support. I honestly
don't know the advantage to this; it seems to me to be easier if it's
enabled by default, as long as MSI support is selected in the kernel
configuration. If not, someone please let me know.
It might be that some platforms that have MSI enabled won't support it
on certain machines where these chips are installed. So it's probably
best to make it an option. I'll defer to jgarzik on this, however.
There's no need for #ifdefs at all. You call pci_enable_msi()
unconditionally, and if it fails, don't configure/use MSI. From the
driver's perspective, everything is handled with runtime function calls.
Jeff
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
