[PATCH 19/29] iommu: exynos: remove useless spinlock

From: Marek Szyprowski
Date: Tue Aug 05 2014 - 06:49:18 EST


This patch removes useless spinlocks and other unused members from
struct exynos_iommu_owner. There is no point is protecting this
structure by spinlock because content of this structure doesn't change
and other structures have their own spinlocks.

Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
---
drivers/iommu/exynos-iommu.c | 11 -----------
1 file changed, 11 deletions(-)

diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
index ec3c882..ed8c518 100644
--- a/drivers/iommu/exynos-iommu.c
+++ b/drivers/iommu/exynos-iommu.c
@@ -190,9 +190,6 @@ struct exynos_iommu_owner {
struct list_head client; /* entry of exynos_iommu_domain.clients */
struct device *dev;
struct device *sysmmu;
- struct iommu_domain *domain;
- void *vmm_data; /* IO virtual memory manager's data */
- spinlock_t lock; /* Lock to preserve consistency of System MMU */
};

struct exynos_iommu_domain {
@@ -478,16 +475,12 @@ static int __exynos_sysmmu_enable(struct device *dev, phys_addr_t pgtable,

BUG_ON(!has_sysmmu(dev));

- spin_lock_irqsave(&owner->lock, flags);
-
data = dev_get_drvdata(owner->sysmmu);

ret = __sysmmu_enable(data, pgtable, domain);
if (ret >= 0)
data->master = dev;

- spin_unlock_irqrestore(&owner->lock, flags);
-
return ret;
}

@@ -500,16 +493,12 @@ static bool exynos_sysmmu_disable(struct device *dev)

BUG_ON(!has_sysmmu(dev));

- spin_lock_irqsave(&owner->lock, flags);
-
data = dev_get_drvdata(owner->sysmmu);

disabled = __sysmmu_disable(data);
if (disabled)
data->master = NULL;

- spin_unlock_irqrestore(&owner->lock, flags);
-
return disabled;
}

--
1.9.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/