[PATCH v1] pci: layerscape: bind LX2160A rev2 aligned with Linux kernel

[PATCH v1] pci: layerscape: bind LX2160A rev2 aligned with Linux kernel

[Vincent Jardin]

Linux describes the PCIe controllers of the LX2160A rev2, and of the
LX2162A, as a DesignWare root complexe: "fsl,lx2160ar2-pcie"
Since both commits
  1a1bf58897d2 ("dt-bindings: PCI: layerscape-pci: Replace fsl,lx2160a-pcie with fsl,lx2160ar2-pcie")
  a8fe6c8dfc40 ("arm64: dts: fsl-lx2160a: add rev2 support")
in v6.13

moved to fsl-lx216xa-pcie-gen3.dtsi by
  8f69555ef54c ("arm64: dts: lx2160a: extract the PCIe from LX2160A Rev2 into a separate file")
in v7.3.

Then the U-Boot's dts/upstream has been imported.

-> let's match "fsl,lx2160ar2-pcie"

Then, LX2160A rev2 boards can use the kernel DTS.

Signed-off-by: Vincent Jardin <vjardin@free.fr>

---

 drivers/pci/pcie_layerscape_rc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pci/pcie_layerscape_rc.c b/drivers/pci/pcie_layerscape_rc.c
index e7913d43a8b..b95c989d25a 100644
--- a/drivers/pci/pcie_layerscape_rc.c
+++ b/drivers/pci/pcie_layerscape_rc.c
@@ -403,6 +403,7 @@ static const struct udevice_id ls_pcie_ids[] = {
 	{ .compatible = "fsl,ls-pcie" },
 	{ .compatible = "fsl,ls1028a-pcie", .data = (ulong)&ls1028a_drvdata },
 	{ .compatible = "fsl,ls1088a-pcie", .data = (ulong)&ls1028a_drvdata },
+	{ .compatible = "fsl,lx2160ar2-pcie", .data = (ulong)&ls1028a_drvdata },
 	{ }
 };
 
---
base-commit: c267fa81d42b5a5bd1055d3858f74496faa6082f
branch: for-upstream/pci-layerscape-lx2160ar2-v1

-- 
2.43.0

Re: [PATCH v1] pci: layerscape: bind LX2160A rev2 aligned with Linux kernel

[Vincent Jardin]

Hi Peng, Tom,

I am not used with uboot release cycle/policy, for such small
case:

> +++ b/drivers/pci/pcie_layerscape_rc.c
> +	{ .compatible = "fsl,lx2160ar2-pcie", .data = (ulong)&ls1028a_drvdata },

since I posted this after the -rc1, should it wait for next (v2027.04), or
could it be considered for v2027.01 ?

It only lets uboot bind the LX2160A rev2 PCIe nodes of the upstream
DTS, and no in-tree board is affected, so I hope it could be acceptable.

Thank you for sharing your view on it,
  Vincent