From: Kumar Gala <galak@codeaurora.org>
To: Lucas Stach <l.stach@pengutronix.de>
Cc: Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
devicetree@vger.kernel.org, Bjorn Helgaas <bhelgaas@google.com>,
linux-pci@vger.kernel.org, Liviu Dudau <Liviu.Dudau@arm.com>
Subject: Re: [PATCH] PCI: add missing DT binding for linux,pci-domain property
Date: Tue, 4 Nov 2014 11:05:39 -0600 [thread overview]
Message-ID: <560410DB-26D4-4921-BFEA-176A05DA06CB@codeaurora.org> (raw)
In-Reply-To: <1415101660-26450-1-git-send-email-l.stach@pengutronix.de>
On Nov 4, 2014, at 5:47 AM, Lucas Stach <l.stach@pengutronix.de> wrote:
> This property was added by 41e5c0f81d3e
> (of/pci: Add pci_get_new_domain_nr() and of_get_pci_domain_nr())
> without the required binding documentation. As this property
> will be supported by a number of host bridge drivers going forward,
> add it to the common PCI binding doc.
>
> Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
> ---
> This is a non-critical fix, but may still qualify for 3.18-rc as
> the property was added in this release cycle.
> ---
> Documentation/devicetree/bindings/pci/pci.txt | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/pci/pci.txt b/Documentation/devicetree/bindings/pci/pci.txt
> index 41aeed38926d..b754f786ed5e 100644
> --- a/Documentation/devicetree/bindings/pci/pci.txt
> +++ b/Documentation/devicetree/bindings/pci/pci.txt
> @@ -7,3 +7,13 @@ And for the interrupt mapping part:
>
> Open Firmware Recommended Practice: Interrupt Mapping
> http://www.openfirmware.org/1275/practice/imap/imap0_9d.pdf
> +
> +Additionally to the properties specified in the above standards a host bridge
> +driver implementation may support the following properties:
> +
> +- linux,pci-domain:
> + If present this property assigns a fixed PCI domain number to a host bridge,
> + otherwise an unstable (across boots) unique number will be assigned.
> + It is recommended to either not set this property at all or set it for all
> + host bridges in the system, otherwise potentially conflicting domain numbers
> + may be assigned to root buses behind different host bridges.
Probably should add something about domain numbers being unique across all host bridgse.
- k
--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
next prev parent reply other threads:[~2014-11-04 17:05 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-04 11:47 [PATCH] PCI: add missing DT binding for linux,pci-domain property Lucas Stach
2014-11-04 12:00 ` Liviu Dudau
2014-11-04 12:07 ` Arnd Bergmann
2014-11-04 12:20 ` Lucas Stach
2014-11-06 15:08 ` Lucas Stach
2014-11-06 19:55 ` Arnd Bergmann
2014-11-04 17:05 ` Kumar Gala [this message]
2014-11-05 23:17 ` Bjorn Helgaas
2014-11-06 10:05 ` Liviu Dudau
2014-11-06 11:42 ` Arnd Bergmann
2014-11-06 12:36 ` Liviu Dudau
2014-11-06 13:29 ` Arnd Bergmann
2014-11-06 14:57 ` Rob Herring
2014-11-06 15:30 ` Liviu Dudau
2014-11-06 19:46 ` Rob Herring
2014-11-07 10:17 ` Liviu Dudau
2014-11-07 14:00 ` Rob Herring
2014-11-07 15:23 ` Liviu Dudau
2014-11-07 15:37 ` Rob Herring
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=560410DB-26D4-4921-BFEA-176A05DA06CB@codeaurora.org \
--to=galak@codeaurora.org \
--cc=Liviu.Dudau@arm.com \
--cc=bhelgaas@google.com \
--cc=devicetree@vger.kernel.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=l.stach@pengutronix.de \
--cc=linux-pci@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox