Thread (41 messages) 41 messages, 5 authors, 2014-09-23

[PATCH v11 07/10] OF: Introduce helper function for getting PCI domain_nr

From: Liviu.Dudau@arm.com (Liviu Dudau)
Date: 2014-09-22 15:33:19
Also in: linux-arch, linux-devicetree, linux-pci, lkml

On Mon, Sep 22, 2014 at 04:25:13PM +0100, Bjorn Helgaas wrote:
On Mon, Sep 22, 2014 at 5:05 AM, Liviu Dudau [off-list ref] wrote:
quoted
On Fri, Sep 19, 2014 at 10:03:13PM +0100, Bjorn Helgaas wrote:
quoted
On Thu, Sep 18, 2014 at 02:30:22AM +0100, Liviu Dudau wrote:
quoted
Add of_pci_get_domain_nr() to retrieve the PCI domain number
of a given device from DT. If the information is not present,
the function can be requested to allocate a new domain number.
Is of_pci_get_domain_nr() used somewhere?  If the use is in some future
series, please mention it explicitly.  I'm just trying to avoid merging
unused code.
It is used in the arm64 specific patch that I have dropped out of my
pull request. After discussions with Catalin I will add the patch back
into the tree that you've pulled from as he is OK with your tree carrying
the whole package.

I need to ask for some guidance here: for addressing some of your comments
and Rob's I can add more patches in my v11 branch and you can pull them
when you think they are ready. But one of your comments was requesting
splitting a patch into two blocks - one that moves of_pci_range_to_resource()
into drivers/of/address.c and one that fixes it's behaviour - and I don't
know how you would like that handled. Should I revert the original patch
and add the new ones, or should I rebase the whole series into a different
branch that you can pull from?
I guess the easiest thing is probably just to send a v12 series.  I
was hoping we were close enough for me to just hand-integrate minor
tweaks into my branch, but I think that will just create more
confusion.
OK, I will send v12 then with all the acquired ACKs and fixes.

Best regards,
Liviu
Bjorn
-- 
====================
| I would like to |
| fix the world,  |
| but they're not |
| giving me the   |
 \ source code!  /
  ---------------
    ?\_(?)_/?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help