Atomic: Mark the modified uint32_t volatile

The atomic_* functions are often used in contexts where the data they
will operate on are volatile (due to being shared between tasks or a
task and an interrupt handler).  Adding volatile here makes using the
atomic_* functions a little easier in those cases and removes a cast
from the call sites (which could be obscuring a bug, if for instance
the variable was modified to be a uint16_t).

Signed-off-by: Anton Staaf <robotboy@chromium.org>

BRANCH=None
BUG=None
TEST=make buildall -j

Change-Id: I71356eb3cf2c0506df38532eee767c7d78f9240e
Reviewed-on: https://chromium-review.googlesource.com/287516
Trybot-Ready: Anton Staaf <robotboy@chromium.org>
Tested-by: Anton Staaf <robotboy@chromium.org>
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Commit-Queue: Anton Staaf <robotboy@chromium.org>

4 files changed, 20 insertions, 20 deletions

diff --git a/core/cortex-m/atomic.h b/core/cortex-m/atomic.h
index 489db51357..a39f816008 100644
--- a/core/cortex-m/atomic.h
+++ b/core/cortex-m/atomic.h
@@ -30,27 +30,27 @@
 			     : "r" (a), "r" (v) : "cc");        \
 } while (0)
 
-static inline void atomic_clear(uint32_t *addr, uint32_t bits)
+static inline void atomic_clear(uint32_t volatile *addr, uint32_t bits)
 {
 	ATOMIC_OP(bic, addr, bits);
 }
 
-static inline void atomic_or(uint32_t *addr, uint32_t bits)
+static inline void atomic_or(uint32_t volatile *addr, uint32_t bits)
 {
 	ATOMIC_OP(orr, addr, bits);
 }
 
-static inline void atomic_add(uint32_t *addr, uint32_t value)
+static inline void atomic_add(uint32_t volatile *addr, uint32_t value)
 {
 	ATOMIC_OP(add, addr, value);
 }
 
-static inline void atomic_sub(uint32_t *addr, uint32_t value)
+static inline void atomic_sub(uint32_t volatile *addr, uint32_t value)
 {
 	ATOMIC_OP(sub, addr, value);
 }
 
-static inline uint32_t atomic_read_clear(uint32_t *addr)
+static inline uint32_t atomic_read_clear(uint32_t volatile *addr)
 {
 	uint32_t ret, tmp;
 
diff --git a/core/cortex-m0/atomic.h b/core/cortex-m0/atomic.h
index 62bc2ff370..32260ada07 100644
--- a/core/cortex-m0/atomic.h
+++ b/core/cortex-m0/atomic.h
@@ -27,27 +27,27 @@
 			     : "b" (a), "r" (v) : "cc");	\
 } while (0)
 
-static inline void atomic_clear(uint32_t *addr, uint32_t bits)
+static inline void atomic_clear(uint32_t volatile *addr, uint32_t bits)
 {
 	ATOMIC_OP(bic, addr, bits);
 }
 
-static inline void atomic_or(uint32_t *addr, uint32_t bits)
+static inline void atomic_or(uint32_t volatile *addr, uint32_t bits)
 {
 	ATOMIC_OP(orr, addr, bits);
 }
 
-static inline void atomic_add(uint32_t *addr, uint32_t value)
+static inline void atomic_add(uint32_t volatile *addr, uint32_t value)
 {
 	ATOMIC_OP(add, addr, value);
 }
 
-static inline void atomic_sub(uint32_t *addr, uint32_t value)
+static inline void atomic_sub(uint32_t volatile *addr, uint32_t value)
 {
 	ATOMIC_OP(sub, addr, value);
 }
 
-static inline uint32_t atomic_read_clear(uint32_t *addr)
+static inline uint32_t atomic_read_clear(uint32_t volatile *addr)
 {
 	uint32_t ret;
 
diff --git a/core/host/atomic.h b/core/host/atomic.h
index 77e9cc6ba8..17721390f8 100644
--- a/core/host/atomic.h
+++ b/core/host/atomic.h
@@ -10,27 +10,27 @@
 
 #include "common.h"
 
-static inline void atomic_clear(uint32_t *addr, uint32_t bits)
+static inline void atomic_clear(uint32_t volatile *addr, uint32_t bits)
 {
 	__sync_and_and_fetch(addr, ~bits);
 }
 
-static inline void atomic_or(uint32_t *addr, uint32_t bits)
+static inline void atomic_or(uint32_t volatile *addr, uint32_t bits)
 {
 	__sync_or_and_fetch(addr, bits);
 }
 
-static inline void atomic_add(uint32_t *addr, uint32_t value)
+static inline void atomic_add(uint32_t volatile *addr, uint32_t value)
 {
 	__sync_add_and_fetch(addr, value);
 }
 
-static inline void atomic_sub(uint32_t *addr, uint32_t value)
+static inline void atomic_sub(uint32_t volatile *addr, uint32_t value)
 {
 	__sync_sub_and_fetch(addr, value);
 }
 
-static inline uint32_t atomic_read_clear(uint32_t *addr)
+static inline uint32_t atomic_read_clear(uint32_t volatile *addr)
 {
 	return __sync_fetch_and_and(addr, 0);
 }
diff --git a/core/nds32/atomic.h b/core/nds32/atomic.h
index 3214067c43..2b49dfe5c1 100644
--- a/core/nds32/atomic.h
+++ b/core/nds32/atomic.h
@@ -11,7 +11,7 @@
 #include "common.h"
 #include "cpu.h"
 
-static inline void atomic_clear(uint32_t *addr, uint32_t bits)
+static inline void atomic_clear(uint32_t volatile *addr, uint32_t bits)
 {
 	uint32_t psw = get_psw();
 	asm volatile ("setgie.d");
@@ -19,7 +19,7 @@ static inline void atomic_clear(uint32_t *addr, uint32_t bits)
 	set_psw(psw);
 }
 
-static inline void atomic_or(uint32_t *addr, uint32_t bits)
+static inline void atomic_or(uint32_t volatile *addr, uint32_t bits)
 {
 	uint32_t psw = get_psw();
 	asm volatile ("setgie.d");
@@ -27,7 +27,7 @@ static inline void atomic_or(uint32_t *addr, uint32_t bits)
 	set_psw(psw);
 }
 
-static inline void atomic_add(uint32_t *addr, uint32_t value)
+static inline void atomic_add(uint32_t volatile *addr, uint32_t value)
 {
 	uint32_t psw = get_psw();
 	asm volatile ("setgie.d");
@@ -35,7 +35,7 @@ static inline void atomic_add(uint32_t *addr, uint32_t value)
 	set_psw(psw);
 }
 
-static inline void atomic_sub(uint32_t *addr, uint32_t value)
+static inline void atomic_sub(uint32_t volatile *addr, uint32_t value)
 {
 	uint32_t psw = get_psw();
 	asm volatile ("setgie.d");
@@ -43,7 +43,7 @@ static inline void atomic_sub(uint32_t *addr, uint32_t value)
 	set_psw(psw);
 }
 
-static inline uint32_t atomic_read_clear(uint32_t *addr)
+static inline uint32_t atomic_read_clear(uint32_t volatile *addr)
 {
 	uint32_t val;
 	uint32_t psw = get_psw();