iommu/arm-smmu: Allow implementation specific write_s2cr
authorBjorn Andersson <bjorn.andersson@linaro.org>
Mon, 19 Oct 2020 18:23:21 +0000 (11:23 -0700)
committerWill Deacon <will@kernel.org>
Thu, 29 Oct 2020 18:12:10 +0000 (18:12 +0000)
The firmware found in some Qualcomm platforms intercepts writes to the
S2CR register in order to replace the BYPASS type with FAULT. Further
more it treats faults at this level as catastrophic and restarts the
device.

Add support for providing implementation specific versions of the S2CR
write function, to allow the Qualcomm driver to work around this
behavior.

Cc: <stable@vger.kernel.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Tested-by: Steev Klimaszewski <steev@kali.org>
Reviewed-by: Robin Murphy <robin.murphy@arm.com>
Link: https://lore.kernel.org/r/20201019182323.3162386-2-bjorn.andersson@linaro.org
Signed-off-by: Will Deacon <will@kernel.org>
drivers/iommu/arm/arm-smmu/arm-smmu.c
drivers/iommu/arm/arm-smmu/arm-smmu.h

index dad7fa86fbd4cf591f2b949984791d2b7a4bd640..bcbacf22331d60c6e701f8ba404f87afd50ea128 100644 (file)
@@ -929,9 +929,16 @@ static void arm_smmu_write_smr(struct arm_smmu_device *smmu, int idx)
 static void arm_smmu_write_s2cr(struct arm_smmu_device *smmu, int idx)
 {
        struct arm_smmu_s2cr *s2cr = smmu->s2crs + idx;
-       u32 reg = FIELD_PREP(ARM_SMMU_S2CR_TYPE, s2cr->type) |
-                 FIELD_PREP(ARM_SMMU_S2CR_CBNDX, s2cr->cbndx) |
-                 FIELD_PREP(ARM_SMMU_S2CR_PRIVCFG, s2cr->privcfg);
+       u32 reg;
+
+       if (smmu->impl && smmu->impl->write_s2cr) {
+               smmu->impl->write_s2cr(smmu, idx);
+               return;
+       }
+
+       reg = FIELD_PREP(ARM_SMMU_S2CR_TYPE, s2cr->type) |
+             FIELD_PREP(ARM_SMMU_S2CR_CBNDX, s2cr->cbndx) |
+             FIELD_PREP(ARM_SMMU_S2CR_PRIVCFG, s2cr->privcfg);
 
        if (smmu->features & ARM_SMMU_FEAT_EXIDS && smmu->smrs &&
            smmu->smrs[idx].valid)
index 1a746476927c993ebbbd0cc2662109849cb73301..b71647eaa319b2598cac6cb214a88cd457742fa3 100644 (file)
@@ -436,6 +436,7 @@ struct arm_smmu_impl {
        int (*alloc_context_bank)(struct arm_smmu_domain *smmu_domain,
                                  struct arm_smmu_device *smmu,
                                  struct device *dev, int start);
+       void (*write_s2cr)(struct arm_smmu_device *smmu, int idx);
 };
 
 #define INVALID_SMENDX                 -1