diff options
author | Will Deacon <will.deacon@arm.com> | 2013-12-10 14:12:27 -0500 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-12-29 07:32:44 -0500 |
commit | 03aa6580e336c19e923372a3b6b72a300f24880c (patch) | |
tree | 175470d87b17a4e3028efe9e968f65d8eacbaa90 /arch/arm/common | |
parent | 145bc292dce9dbdface2acf1e7e1f175729fb5fb (diff) |
ARM: 7921/1: mcpm: remove redundant dsb instructions prior to sev
sync_cache_w already includes a dsb, so we can just use sev() directly
then following a cache-sync.
Acked-by: Dave Martin <Dave.Martin@arm.com>
Acked-by: Nicolas Pitre <nico@linaro.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/common')
-rw-r--r-- | arch/arm/common/mcpm_entry.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/arm/common/mcpm_entry.c b/arch/arm/common/mcpm_entry.c index 87c5f1b595bb..1e361abc29eb 100644 --- a/arch/arm/common/mcpm_entry.c +++ b/arch/arm/common/mcpm_entry.c | |||
@@ -166,7 +166,7 @@ void __mcpm_cpu_down(unsigned int cpu, unsigned int cluster) | |||
166 | dmb(); | 166 | dmb(); |
167 | mcpm_sync.clusters[cluster].cpus[cpu].cpu = CPU_DOWN; | 167 | mcpm_sync.clusters[cluster].cpus[cpu].cpu = CPU_DOWN; |
168 | sync_cache_w(&mcpm_sync.clusters[cluster].cpus[cpu].cpu); | 168 | sync_cache_w(&mcpm_sync.clusters[cluster].cpus[cpu].cpu); |
169 | dsb_sev(); | 169 | sev(); |
170 | } | 170 | } |
171 | 171 | ||
172 | /* | 172 | /* |
@@ -182,7 +182,7 @@ void __mcpm_outbound_leave_critical(unsigned int cluster, int state) | |||
182 | dmb(); | 182 | dmb(); |
183 | mcpm_sync.clusters[cluster].cluster = state; | 183 | mcpm_sync.clusters[cluster].cluster = state; |
184 | sync_cache_w(&mcpm_sync.clusters[cluster].cluster); | 184 | sync_cache_w(&mcpm_sync.clusters[cluster].cluster); |
185 | dsb_sev(); | 185 | sev(); |
186 | } | 186 | } |
187 | 187 | ||
188 | /* | 188 | /* |