commit b3d92027049024897e98a80f089fb07883211e3e
author Flavio Ceolin <flavio.ceolin@intel.com> Mon Sep 17 15:56:06 2018 -0700
committer Anas Nashif <anas.nashif@intel.com> Tue Sep 18 13:57:15 2018 -0400
tree 62908c2afc7b202b74358467233ebc8f08c2caa1
parent 4218d5f8f04795c9508c36d51b0b1179ba57b7c4

kernel: Using boolean constants instead of 0 or 1

MISRA C requires that every controlling expression of and if or while
statement have a boolean type.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>

kernel/idle.c

diff --git a/kernel/idle.c b/kernel/idle.c
index 556ad7e..2afd7ac 100644
--- a/kernel/idle.c
+++ b/kernel/idle.c
@@ -11,6 +11,7 @@
 #include <drivers/system_timer.h>
 #include <wait_q.h>
 #include <power.h>
+#include <stdbool.h>
 
 #if defined(CONFIG_TICKLESS_IDLE)
 /*
@@ -20,7 +21,7 @@
 s32_t _sys_idle_threshold_ticks = CONFIG_TICKLESS_IDLE_THRESH;
 
 #if defined(CONFIG_TICKLESS_KERNEL)
-#define _must_enter_tickless_idle(ticks) (1)
+#define _must_enter_tickless_idle(ticks) (true)
 #else
 #define _must_enter_tickless_idle(ticks) \
 		((ticks == K_FOREVER) || (ticks >= _sys_idle_threshold_ticks))
@@ -47,7 +48,6 @@
 {
 }
 #endif
-
 /**
  *
  * @brief Indicate that kernel is idling in tickless mode
@@ -64,7 +64,7 @@
 	_kernel.idle = ticks;
 }
 #else
-#define set_kernel_idle_time_in_ticks(x) do { } while (0)
+#define set_kernel_idle_time_in_ticks(x) do { } while (false)
 #endif
 
 #ifndef CONFIG_SMP
@@ -175,7 +175,7 @@
 	 * busy waiting is needed to prevent lock contention.  Long
 	 * term we need to wake up idle CPUs with an IPI.
 	 */
-	while (1) {
+	while (true) {
 		k_busy_wait(100);
 		k_yield();
 	}