On Wed, Oct 09, 2013 at 10:09:23AM +0100, Liviu Dudau wrote: > On Tue, Oct 08, 2013 at 09:28:55PM +0100, Benjamin Herrenschmidt wrote: > > On Tue, 2013-10-08 at 11:55 -0500, Rob Herring wrote: > > > > > > I wonder if pci_process_bridge_OF_ranges() would fit somewhere in > > > > drivers/of? The implementations I looked at are mostly concerned with > > > > parsing OF resources, and they don't have much to do with PCI > > > > directly. > > > > > > This was being done until Ben weighed in: > > > > > > https://lkml.org/lkml/2013/5/4/103 > > > > Well, I proposed an alternative (better) approach which I of course had > > no time to actually implement yet :-) > > In order to avoid any confusion, could you please point me again to the > relevant message(s) where you proposed your approach? Sorry, I'm being sloppy. I've made the wrong assumption that the link listed above was only Ben talking about why it cannot get into powerpc. Now I see that it is also the place where Ben talks about moving the "intermediary" set of resources out of struct pci_controller. Is there anything else that I might be missing? Cheers, Liviu > > > > > I have done the changes I needed to do to powerpc > > pci_process_bridge_OF_ranges so it would be possible to move that now to > > a generic place, but I still think it's not a great idea. It means the > > pci_controller structure with its resources will have to become generic > > which somewhat overlaps with the pci_host_bridge that Bjorn introduced, > > so that's really not great. > > > > I still think an arch with DT and simpler PCI code that powerpc could > > start looking at the transition to a better model that I hinted at... > > As tempting as it is to start anew and with a cleaner code, I am wary > that porting the existing platforms to the new code will take longer > that way. My intentions are to make the (probably infrequent) task of > adding a new architecture to the PCI infrastructure a simple and > straighforward task. But adding code for my platform is no guarantee > that new ones will have an easier job. > > Best regards, > Liviu > > > > > Cheers, > > Ben. > > > > > > -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯ -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
