Skip to content

Commit

Permalink
iommu/arm-smmu: Delete an unnecessary check before free_io_pgtable_ops()
Browse files Browse the repository at this point in the history
The free_io_pgtable_ops() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <[email protected]>
Signed-off-by: Will Deacon <[email protected]>
  • Loading branch information
elfring authored and wildea01 committed Dec 17, 2015
1 parent 04fa26c commit 44830b0
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions drivers/iommu/arm-smmu.c
Original file line number Diff line number Diff line change
Expand Up @@ -945,9 +945,7 @@ static void arm_smmu_destroy_domain_context(struct iommu_domain *domain)
free_irq(irq, domain);
}

if (smmu_domain->pgtbl_ops)
free_io_pgtable_ops(smmu_domain->pgtbl_ops);

free_io_pgtable_ops(smmu_domain->pgtbl_ops);
__arm_smmu_free_bitmap(smmu->context_map, cfg->cbndx);
}

Expand Down

0 comments on commit 44830b0

Please sign in to comment.