On Wed, Nov 08, 2000 at 10:56:23AM -0500, Jeff Garzik wrote:
> Setting bit 1 in dev->resource[x].start, below, seems incorrect. Should
> you be programming the PCI BAR directly, instead?
No, that's the reason this is a quirk. The hardware is already
only responding to one and only one address. The old code did
exactly the same thing, only not inside the quirk framework,
which made it kinda harder to figure out what was going on.
> I wonder about this code:
>
> > + /* ??? Reserve some resources for CardBus */
> > + if (dev->class >> 8 == PCI_CLASS_BRIDGE_CARDBUS) {
> > + io_reserved += 8*1024;
> > + mem_reserved += 32*1024*1024;
> > + continue;
> > + }
Got a better suggestion? It does seem completely reasonable to
reserve some address space for a CardBus bridge if we find one.
r~