Hi Sergei,

On: 27/03/2014 11:08, Sergei wrote:

Subject: Re: [PATCH v6 05/10] dt-bindings: pci: rcar pcie device tree 

bindings

Hello.

On 27-03-2014 13:26, Phil Edworthy wrote:

quoted

This patch adds the bindings for the R-Car PCIe driver. The driver
resides under drivers/pci/host/pcie-rcar.c

quoted

Signed-off-by: Phil Edworthy <redacted>

quoted

v6:
  - Correct DT bindings description for reg and clocks

quoted

v5:
  - Add PCIe bus clock reference
  - Add additional interrupt bindings
  - Use dma-ranges property to specify inbound memory regions

    Put these under --- tear line, please.

quoted

---
  Documentation/devicetree/bindings/pci/rcar-pci.txt | 45 

++++++++++++++++++++++

quoted

  1 file changed, 45 insertions(+)
  create mode 100644 

Documentation/devicetree/bindings/pci/rcar-pci.txt

quoted

diff --git a/Documentation/devicetree/bindings/pci/rcar-pci.txt b/

Documentation/devicetree/bindings/pci/rcar-pci.txt

quoted

new file mode 100644
index 0000000..a97ebe6

--- /dev/null
+++ b/Documentation/devicetree/bindings/pci/rcar-pci.txt

@@ -0,0 +1,45 @@
+* Renesas RCar PCIe interface
+
+Required properties:
+- compatible: should contain one of the following
+   "renesas,pcie-r8a7779", "renesas,pcie-r8a7790", 

"renesas,pcie-r8a7791"

quoted

+- reg: base address and length of the pcie controller registers.
+- #address-cells: set to <3>
+- #size-cells: set to <2>
+- device_type: set to "pci"
+- ranges: ranges for the PCI memory and I/O regions.
+- dma-ranges: ranges for the inbound memory regions.
+- interrupts: two interrupt sources for MSI interrupts, followed by 

interrupt

quoted

+   source for hardware related interrupts (e.g. link speed change).
+- #interrupt-cells: set to <1>
+- interrupt-map-mask and interrupt-map: standard PCI properties
+   to define the mapping of the PCIe interface to interrupt
+   numbers.
+- clocks: from common clock binding: phandles to PCIe controller and 

PCIe bus

quoted

+   clocks.

    I told it's not only phandles (e.g. &mstp3_clks) but the clock 

specifiers.
Ah, yes you did.

Thanks
Phil