diff options
author | Dawid Niedzwiecki <dn@semihalf.com> | 2020-10-28 10:02:32 +0100 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-11-02 10:17:27 +0000 |
commit | 71069893536962157b3b25330851a1c2c7cd6da5 (patch) | |
tree | 22c6f60ff900b68871f81e44cfe01b7616561bb6 /core/riscv-rv32i | |
parent | 0438542f2690e958d47dd28636797acbd29fcd9d (diff) | |
download | chrome-ec-71069893536962157b3b25330851a1c2c7cd6da5.tar.gz |
atomic: rename atomic_read_clear to atomic_clear
Rename atomic_read_clear to atomic_clear to be consistent with the rest
of the atomic functions, which return the previous value of the
variable.
BUG=b:169151160
BRANCH=none
TEST=buildall
Signed-off-by: Dawid Niedzwiecki <dn@semihalf.com>
Change-Id: I2588971bd7687879a28ec637cf5f6c3d27d393f4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2505143
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Diffstat (limited to 'core/riscv-rv32i')
-rw-r--r-- | core/riscv-rv32i/atomic.h | 2 | ||||
-rw-r--r-- | core/riscv-rv32i/task.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/core/riscv-rv32i/atomic.h b/core/riscv-rv32i/atomic.h index c8baf9d42b..a4e9db66ed 100644 --- a/core/riscv-rv32i/atomic.h +++ b/core/riscv-rv32i/atomic.h @@ -35,7 +35,7 @@ static inline atomic_val_t atomic_sub(atomic_t *addr, atomic_val_t value) return __atomic_fetch_sub(addr, value, __ATOMIC_SEQ_CST); } -static inline atomic_val_t atomic_read_clear(atomic_t *addr) +static inline atomic_val_t atomic_clear(atomic_t *addr) { return __atomic_exchange_n(addr, 0, __ATOMIC_SEQ_CST); } diff --git a/core/riscv-rv32i/task.c b/core/riscv-rv32i/task.c index b5f78b12d4..e39e854a3e 100644 --- a/core/riscv-rv32i/task.c +++ b/core/riscv-rv32i/task.c @@ -397,7 +397,7 @@ static uint32_t __ram_code __wait_evt(int timeout_us, task_id_t resched) ret = timer_arm(deadline, me); ASSERT(ret == EC_SUCCESS); } - while (!(evt = atomic_read_clear(&tsk->events))) { + while (!(evt = atomic_clear(&tsk->events))) { /* Remove ourself and get the next task in the scheduler */ __schedule(1, resched, 0); resched = TASK_ID_IDLE; |