commit 2c374e07cb5af04dc16ae8cb0fc0ad56eade4949
author Guillermo Maturana <maturana@google.com> Wed Mar 02 23:46:05 2022 -0800
committer Matute <maturana@google.com> Thu Mar 03 13:22:48 2022 -0800
tree 7798162290ae7b1265c94f5e5177f94a31a2e0bb
parent e2cbb314757be3faf902575b664360e7ad4c1a56

[sw/runtime] Fix ibex_timeout_check

The return value should match the function documentation.
Fix the IBEX_SPIN_FOR macro to match.

Signed-off-by: Guillermo Maturana <maturana@google.com>

sw/device/lib/runtime/ibex.h

diff --git a/sw/device/lib/runtime/ibex.h b/sw/device/lib/runtime/ibex.h
index 789dbda..4b12a49 100644
--- a/sw/device/lib/runtime/ibex.h
+++ b/sw/device/lib/runtime/ibex.h
@@ -149,7 +149,7 @@
  * @return Boolean indicating the timeout expired.
  */
 inline bool ibex_timeout_check(const ibex_timeout_t *timeout) {
-  return ibex_mcycle_read() - timeout->start < timeout->cycles;
+  return ibex_mcycle_read() - timeout->start > timeout->cycles;
 }
 
 /**
@@ -173,7 +173,7 @@
   do {                                                                    \
     const ibex_timeout_t timeout_ = ibex_timeout_init(timeout_usec);      \
     while (!(expr)) {                                                     \
-      CHECK(!ibex_timeout_check(&timeout_),                               \
+      CHECK(ibex_timeout_check(&timeout_),                                \
             "Timed out after %d usec (%d CPU cycles) waiting for " #expr, \
             timeout_usec, (uint32_t)timeout_.cycles);                     \
     }                                                                     \