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 | |
| 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>
| -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 | /* |
