iommu/arm-smmu: Constify iommu_gather_ops structures

goyalbhumika · wildea01 · commit 5896f3a3a1a8 · 2016-11-29T15:57:40.000Z
Check for iommu_gather_ops structures that are only stored in the tlb
field of an io_pgtable_cfg structure. The tlb field is of type
const struct iommu_gather_ops *, so iommu_gather_ops structures
having this property can be declared as const.

Acked-by: Julia Lawall &lt;julia.lawall@lip6.fr&gt;
Signed-off-by: Bhumika Goyal &lt;bhumirks@gmail.com&gt;
Signed-off-by: Will Deacon &lt;will.deacon@arm.com&gt;
diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c
@@ -642,7 +642,7 @@ static void arm_smmu_tlb_inv_range_nosync(unsigned long iova, size_t size,
 	}
 }
 
-static struct iommu_gather_ops arm_smmu_gather_ops = {
+static const struct iommu_gather_ops arm_smmu_gather_ops = {
 	.tlb_flush_all	= arm_smmu_tlb_inv_context,
 	.tlb_add_flush	= arm_smmu_tlb_inv_range_nosync,
 	.tlb_sync	= arm_smmu_tlb_sync,

Original file line number	Diff line number	Diff line change
`@@ -642,7 +642,7 @@ static void arm_smmu_tlb_inv_range_nosync(unsigned long iova, size_t size,`
`642`	`642`	`}`
`643`	`643`	`}`
`644`	`644`
`645`		`-static struct iommu_gather_ops arm_smmu_gather_ops = {`
	`645`	`+static const struct iommu_gather_ops arm_smmu_gather_ops = {`
`646`	`646`	`.tlb_flush_all = arm_smmu_tlb_inv_context,`
`647`	`647`	`.tlb_add_flush = arm_smmu_tlb_inv_range_nosync,`
`648`	`648`	`.tlb_sync = arm_smmu_tlb_sync,`