[U-Boot] [PATCH 3/8] pci: layerscape: Update API names for layerscape fixup

Wasim Khan wasim.khan at nxp.com
Mon Oct 21 06:19:34 UTC 2019


Hi Priyanka,

> -----Original Message-----
> From: Priyanka Jain
> Sent: Monday, October 21, 2019 11:13 AM
> To: Wasim Khan <wasim.khan at nxp.com>; Z.q. Hou <zhiqiang.hou at nxp.com>
> Cc: u-boot at lists.denx.de
> Subject: RE: [PATCH 3/8] pci: layerscape: Update API names for layerscape fixup 
> 
> 
> >-----Original Message-----
> >From: Wasim Khan
> >Sent: Friday, October 18, 2019 7:38 PM
> >To: Priyanka Jain <priyanka.jain at nxp.com>
> >Cc: u-boot at lists.denx.de; Wasim Khan <wasim.khan at nxp.com>
> >Subject: [PATCH 3/8] pci: layerscape: Update API names for layerscape
> >fixup
> >
> >Update API names for layerscape fixup.
> Better description woul be suffixing API name with _ls Why you need this
> change?

OK. Will update the description.
We have organized the code into common DT fixup and layerscape/layerscape_gen4 specific fixup.

> >
> >Signed-off-by: Wasim Khan <wasim.khan at nxp.com>
> >---
> > drivers/pci/pcie_layerscape_fixup.c | 22 +++++++++++-----------
> > 1 file changed, 11 insertions(+), 11 deletions(-)
> >
> >diff --git a/drivers/pci/pcie_layerscape_fixup.c
> >b/drivers/pci/pcie_layerscape_fixup.c
> >index 089e031..e0dcd97 100644
> >--- a/drivers/pci/pcie_layerscape_fixup.c
> >+++ b/drivers/pci/pcie_layerscape_fixup.c
> >@@ -1,6 +1,6 @@
> > // SPDX-License-Identifier: GPL-2.0+
> > /*
> >- * Copyright 2017 NXP
> >+ * Copyright 2017, 2019 NXP
> 2017-2019
> >  * Copyright 2014-2015 Freescale Semiconductor, Inc.
> >  * Layerscape PCIe driver
> >  */
> >@@ -69,8 +69,8 @@ static void ls_pcie_lut_set_mapping(struct ls_pcie
> >*pcie, int index, u32 devid,
> >  *      msi-map = <[devid] [phandle-to-msi-ctrl] [stream-id] [count]
> >  *                 [devid] [phandle-to-msi-ctrl] [stream-id] [count]>;
> >  */
> >-static void fdt_pcie_set_msi_map_entry(void *blob, struct ls_pcie *pcie,
> >-				       u32 devid, u32 streamid)
> >+static void fdt_pcie_set_msi_map_entry_ls(void *blob, struct ls_pcie *pcie,
> >+					  u32 devid, u32 streamid)
> > {
> > 	u32 *prop;
> > 	u32 phandle;
> >@@ -122,8 +122,8 @@ static void fdt_pcie_set_msi_map_entry(void *blob,
> >struct ls_pcie *pcie,
> >  *      iommu-map = <[devid] [phandle-to-iommu-ctrl] [stream-id] [count]
> >  *                 [devid] [phandle-to-iommu-ctrl] [stream-id] [count]>;
> >  */
> >-static void fdt_pcie_set_iommu_map_entry(void *blob, struct ls_pcie *pcie,
> >-				       u32 devid, u32 streamid)
> >+static void fdt_pcie_set_iommu_map_entry_ls(void *blob, struct ls_pcie
> *pcie,
> >+					    u32 devid, u32 streamid)
> > {
> > 	u32 *prop;
> > 	u32 iommu_map[4];
> >@@ -175,7 +175,7 @@ static void fdt_pcie_set_iommu_map_entry(void
> >*blob, struct ls_pcie *pcie,
> > 	}
> > }
> >
> >-static void fdt_fixup_pcie(void *blob)
> >+static void fdt_fixup_pcie_ls(void *blob)
> > {
> > 	struct udevice *dev, *bus;
> > 	struct ls_pcie *pcie;
> >@@ -209,11 +209,11 @@ static void fdt_fixup_pcie(void *blob)
> > 		ls_pcie_lut_set_mapping(pcie, index, bdf >> 8,
> > 					streamid);
> > 		/* update msi-map in device tree */
> >-		fdt_pcie_set_msi_map_entry(blob, pcie, bdf >> 8,
> >-					   streamid);
> >+		fdt_pcie_set_msi_map_entry_ls(blob, pcie, bdf >> 8,
> >+					      streamid);
> > 		/* update iommu-map in device tree */
> >-		fdt_pcie_set_iommu_map_entry(blob, pcie, bdf >> 8,
> >-					     streamid);
> >+		fdt_pcie_set_iommu_map_entry_ls(blob, pcie, bdf >> 8,
> >+						streamid);
> > 	}
> > }
> > #endif
> >@@ -279,7 +279,7 @@ void ft_pci_setup(void *blob, bd_t *bd)
> > 		ft_pcie_ls_setup(blob, pcie);
> >
> > #if defined(CONFIG_FSL_LSCH3) || defined(CONFIG_FSL_LSCH2)
> >-	fdt_fixup_pcie(blob);
> >+	fdt_fixup_pcie_ls(blob);
> > #endif
> > }
> >
> >--
> >2.7.4
> 
> --priyankajain


More information about the U-Boot mailing list