diff options
| author | Hiroshi DOYU <hdoyu@nvidia.com> | 2011-11-16 11:27:53 +0200 |
|---|---|---|
| committer | Varun Wadekar <vwadekar@nvidia.com> | 2011-12-22 11:12:43 +0530 |
| commit | 8af87c972280972b60285b0ed68e324dcddaa30a (patch) | |
| tree | 0128f027328ce790c0e56ee4f26c6bde06c201b2 /arch/arm/mach-tegra/iovmm-gart.c | |
| parent | f0fa117c3fef2df54c2151c0d2c6ec7278cd45ee (diff) | |
ARM: tegra: gart: Remove unnecessary wmb()
Removed wmb()s which are:
- Logically unnecessary after while loop.
- Right after spinlock which has the same effect as wmb().
Change-Id: I2c4978b12d6bd9d4046a900b5ce7d6f0bdbd70e0
Signed-off-by: Hiroshi DOYU <hdoyu@nvidia.com>
Reviewed-on: http://git-master/r/66357
Reviewed-by: Rohan Somvanshi <rsomvanshi@nvidia.com>
Tested-by: Rohan Somvanshi <rsomvanshi@nvidia.com>
Diffstat (limited to 'arch/arm/mach-tegra/iovmm-gart.c')
| -rw-r--r-- | arch/arm/mach-tegra/iovmm-gart.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/arch/arm/mach-tegra/iovmm-gart.c b/arch/arm/mach-tegra/iovmm-gart.c index 290fd484ee85..9616af6672a1 100644 --- a/arch/arm/mach-tegra/iovmm-gart.c +++ b/arch/arm/mach-tegra/iovmm-gart.c @@ -123,7 +123,6 @@ static void do_gart_setup(struct gart_device *gart, const u32 *data) wmb(); reg += 1 << GART_PAGE_SHIFT; } - wmb(); } static void gart_resume(struct tegra_iovmm_device *dev) @@ -280,7 +279,7 @@ static int gart_map(struct tegra_iovmm_domain *domain, spin_unlock(&gart->pte_lock); } - wmb(); + return 0; fail: @@ -293,7 +292,7 @@ fail: wmb(); } spin_unlock(&gart->pte_lock); - wmb(); + return -ENOMEM; } @@ -319,7 +318,6 @@ static void gart_unmap(struct tegra_iovmm_domain *domain, gart_page += 1 << GART_PAGE_SHIFT; } spin_unlock(&gart->pte_lock); - wmb(); } static void gart_map_pfn(struct tegra_iovmm_domain *domain, @@ -335,7 +333,6 @@ static void gart_map_pfn(struct tegra_iovmm_domain *domain, writel(GART_PTE(pfn), gart->regs + GART_ENTRY_DATA); wmb(); spin_unlock(&gart->pte_lock); - wmb(); } static struct tegra_iovmm_domain *gart_alloc_domain( |