On 2017/10/6 22:04, Shameer Kolothum wrote:
> The HiSilicon erratum 161010801 describes the limitation of
> HiSilicon platforms hip06/hip07 to support the SMMUv3 mappings
> for MSI transactions.
> 
> PCIe controller on these platforms has to differentiate the MSI
> payload against other DMA payload and has to modify the MSI
> payload. This basically makes it difficult for this platforms to
> have a SMMU translation for MSI. In order to workaround this, ARM
> SMMUv3 driver requires a quirk to treat the MSI regions separately.
> Such a quirk is currently missing for DT based systems and therefore
> we need to blacklist the hip06/hip07 PCIe controllers.
> 
> Signed-off-by: Shameer Kolothum <shameerali.kolothum.th...@huawei.com>
> ---
>  drivers/pci/dwc/pcie-hisi.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/drivers/pci/dwc/pcie-hisi.c b/drivers/pci/dwc/pcie-hisi.c
> index a201791..6800747 100644
> --- a/drivers/pci/dwc/pcie-hisi.c
> +++ b/drivers/pci/dwc/pcie-hisi.c
> @@ -270,6 +270,12 @@ static int hisi_pcie_probe(struct platform_device *pdev)
>       struct resource *reg;
>       int ret;
>  
> +     if ((IS_BUILTIN(CONFIG_ARM_SMMU_V3)) &&
> +                     of_property_read_bool(dev->of_node, "iommu-map")) {
> +             dev_warn(dev, "HiSilicon erratum 161010801: blacklisting PCIe 
> controllers behind SMMUv3\n");
> +             return -ENODEV;
> +     }
> +
>       hisi_pcie = devm_kzalloc(dev, sizeof(*hisi_pcie), GFP_KERNEL);
>       if (!hisi_pcie)
>               return -ENOMEM;
> @@ -340,6 +346,12 @@ static int hisi_pcie_almost_ecam_probe(struct 
> platform_device *pdev)
>       struct device *dev = &pdev->dev;
>       struct pci_ecam_ops *ops;
>  
> +     if ((IS_BUILTIN(CONFIG_ARM_SMMU_V3)) &&
> +                     of_property_read_bool(dev->of_node, "iommu-map")) {
> +             dev_warn(dev, "HiSilicon erratum 161010801: blacklisting PCIe 
> controllers behind SMMUv3\n");
> +             return -ENODEV;
> +     }
> +
>       ops = (struct pci_ecam_ops *)of_device_get_match_data(dev);
>       return pci_host_common_probe(pdev, ops);
>  }
>

Acked-by: Zhou Wang <wangzh...@hisilicon.com>

Thanks,
Zhou



_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to