[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20181017135323.GA31980@bogus>
Date: Wed, 17 Oct 2018 08:53:23 -0500
From: Rob Herring <robh@...nel.org>
To: Zhiqiang Hou <Zhiqiang.Hou@....com>
Cc: linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
linux-pci@...r.kernel.org, devicetree@...r.kernel.org,
leoyang.li@....com, shawnguo@...nel.org, mark.rutland@....com,
bhelgaas@...gle.com, Mingkai.Hu@....com, Minghuan.Lian@....com
Subject: Re: [PATCH 2/4] doc/layerscape-pci: removed unsuitable compatible
string
On Mon, Oct 08, 2018 at 11:14:26AM +0800, Zhiqiang Hou wrote:
> From: Hou Zhiqiang <Zhiqiang.Hou@....com>
Similar comments on this one too.
>
> Removed the compatible string "snps,dw-pcie", it is for the reference
> platform driver for PCI RC IP Protoyping Kits based on the ARC SDP,
> so it is not suitable for all platform with designware PCIe controller,
> and platform vendors have themselves' drivers.
>
> The compatible string "snsp,dw-pcie" was added by mistake and it's not
> matched that time, but it is matched because pcie drivers has been
> collected recently.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@....com>
> ---
> Documentation/devicetree/bindings/pci/layerscape-pci.txt | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/pci/layerscape-pci.txt b/Documentation/devicetree/bindings/pci/layerscape-pci.txt
> index 5eb1c202932f..9b2b8d66d1f4 100644
> --- a/Documentation/devicetree/bindings/pci/layerscape-pci.txt
> +++ b/Documentation/devicetree/bindings/pci/layerscape-pci.txt
> @@ -13,8 +13,8 @@ information.
>
> Required properties:
> - compatible: should contain the platform identifier such as:
> - "fsl,ls1021a-pcie", "snps,dw-pcie"
> - "fsl,ls2080a-pcie", "fsl,ls2085a-pcie", "snps,dw-pcie"
> + "fsl,ls1021a-pcie"
> + "fsl,ls2080a-pcie", "fsl,ls2085a-pcie"
> "fsl,ls2088a-pcie"
> "fsl,ls1088a-pcie"
> "fsl,ls1046a-pcie"
> @@ -36,7 +36,7 @@ Required properties:
> Example:
>
> pcie@...0000 {
> - compatible = "fsl,ls1021a-pcie", "snps,dw-pcie";
> + compatible = "fsl,ls1021a-pcie";
> reg = <0x00 0x03400000 0x0 0x00010000 /* controller registers */
> 0x40 0x00000000 0x0 0x00002000>; /* configuration space */
> reg-names = "regs", "config";
> --
> 2.17.1
>
Powered by blists - more mailing lists