iommu/arm-smmu: Constify iommu_gather_ops structures

goyalbhumika · wildea01 · commit ca297aad17e1 · 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-v3.c b/drivers/iommu/arm-smmu-v3.c
@@ -1358,7 +1358,7 @@ static void arm_smmu_tlb_inv_range_nosync(unsigned long iova, size_t size,
 	} while (size -= granule);
 }
 
-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
`@@ -1358,7 +1358,7 @@ static void arm_smmu_tlb_inv_range_nosync(unsigned long iova, size_t size,`
`1358`	`1358`	`} while (size -= granule);`
`1359`	`1359`	`}`
`1360`	`1360`
`1361`		`-static struct iommu_gather_ops arm_smmu_gather_ops = {`
	`1361`	`+static const struct iommu_gather_ops arm_smmu_gather_ops = {`
`1362`	`1362`	`.tlb_flush_all = arm_smmu_tlb_inv_context,`
`1363`	`1363`	`.tlb_add_flush = arm_smmu_tlb_inv_range_nosync,`
`1364`	`1364`	`.tlb_sync = arm_smmu_tlb_sync,`