test: dm: regmap: Fix the long test delay
authorSimon Glass <sjg@chromium.org>
Mon, 10 Dec 2018 00:11:10 +0000 (17:11 -0700)
committerSimon Glass <sjg@chromium.org>
Tue, 15 Jan 2019 00:47:13 +0000 (17:47 -0700)
At present one of the regmap tests takes 5 seconds to run since it waits
for a timeout. This should be handled using sandbox_timer_add_offset()
which advances time for test purposes.

This requires a little change to make the regmap_read_poll_timeout()
testable.

Update the macro and the test.

Fixes: ebe3497c9c ("test: regmap: add regmap_read_poll_timeout test")

Signed-off-by: Simon Glass <sjg@chromium.org>
include/regmap.h
test/dm/regmap.c

index a3afb72df51b2250cd1eaaf2a38b56da5dee16aa..8359c511d25f77e6cacd99b2da066f9d485b40d7 100644 (file)
@@ -248,6 +248,8 @@ int regmap_raw_read_range(struct regmap *map, uint range_num, uint offset,
  * @cond:      Break condition (usually involving @val)
  * @sleep_us:  Maximum time to sleep between reads in us (0 tight-loops).
  * @timeout_ms:        Timeout in ms, 0 means never timeout
+ * @test_add_time: Used for sandbox testing - amount of time to add after
+ *             starting the loop (0 if not testing)
  *
  * Returns 0 on success and -ETIMEDOUT upon a timeout or the regmap_read
  * error return value in case of a error read. In the two former cases,
@@ -256,8 +258,12 @@ int regmap_raw_read_range(struct regmap *map, uint range_num, uint offset,
  *
  * This is modelled after the regmap_read_poll_timeout macros in linux but
  * with millisecond timeout.
+ *
+ * The _test version is for sandbox testing only. Do not use this in normal
+ * code as it advances the timer.
  */
-#define regmap_read_poll_timeout(map, addr, val, cond, sleep_us, timeout_ms) \
+#define regmap_read_poll_timeout_test(map, addr, val, cond, sleep_us, \
+                                     timeout_ms, test_add_time) \
 ({ \
        unsigned long __start = get_timer(0); \
        int __ret; \
@@ -267,6 +273,8 @@ int regmap_raw_read_range(struct regmap *map, uint range_num, uint offset,
                        break; \
                if (cond) \
                        break; \
+               if (IS_ENABLED(CONFIG_SANDBOX) && test_add_time) \
+                       sandbox_timer_add_offset(test_add_time); \
                if ((timeout_ms) && get_timer(__start) > (timeout_ms)) { \
                        __ret = regmap_read((map), (addr), &(val)); \
                        break; \
@@ -277,6 +285,10 @@ int regmap_raw_read_range(struct regmap *map, uint range_num, uint offset,
        __ret ?: ((cond) ? 0 : -ETIMEDOUT); \
 })
 
+#define regmap_read_poll_timeout(map, addr, val, cond, sleep_us, timeout_ms) \
+       regmap_read_poll_timeout_test(map, addr, val, cond, sleep_us, \
+                                     timeout_ms, 0) \
+
 /**
  * regmap_update_bits() - Perform a read/modify/write using a mask
  *
index 9a70c159ddb9e9b4eec49f2c40088002c9cca667..82de295cb8f651940876d58867f1423b4fd436fc 100644 (file)
@@ -160,9 +160,10 @@ static int dm_test_regmap_poll(struct unit_test_state *uts)
        start = get_timer(0);
 
        ut_asserteq(-ETIMEDOUT,
-                   regmap_read_poll_timeout(map, 0, reg,
-                                            (reg == 0xcacafafa),
-                                            1, 5 * CONFIG_SYS_HZ));
+                   regmap_read_poll_timeout_test(map, 0, reg,
+                                                 (reg == 0xcacafafa),
+                                                 1, 5 * CONFIG_SYS_HZ,
+                                                 5 * CONFIG_SYS_HZ));
 
        ut_assert(get_timer(start) > (5 * CONFIG_SYS_HZ));