Re: [BKPATCH] allow pci primary peer busses to have parents

Ivan Kokshaysky (ink@jurassic.park.msu.ru)
Thu, 5 Dec 2002 16:12:05 +0300


On Wed, Dec 04, 2002 at 11:18:24AM -0600, James Bottomley wrote:
> Now that the generic device model allows a coherent bus tree to be built

Unfortunately it doesn't. Currently those legacy, PnP, EISA devices all have
virtual parents, which has nothing to do with reality. Modern systems
(including most PCs) hang these buses off PCI bus using PCI-to-{E}ISA
bridge. Such systems must be able to register these buses upon
discovery of the ISA bridges (from pci layer), and use them as a
parent device for legacy/isa/pnp stuff. This will be absolutely required
if DMA operations are moved from pci_dev to the generic device.

> -struct pci_bus * __devinit pci_scan_bus(int bus, struct pci_ops *ops, void
> *sysdata)
> +struct pci_bus * __devinit pci_scan_bus_parented(struct device *parent, int
> bus, struct pci_ops *ops, void *sysdata)
> {
> - struct pci_bus *b = pci_alloc_primary_bus(bus);
> + struct pci_bus *b = pci_alloc_primary_bus_parented(parent, bus);
> if (b) {
> b->sysdata = sysdata;
> b->ops = ops;

The `sysdata' arg already contains info about parent host-to-pci controller
on many platforms. I don't think that we need to duplicate it with
another one.
I was thinking about something like this instead of `sysdata':

struct io_controller { /* Level 0 I/O controller */
... arch specific fields ...;
... generic fields ...; /* like `index' */
struct device dev;
}

Ivan.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/