Hi Rob,
On 2018/2/10 7:38, Rob Herring wrote:
On Wed, Feb 07, 2018 at 10:47:28AM +0800, Shawn Lin wrote:
To avoid confusion when folks use this controller as endponint
s/endponint/endpoint/
mode, we add rockchip,rk3399-pcie-host to clearly state that
it's driver for host here. And rename/update the document to
reflect this fact.
Signed-off-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx>
---
.../bindings/pci/{rockchip-pcie.txt => rockchip-pcie-host.txt} | 3 ++-
drivers/pci/rockchip/pcie-rockchip-host.c | 1 +
2 files changed, 3 insertions(+), 1 deletion(-)
rename Documentation/devicetree/bindings/pci/{rockchip-pcie.txt => rockchip-pcie-host.txt} (98%)
diff --git a/Documentation/devicetree/bindings/pci/rockchip-pcie.txt b/Documentation/devicetree/bindings/pci/rockchip-pcie-host.txt
similarity index 98%
rename from Documentation/devicetree/bindings/pci/rockchip-pcie.txt
rename to Documentation/devicetree/bindings/pci/rockchip-pcie-host.txt
index af34c65..f2f3091 100644
--- a/Documentation/devicetree/bindings/pci/rockchip-pcie.txt
+++ b/Documentation/devicetree/bindings/pci/rockchip-pcie-host.txt
@@ -5,7 +5,8 @@ Required properties:
- #size-cells: Size representation for root ports, set to <2>
- #interrupt-cells: specifies the number of cells needed to encode an
interrupt source. The value must be 1.
-- compatible: Should contain "rockchip,rk3399-pcie"
+- compatible: Should contain "rockchip,rk3399-pcie" or
+ "rockchip,rk3399-pcie-host"
I don't think this is necessary. Endpoint is generally the exception and
having "endpoint" or "ep" in the endpoint compatible should be enough.
I was following the convention of other controllers that support
both of RC mode and EP mode, for instance, pcie-cadence-{host, ep}.txt
Sure, I agree that endpoint is generally the exception, so will drop
this in v2.
- reg: Two register ranges as listed in the reg-names property
- reg-names: Must include the following names
- "axi-base"
diff --git a/drivers/pci/rockchip/pcie-rockchip-host.c b/drivers/pci/rockchip/pcie-rockchip-host.c
index f5b0492..efb740d 100644
--- a/drivers/pci/rockchip/pcie-rockchip-host.c
+++ b/drivers/pci/rockchip/pcie-rockchip-host.c
@@ -1120,6 +1120,7 @@ static int rockchip_pcie_remove(struct platform_device *pdev)
static const struct of_device_id rockchip_pcie_of_match[] = {
{ .compatible = "rockchip,rk3399-pcie", },
+ { .compatible = "rockchip,rk3399-pcie-host", },
{}
};
MODULE_DEVICE_TABLE(of, rockchip_pcie_of_match);
--
1.9.1