Re: PATCH: pci_enable_device_mask

From: Jeff Garzik (jgarzik@mandrakesoft.com)
Date: Fri Sep 22 2000 - 20:43:46 EST


Martin Mares wrote:
> > I didn't update all the archs, just x86.. Enjoy. This is not an
> > official patch submission (so Linus if you see it, don't apply...)
>
> Is there any reason for adding such a thing? I was thinking about it some months
> ago and it seemed to me that we always have plenty of space for all the resources.

eh, uh? :) This doesn't have much to do with resource space... We are
talking about having pci_enable_device -not- enable PCI_COMMAND_IO
and/or PCI_COMMAND_MEMORY because some hardware needs to be treated
specially, and blindly enabling IO|MEM is bad news for that hardware.

> Anyway, I'd prefer selecting such options via extra fields in struct pci_dev
> which would the driver set before calling pci_enable_device() -- this way we
> can add new options later without introducing new functions.

Makes sense to me...
-
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:28 EST