@@ -232,20 +232,22 @@ static inline void smmu_flush(struct tegra_smmu *smmu)
static int tegra_smmu_alloc_asid(struct tegra_smmu *smmu, unsigned int *idp)
{
unsigned long id;
+ int ret = 0;
mutex_lock(&smmu->lock);
id = find_first_zero_bit(smmu->asids, smmu->soc->num_asids);
if (id >= smmu->soc->num_asids) {
- mutex_unlock(&smmu->lock);
- return -ENOSPC;
+ ret = -ENOSPC;
+ goto Exit;
}
set_bit(id, smmu->asids);
*idp = id;
+Exit:
mutex_unlock(&smmu->lock);
- return 0;
+ return ret;
}
static void tegra_smmu_free_asid(struct tegra_smmu *smmu, unsigned int id)
Code can be optimized more by making single exit point of function and calling mutex_unlock at the end Signed-off-by: Pushkar Jambhlekar <pushkar.iit@gmail.com> --- drivers/iommu/tegra-smmu.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)