Messages in this thread | | | Subject | Re: [PATCH 07/11] x86/dtb: add support for PCI devices backed by dtb nodes | From | Benjamin Herrenschmidt <> | Date | Mon, 29 Nov 2010 09:32:06 +1100 |
| |
On Sun, 2010-11-28 at 15:04 +0100, Sebastian Andrzej Siewior wrote: > Microblaze had its own copy of this code so I though there is > something > specific about it. If it is okay with you, I would move it to > drivers/of > and share. Then I would have the swizzle part :)
Appart from the accessor pci_device_to_OF_node() which might or might not be specific, I thin the code is pretty common, probably something Grant didn't have time to tackle yet :-)
> >Eventually we want that matching between PCI devices and OF nodes to > be > >in generic code, so that's not a big deal to have an "inferior" > version > >temporarily in there I suppose. > > Inferior, I see :)
Hehe yeah :-) It's actually not a simple problem. For example, we can't just move the powerpc variant over to generic code as-is bcs ... we have 2 completely different ways of doing it between ppc32 and ppc64 for historical reasons :-) They also have different "features". This is something I need to reconcile at some stage.
For example our ppc32 variant support bus renumbering (ie, Linux assigning different bus numbers than what the DT encodes) while our ppc64 doesn't, but our ppc64 variant has additional "stuff" to deal with hotplug for example etc...
> >Also, aren't you missing a pci_dev_put() after pci_get_slot() ? > probably. I will check.
Cheers, Ben.
| |