commit 025123992b7a04bb0749342dd2a1369f43650724
author Srikrishna Iyer <sriyer@google.com> Thu Oct 01 11:39:37 2020 -0700
committer sriyerg <46467186+sriyerg@users.noreply.github.com> Sat Oct 03 09:49:17 2020 -0700
tree 965bc4c796de038910e1169d059739787cda48ef
parent e81ab7578ade039360e358a1a11a2198ca694330

[chip dv] Fix for RV timer test

- Reduce timer deadline for DV to 100 us

Signed-off-by: Srikrishna Iyer <sriyer@google.com>

sw/device/tests/dif/dif_rv_timer_sanitytest.c

diff --git a/sw/device/tests/dif/dif_rv_timer_sanitytest.c b/sw/device/tests/dif/dif_rv_timer_sanitytest.c
index 1aa80ae..0e8d62d 100644
--- a/sw/device/tests/dif/dif_rv_timer_sanitytest.c
+++ b/sw/device/tests/dif/dif_rv_timer_sanitytest.c
@@ -28,7 +28,6 @@
 static const uint32_t kComparator = 0;
 
 static const uint64_t kTickFreqHz = 1000 * 1000;  // 1 MHz.
-static const uint64_t kDeadline = 30000;          // 30 ms.
 
 static void test_handler(void) {
   CHECK(!irq_fired, "Entered IRQ handler, but `irq_fired` was not false!");
@@ -75,6 +74,12 @@
                                 kDifRvTimerEnabled) == kDifRvTimerOk);
 
   uint64_t current_time;
+  // Logs over UART incur a large runtime overhead. To accommodate that, the
+  // timer deadline needs to be large as well. In DV simulations, logs are not
+  // sent over UART, so we can reduce the runtime / sim time with a much shorter
+  // deadline (30 ms vs 100 us).
+  uint64_t kDeadline =
+      (kDeviceType == kDeviceSimDV) ? 100 /* 100 us */ : 30000 /* 30 ms */;
   CHECK(dif_rv_timer_counter_read(&timer, kHart, &current_time) ==
         kDifRvTimerOk);
   LOG_INFO("Current time: %d; timer theshold: %d", (uint32_t)current_time,