Re: broken reverse_scan in aic7xxx

[Matthew Wilcox <matthew@wil.cx>]

On Sat, Jul 08, 2006 at 12:37:52PM -0600, Matthew Wilcox wrote:
> On Sat, Jul 08, 2006 at 06:48:44PM +0100, Christoph Hellwig wrote:
> > Or you could submit a patch to implement a reverse scanning option in
> > the core PCI code..
> 
> You'd want to specify that on a per-driver basis, though, surely.  Or
> all your ethernet cards would get reversed too.  Oh, but it wouldn't be
> enough for sym2 (iterate through card types instead of through pci ids),
> so you'd want another option for that.

Actually, it's even worse.  PCI doesn't do its own matching any more;
it's now part of the driver core ... so the flag/alternate
implementation has to go all through the callchain:

pci_register_driver
  __pci_register_driver
    driver_register
      bus_add_driver
        driver_attach

At this point, you'd want to change/replace the bus_for_each_dev()
macro.  Wouldn't be too hard to have a bus_for_each_dev_rev() to give
aic7xxx backward compatibility, but sym2 is completely SOL.  The real
pain is passing the information that we want to do this in reverse
all the way down the call chain.