On Mon, 18 Sep 2000, Alan Cox wrote:
> > All I wanted was a function that allows the driver to decide that which
> > needs to be enabled.
> >
> > pci_enable_device(struct pci_dev *dev, byte enable_mask)
> >
> > This would allow drivers to enable that which it needs and not weird out
> > the hardware that does not like all of this extra fluff.
>
> Sounds not too daft
>
> static inline int pci_enable_device(struct pci_device *dev)
> {
> return pci_enable_device_features(USE_IO|USE_MM);
> }
Should be worded as "Respond to IO", "Respond to Memory" transactions,
given the explicit PCI context.
And what about other features ?
I mean:
- Bus Master
- Memory Write and Invalidate
- Parity Error response
Etc ...
Are they considered as misfeatures ? ;-)
> and then just go and turn the existing enable_device into enable_device_Features ?
Gérard.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sat Sep 23 2000 - 21:00:18 EST