Skip to content

Commit 94f5646

Browse files
committed
Atomics: make barrier placement uniform
1 parent 3d4582b commit 94f5646

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

platform/mbed_critical.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,8 @@ void core_util_critical_section_exit(void)
109109

110110
bool core_util_atomic_flag_test_and_set(volatile core_util_atomic_flag *flagPtr)
111111
{
112-
uint8_t currentValue;
113112
MBED_BARRIER();
113+
uint8_t currentValue;
114114
do {
115115
currentValue = __LDREXB(&flagPtr->_flag);
116116
} while (__STREXB(true, &flagPtr->_flag));
@@ -188,8 +188,8 @@ uint16_t core_util_atomic_incr_u16(volatile uint16_t *valuePtr, uint16_t delta)
188188

189189
uint32_t core_util_atomic_incr_u32(volatile uint32_t *valuePtr, uint32_t delta)
190190
{
191-
uint32_t newValue;
192191
MBED_BARRIER();
192+
uint32_t newValue;
193193
do {
194194
newValue = __LDREXW(valuePtr) + delta;
195195
} while (__STREXW(newValue, valuePtr));
@@ -200,8 +200,8 @@ uint32_t core_util_atomic_incr_u32(volatile uint32_t *valuePtr, uint32_t delta)
200200

201201
uint8_t core_util_atomic_decr_u8(volatile uint8_t *valuePtr, uint8_t delta)
202202
{
203-
uint8_t newValue;
204203
MBED_BARRIER();
204+
uint8_t newValue;
205205
do {
206206
newValue = __LDREXB(valuePtr) - delta;
207207
} while (__STREXB(newValue, valuePtr));
@@ -211,8 +211,8 @@ uint8_t core_util_atomic_decr_u8(volatile uint8_t *valuePtr, uint8_t delta)
211211

212212
uint16_t core_util_atomic_decr_u16(volatile uint16_t *valuePtr, uint16_t delta)
213213
{
214-
uint16_t newValue;
215214
MBED_BARRIER();
215+
uint16_t newValue;
216216
do {
217217
newValue = __LDREXH(valuePtr) - delta;
218218
} while (__STREXH(newValue, valuePtr));
@@ -222,8 +222,8 @@ uint16_t core_util_atomic_decr_u16(volatile uint16_t *valuePtr, uint16_t delta)
222222

223223
uint32_t core_util_atomic_decr_u32(volatile uint32_t *valuePtr, uint32_t delta)
224224
{
225-
uint32_t newValue;
226225
MBED_BARRIER();
226+
uint32_t newValue;
227227
do {
228228
newValue = __LDREXW(valuePtr) - delta;
229229
} while (__STREXW(newValue, valuePtr));

0 commit comments

Comments
 (0)