> -----Original Message----- > From: Rob Herring [mailto:robh@xxxxxxxxxx] > Sent: Tuesday, September 19, 2017 3:53 PM > To: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@xxxxxxxxxx> > Cc: lorenzo.pieralisi@xxxxxxx; marc.zyngier@xxxxxxx; > sudeep.holla@xxxxxxx; will.deacon@xxxxxxx; robin.murphy@xxxxxxx; > joro@xxxxxxxxxx; mark.rutland@xxxxxxx; hanjun.guo@xxxxxxxxxx; Gabriele > Paoloni <gabriele.paoloni@xxxxxxxxxx>; John Garry > <john.garry@xxxxxxxxxx>; iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx; linux-acpi@xxxxxxxxxxxxxxx; > devicetree@xxxxxxxxxxxxxxx; devel@xxxxxxxxxx; Linuxarm > <linuxarm@xxxxxxxxxx>; Wangzhou (B) <wangzhou1@xxxxxxxxxxxxx>; > Guohanjun (Hanjun Guo) <guohanjun@xxxxxxxxxx> > Subject: Re: [PATCH v7 1/5] Doc: iommu/arm-smmu-v3: Add workaround for > HiSilicon erratum 161010801 > > On Thu, Sep 14, 2017 at 01:57:52PM +0100, Shameer Kolothum wrote: > > From: John Garry <john.garry@xxxxxxxxxx> > > > > The HiSilicon erratum 161010801 describes the limitation of HiSilicon > platforms > > hip06/hip07 to support the SMMU mappings for MSI transactions. > > > > On these platforms, GICv3 ITS translator is presented with the deviceID > > by extending the MSI payload data to 64 bits to include the deviceID. > > Hence, the PCIe controller on this 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. > > > > This patch adds a SMMUv3 binding to flag that the SMMU breaks msi > > translation at ITS. > > > > Also, the arm64 silicon errata is updated with this same erratum. > > > > Signed-off-by: John Garry <john.garry@xxxxxxxxxx> > > Signed-off-by: Shameer Kolothum > <shameerali.kolothum.thodi@xxxxxxxxxx> [...] > > --- a/Documentation/devicetree/bindings/iommu/arm,smmu-v3.txt > > +++ b/Documentation/devicetree/bindings/iommu/arm,smmu-v3.txt > > @@ -55,6 +55,9 @@ the PCIe specification. > > - hisilicon,broken-prefetch-cmd > > : Avoid sending CMD_PREFETCH_* commands to the SMMU. > > > > +- hisilicon,broken-untranslated-msi > > + : Reserve ITS HW region to avoid translating msi. > > + > > This should be determined from the compatible string. Continuing to add > properties for each errata doesn't scale. Ok. I think the suggestion here is to follow the arm-smmu.c (SMMUv1/v2) driver way of implementing the errata. As you might have noticed, the SMMUv3 driver dt errata framework depends on properties and this will change the way errata is implemented in the driver now. Hi Will/Robin, Could you please take a look and let us know your thoughts on changing the SMMUv3 dt errata implementation to version/model/compatible string framework for this quirk. Thanks, Shameer -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
