Message ID | 1487919949-4929-1-git-send-email-Bharat.Bhushan@nxp.com |
---|---|
State | Superseded |
Delegated to: | York Sun |
Headers | show |
On 2/24/2017 10:04 AM, Bharat Bhushan wrote: > This patch fixup iommu-map property on pci node to have > a valid mapping of requester-id to stream-id. > The requester-id to stream-id mapping is based on > PCI-LUT table initialization. > > Signed-off-by: Bharat Bhushan <Bharat.Bhushan@nxp.com> > --- > drivers/pci/pcie_layerscape_fixup.c | 53 +++++++++++++++++++++++++++++++++++++ > 1 file changed, 53 insertions(+) > > diff --git a/drivers/pci/pcie_layerscape_fixup.c b/drivers/pci/pcie_layerscape_fixup.c > index 8f8f6b6..301afee 100644 > --- a/drivers/pci/pcie_layerscape_fixup.c > +++ b/drivers/pci/pcie_layerscape_fixup.c > @@ -103,6 +103,56 @@ static void fdt_pcie_set_msi_map_entry(void *blob, struct ls_pcie *pcie, > fdt_appendprop_u32(blob, nodeoffset, "msi-map", 1); > } > > +/* > + * An iommu-map is a property to be added to the pci controller > + * node. It is a table, where each entry consists of 4 fields > + * e.g.: > + * > + * 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) > +{ > + u32 *prop; > + u32 iommu_map[4]; > + int nodeoffset; > + int lenp; > + > + /* find pci controller node */ > + nodeoffset = fdt_node_offset_by_compat_reg(blob, "fsl,ls-pcie", > + pcie->dbi_res.start); > + if (nodeoffset < 0) { > +#ifdef CONFIG_FSL_PCIE_COMPAT /* Compatible with older version of dts node */ > + nodeoffset = fdt_node_offset_by_compat_reg(blob, > + CONFIG_FSL_PCIE_COMPAT, pcie->dbi_res.start); > + if (nodeoffset < 0) > + return; > +#else > + return; > +#endif > + } > + > + /* get phandle to MSI controller */ I guess that the phandle is to the iommu node, not to the MSI controller. > + prop = fdt_getprop_w(blob, nodeoffset, "iommu-map", &lenp); > + if (prop == NULL) { > + debug("\n%s: ERROR: missing iommu-map: PCIe%d\n", > + __func__, pcie->idx); > + return; > + } > + > + /* set iommu-map row */ > + iommu_map[0] = cpu_to_fdt32(devid); > + iommu_map[1] = *++prop; > + iommu_map[2] = cpu_to_fdt32(streamid); > + iommu_map[3] = cpu_to_fdt32(1); > + > + if (devid == 0) > + fdt_setprop_inplace(blob, nodeoffset, "iommu-map", iommu_map, 16); > + else > + fdt_appendprop(blob, nodeoffset, "iommu-map", iommu_map, 16); > +} > + > static void fdt_fixup_pcie(void *blob) > { > struct udevice *dev, *bus; > @@ -139,6 +189,9 @@ static void fdt_fixup_pcie(void *blob) > /* update msi-map in device tree */ > fdt_pcie_set_msi_map_entry(blob, pcie, bdf >> 8, > streamid); > + /* update iommu-map in device tree */ > + fdt_pcie_set_iommu_map_entry(blob, pcie, bdf >> 8, > + streamid); > } > } > #endif Diana
> -----Original Message----- > From: Diana Madalina Craciun > Sent: Friday, March 10, 2017 8:54 PM > To: Bharat Bhushan <bharat.bhushan@nxp.com>; york sun > <york.sun@nxp.com>; u-boot@lists.denx.de > Cc: Z.Q. Hou <zhiqiang.hou@nxp.com> > Subject: Re: [U-Boot] [PATCH] pcie-layerscape: Fixup iommu-map property > of pci node > > On 2/24/2017 10:04 AM, Bharat Bhushan wrote: > > This patch fixup iommu-map property on pci node to have a valid > > mapping of requester-id to stream-id. > > The requester-id to stream-id mapping is based on PCI-LUT table > > initialization. > > > > Signed-off-by: Bharat Bhushan <Bharat.Bhushan@nxp.com> > > --- > > drivers/pci/pcie_layerscape_fixup.c | 53 > > +++++++++++++++++++++++++++++++++++++ > > 1 file changed, 53 insertions(+) > > > > diff --git a/drivers/pci/pcie_layerscape_fixup.c > > b/drivers/pci/pcie_layerscape_fixup.c > > index 8f8f6b6..301afee 100644 > > --- a/drivers/pci/pcie_layerscape_fixup.c > > +++ b/drivers/pci/pcie_layerscape_fixup.c > > @@ -103,6 +103,56 @@ static void fdt_pcie_set_msi_map_entry(void > *blob, struct ls_pcie *pcie, > > fdt_appendprop_u32(blob, nodeoffset, "msi-map", 1); } > > > > +/* > > + * An iommu-map is a property to be added to the pci controller > > + * node. It is a table, where each entry consists of 4 fields > > + * e.g.: > > + * > > + * 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) > > +{ > > + u32 *prop; > > + u32 iommu_map[4]; > > + int nodeoffset; > > + int lenp; > > + > > + /* find pci controller node */ > > + nodeoffset = fdt_node_offset_by_compat_reg(blob, "fsl,ls-pcie", > > + pcie->dbi_res.start); > > + if (nodeoffset < 0) { > > +#ifdef CONFIG_FSL_PCIE_COMPAT /* Compatible with older version of > dts node */ > > + nodeoffset = fdt_node_offset_by_compat_reg(blob, > > + CONFIG_FSL_PCIE_COMPAT, pcie- > >dbi_res.start); > > + if (nodeoffset < 0) > > + return; > > +#else > > + return; > > +#endif > > + } > > + > > + /* get phandle to MSI controller */ > > I guess that the phandle is to the iommu node, not to the MSI controller. Yes, Thanks I will correct this in next version. Thanks -Bharat > > > + prop = fdt_getprop_w(blob, nodeoffset, "iommu-map", &lenp); > > + if (prop == NULL) { > > + debug("\n%s: ERROR: missing iommu-map: PCIe%d\n", > > + __func__, pcie->idx); > > + return; > > + } > > + > > + /* set iommu-map row */ > > + iommu_map[0] = cpu_to_fdt32(devid); > > + iommu_map[1] = *++prop; > > + iommu_map[2] = cpu_to_fdt32(streamid); > > + iommu_map[3] = cpu_to_fdt32(1); > > + > > + if (devid == 0) > > + fdt_setprop_inplace(blob, nodeoffset, "iommu-map", > iommu_map, 16); > > + else > > + fdt_appendprop(blob, nodeoffset, "iommu-map", > iommu_map, 16); } > > + > > static void fdt_fixup_pcie(void *blob) { > > struct udevice *dev, *bus; > > @@ -139,6 +189,9 @@ static void fdt_fixup_pcie(void *blob) > > /* update msi-map in device tree */ > > fdt_pcie_set_msi_map_entry(blob, pcie, bdf >> 8, > > streamid); > > + /* update iommu-map in device tree */ > > + fdt_pcie_set_iommu_map_entry(blob, pcie, bdf >> 8, > > + streamid); > > } > > } > > #endif > > Diana
diff --git a/drivers/pci/pcie_layerscape_fixup.c b/drivers/pci/pcie_layerscape_fixup.c index 8f8f6b6..301afee 100644 --- a/drivers/pci/pcie_layerscape_fixup.c +++ b/drivers/pci/pcie_layerscape_fixup.c @@ -103,6 +103,56 @@ static void fdt_pcie_set_msi_map_entry(void *blob, struct ls_pcie *pcie, fdt_appendprop_u32(blob, nodeoffset, "msi-map", 1); } +/* + * An iommu-map is a property to be added to the pci controller + * node. It is a table, where each entry consists of 4 fields + * e.g.: + * + * 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) +{ + u32 *prop; + u32 iommu_map[4]; + int nodeoffset; + int lenp; + + /* find pci controller node */ + nodeoffset = fdt_node_offset_by_compat_reg(blob, "fsl,ls-pcie", + pcie->dbi_res.start); + if (nodeoffset < 0) { +#ifdef CONFIG_FSL_PCIE_COMPAT /* Compatible with older version of dts node */ + nodeoffset = fdt_node_offset_by_compat_reg(blob, + CONFIG_FSL_PCIE_COMPAT, pcie->dbi_res.start); + if (nodeoffset < 0) + return; +#else + return; +#endif + } + + /* get phandle to MSI controller */ + prop = fdt_getprop_w(blob, nodeoffset, "iommu-map", &lenp); + if (prop == NULL) { + debug("\n%s: ERROR: missing iommu-map: PCIe%d\n", + __func__, pcie->idx); + return; + } + + /* set iommu-map row */ + iommu_map[0] = cpu_to_fdt32(devid); + iommu_map[1] = *++prop; + iommu_map[2] = cpu_to_fdt32(streamid); + iommu_map[3] = cpu_to_fdt32(1); + + if (devid == 0) + fdt_setprop_inplace(blob, nodeoffset, "iommu-map", iommu_map, 16); + else + fdt_appendprop(blob, nodeoffset, "iommu-map", iommu_map, 16); +} + static void fdt_fixup_pcie(void *blob) { struct udevice *dev, *bus; @@ -139,6 +189,9 @@ static void fdt_fixup_pcie(void *blob) /* update msi-map in device tree */ fdt_pcie_set_msi_map_entry(blob, pcie, bdf >> 8, streamid); + /* update iommu-map in device tree */ + fdt_pcie_set_iommu_map_entry(blob, pcie, bdf >> 8, + streamid); } } #endif
This patch fixup iommu-map property on pci node to have a valid mapping of requester-id to stream-id. The requester-id to stream-id mapping is based on PCI-LUT table initialization. Signed-off-by: Bharat Bhushan <Bharat.Bhushan@nxp.com> --- drivers/pci/pcie_layerscape_fixup.c | 53 +++++++++++++++++++++++++++++++++++++ 1 file changed, 53 insertions(+)