From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Murray Subject: Re: pci and pcie device-tree binding - range No cells Date: Wed, 12 Dec 2012 16:44:58 +0000 Message-ID: <20121212164458.GA17425@arm.com> References: <50C5D387.90908@monstr.eu> <50C5F11D.9060006@gmail.com> <50C5FA3E.9030303@monstr.eu> <50C5FE0F.3050108@gmail.com> <20121210214127.D51773E0796@localhost> <50C85E7D.5080006@monstr.eu> <20121212121912.GA2776@arm.com> <20121212133424.GA26280@avionic-0098.adnet.avionic-design.de> Mime-Version: 1.0 Content-Type: text/plain; charset=WINDOWS-1252 Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <20121212133424.GA26280@avionic-0098.adnet.avionic-design.de> Content-Disposition: inline Sender: linux-pci-owner@vger.kernel.org To: Thierry Reding Cc: Grant Likely , Michal Simek , Rob Herring , devicetree-discuss , "linux-pci@vger.kernel.org" , "rob.herring@calxeda.com" , linuxppc-dev , Benjamin Herrenschmidt , Liviu.Dudau@arm.com List-Id: devicetree@vger.kernel.org On Wed, Dec 12, 2012 at 01:34:24PM +0000, Thierry Reding wrote: > On Wed, Dec 12, 2012 at 12:19:12PM +0000, Andrew Murray wrote: > > I've been working on a relatively architecture agnostic PCI host bridge driver > > and also wanted to avoid duplicating more generic DT parsing code for PCI > > bindings. > > > > I've ended up with a patch which provides an iterator for returning resources > > based on the the typical 'ranges' binding. This has ended up living in > > drivers/of/address.c. I originally started out in drivers/of/pci.c and > > drivers/pci/pci-of.c but found there were good (and static) implementations in > > drivers/of/address.c which can be reused (e.g. of_bus_pci_get_flags, > > bus->count_cells). > > > > I'm not just ready to post it - but can do before early next week if you can > > wait. > > I already posted a similar patch[0] as part of a larger series to bring > DT support to Tegra PCIe back in July. I suppose what you have must be > something pretty close to that. Most of the stuff that had me occupied > since then should be done soon and I was planning on resurrecting the > series one of these days. Thanks for the reference. I've submitted my patch, it's along the lines of your existing patch. I'm happy to take the best bits from both, drop mine, etc. Andrew Murray