Linu, On 11.04.17 20:12:43, linucherian@xxxxxxxxx wrote: > From: Linu Cherian <linu.cherian@xxxxxxxxxx> > > Enable SKIP_PREFETCH option for HiSilicon SMMUv3 model. > Enable PAGE0_REGS_ONLY and USE_SHARED_IRQS options for > Cavium 99xx SMMUv3 model. > > Signed-off-by: Linu Cherian <linu.cherian@xxxxxxxxxx> > --- > drivers/iommu/arm-smmu-v3.c | 27 +++++++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > > diff --git a/drivers/iommu/arm-smmu-v3.c b/drivers/iommu/arm-smmu-v3.c > index 1475ad8..00b48b4 100644 > --- a/drivers/iommu/arm-smmu-v3.c > +++ b/drivers/iommu/arm-smmu-v3.c > @@ -2634,18 +2634,45 @@ static int arm_smmu_device_hw_probe(struct arm_smmu_device *smmu) > } > > #ifdef CONFIG_ACPI > +static int acpi_smmu_get_options(u32 model, struct arm_smmu_device *smmu) > +{ > + int ret = 0; > + > + switch (model) { > + case ACPI_IORT_SMMU_V3: > + case ACPI_IORT_SMMU_CORELINK_MMU600: > + break; > + case ACPI_IORT_SMMU_V3_HISILICON: > + smmu->options |= ARM_SMMU_OPT_SKIP_PREFETCH; > + break; > + case ACPI_IORT_SMMU_V3_CAVIUM_CN99XX: > + smmu->options |= (ARM_SMMU_OPT_PAGE0_REGS_ONLY | > + ARM_SMMU_OPT_USE_SHARED_IRQS); > + break; > + default: > + ret = -ENODEV; > + } > + > + return ret; > +} > + > static int arm_smmu_device_acpi_probe(struct platform_device *pdev, > struct arm_smmu_device *smmu) > { > struct acpi_iort_smmu_v3 *iort_smmu; > struct device *dev = smmu->dev; > struct acpi_iort_node *node; > + int ret; > > node = *(struct acpi_iort_node **)dev_get_platdata(dev); > > /* Retrieve SMMUv3 specific data */ > iort_smmu = (struct acpi_iort_smmu_v3 *)node->node_data; > > + ret = acpi_smmu_get_options(iort_smmu->model, smmu); > + if (ret < 0) > + return ret; > + could we add at least: dev_notice(smmu->dev, "option mask 0x%x\n", smmu->options); We need a note to see which options have been enabled. It could be even more fancy and maybe unified with DT code to see the same messages for DT and ACPI. Thanks, -Robert > if (iort_smmu->flags & ACPI_IORT_SMMU_V3_COHACC_OVERRIDE) > smmu->features |= ARM_SMMU_FEAT_COHERENCY; > > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html