>-----Original Message----- >From: Karicheri, Muralidharan >Sent: Wednesday, May 21, 2014 6:52 PM >To: jg1....@samsung.com; x...@kosagi.com; devicetree@vger.kernel.org; linux- >d...@vger.kernel.org; linux-ker...@vger.kernel.org; a...@arndb.de >Cc: Karicheri, Muralidharan; Mohit Kumar; Pratyush Anand; Richard Zhu; ABRAHAM, >KISHON VIJAY; Marek Vasut >Subject: [RFC PATCH] pcie-designware: DT documentation update to clarify DT >properties > >Current documentation is not clear enough about the mandatory bindings to be >present in >the device node of a snps,dw-pcie compatible pcie controller. In some cases >the property is >not present in all drivers. >For example pcie_bus is specified as a clock, but only one of the driver uses >it. This patch >attempts to make the documentation consistent with current implementation so >that it is >clear enough for anyone who develops a dw-pcie compatible pcie driver. > >CC: Mohit Kumar <mohit.ku...@st.com> >CC: Jingoo Han <jg1....@samsung.com> >CC: Pratyush Anand <pratyush.an...@st.com> >CC: Richard Zhu <r65...@freescale.com> >CC: Kishon Vijay Abraham I <kis...@ti.com> >CC: Marek Vasut <ma...@denx.de> > >Signed-off-by: Murali Karicheri <m-kariche...@ti.com> >--- > .../devicetree/bindings/pci/designware-pcie.txt | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > >diff --git a/Documentation/devicetree/bindings/pci/designware-pcie.txt >b/Documentation/devicetree/bindings/pci/designware-pcie.txt >index d6fae13..8b9dc52 100644 >--- a/Documentation/devicetree/bindings/pci/designware-pcie.txt >+++ b/Documentation/devicetree/bindings/pci/designware-pcie.txt >@@ -4,12 +4,14 @@ Required properties: > - compatible: should contain "snps,dw-pcie" to identify the > core, plus an identifier for the specific instance, such > as "samsung,exynos5440-pcie" or "fsl,imx6q-pcie". >-- reg: base addresses and lengths of the pcie controller, >- the phy controller, additional register for the phy controller. >+- reg: base addresses and lengths of the pcie controller. >+ index 0 - base address and length of RC's config space. >+ index 1 and above: additional registers for the PCI controller >+ that are specific to an implementation. > - interrupts: interrupt values for level interrupt, > pulse interrupt, special interrupt. >-- clocks: from common clock binding: handle to pci clock. >-- clock-names: from common clock binding: should be "pcie" and "pcie_bus". >+- clocks: from common clock binding: handle to list of pci clock. >+- clock-names: from common clock binding: pci clock names: "pcie" > - #address-cells: set to <3> > - #size-cells: set to <2> > - device_type: set to "pci" >-- >1.7.9.5
+ Adding to the pci list. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
