Re: [RFC PATCH] PCI: remove access to pci_[enable|disable]_msi() for drivers - take 2

Jeff Garzik
2005-06-08T06:15:37Z

Andrew Grover wrote:
> On 6/7/05, Jeff Garzik <jgarzik@pobox.com> wrote:
>>Also, it looks like all the PCI MSI drivers need touching for this
>>scheme -- which defeats the original intention. At this rate, the best
>>API is the one we've already got.
>
>
> For now...but I'm bringing this up again in five years!! *sets egg timer*

Re-read the start of the thread :)

My suggestion...

short term: do nothing

long term: move drivers to a new pci_enable()/pci_disable() API which
makes it easy for us to roll a lot of these singleton function calls,
repeated over and over again in PCI drivers, into generic code.

Then just let evolution happen. That way, progress occurs, but no
existing drivers are broken by a sudden pci_enable_device() behavior change.

Since we're in a "rolling stable series" that might be a better path to
take. If evolution goes as expected, maybe there will no longer be any
public users of pci_enable_msi()...

Jeff