| Message ID | 20180921102155.22839-14-kishon@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
| Series | Cleanup pci-keystone.c and Add AM654 PCIe Support | expand |
Hi Rob, On 25/09/18 4:30 AM, Rob Herring wrote: > On Fri, Sep 21, 2018 at 03:51:28PM +0530, Kishon Vijay Abraham I wrote: >> Add "reg-names" binding information in order for device tree node >> to be populated with the correct register strings. >> >> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> >> --- >> Documentation/devicetree/bindings/pci/pci-keystone.txt | 6 ++++-- >> 1 file changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/pci/pci-keystone.txt b/Documentation/devicetree/bindings/pci/pci-keystone.txt >> index 2030ee0dc4f9..3a551687cfa2 100644 >> --- a/Documentation/devicetree/bindings/pci/pci-keystone.txt >> +++ b/Documentation/devicetree/bindings/pci/pci-keystone.txt >> @@ -12,8 +12,10 @@ described here as well as properties that are not applicable. >> Required Properties:- >> >> compatibility: "ti,keystone-pcie" >> -reg: index 1 is the base address and length of DW application registers. >> - index 2 is the base address and length of PCI device ID register. >> +reg: Three register ranges as listed in the reg-names property >> +reg-names: "dbics" for the DesignWare PCIe registers, "app" for the >> + TI specific application registers, "config" for the >> + configuration space address > > This doesn't doesn't look like a compatible change. The pcie-keystone driver hasn't worked in mainline because of lack of serdes support. Since the same driver is used for am654 SoC (which has serdes support included in this series), I'm trying to cleanup the binding. Thanks Kishon
diff --git a/Documentation/devicetree/bindings/pci/pci-keystone.txt b/Documentation/devicetree/bindings/pci/pci-keystone.txt index 2030ee0dc4f9..3a551687cfa2 100644 --- a/Documentation/devicetree/bindings/pci/pci-keystone.txt +++ b/Documentation/devicetree/bindings/pci/pci-keystone.txt @@ -12,8 +12,10 @@ described here as well as properties that are not applicable. Required Properties:- compatibility: "ti,keystone-pcie" -reg: index 1 is the base address and length of DW application registers. - index 2 is the base address and length of PCI device ID register. +reg: Three register ranges as listed in the reg-names property +reg-names: "dbics" for the DesignWare PCIe registers, "app" for the + TI specific application registers, "config" for the + configuration space address pcie_msi_intc : Interrupt controller device node for MSI IRQ chip interrupt-cells: should be set to 1
Add "reg-names" binding information in order for device tree node to be populated with the correct register strings. Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com> --- Documentation/devicetree/bindings/pci/pci-keystone.txt | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)