From ab4d69baf3ea1c2fd57c0d7f810f9482417d4d7c Mon Sep 17 00:00:00 2001 From: Chris Reed Date: Wed, 13 Oct 2021 15:02:42 -0500 Subject: [PATCH] kernel: work_q: use flags_get() in work_delayable_busy_get_locked(). The k_work::flags field is not an atomic_t and would cause -Wpointer-sign warning on some compilers. This function was the only one in work.c to use atomic_get() so there is no benefit to atomicity. Signed-off-by: Chris Reed --- kernel/work.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/work.c b/kernel/work.c index 7d32c73ee7c..ebd3b3f8b14 100644 --- a/kernel/work.c +++ b/kernel/work.c @@ -829,7 +829,7 @@ void k_work_init_delayable(struct k_work_delayable *dwork, static inline int work_delayable_busy_get_locked(const struct k_work_delayable *dwork) { - return atomic_get(&dwork->work.flags) & K_WORK_MASK; + return flags_get(&dwork->work.flags) & K_WORK_MASK; } int k_work_delayable_busy_get(const struct k_work_delayable *dwork)