From 92c2fbdce410e3c679be4b7edc3551b1e14217ce Mon Sep 17 00:00:00 2001
From: Kevin O'Connor <kevin@koconnor.net>
Date: Sat, 27 Jul 2019 22:05:50 -0400
Subject: [PATCH] armcm_timer: Improve comments on wrap_timer code

Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
---
 src/generic/armcm_timer.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/generic/armcm_timer.c b/src/generic/armcm_timer.c
index dad768e48..f4ce2b8bf 100644
--- a/src/generic/armcm_timer.c
+++ b/src/generic/armcm_timer.c
@@ -68,7 +68,7 @@ udelay(uint32_t usecs)
         ;
 }
 
-// On fast cpus, schedule a recurring timer so SysTick doesn't overflow
+// Dummy timer to avoid scheduling a SysTick irq greater than 0xffffff
 static uint_fast8_t
 timer_wrap_event(struct timer *t)
 {
@@ -82,8 +82,10 @@ static struct timer wrap_timer = {
 void
 timer_reset(void)
 {
-    if (CONFIG_CLOCK_FREQ > 0xffffff * 10)
-        sched_add_timer(&wrap_timer);
+    if (timer_from_us(100000) <= 0xffffff)
+        // Timer in sched.c already ensures SysTick wont overflow
+        return;
+    sched_add_timer(&wrap_timer);
 }
 DECL_SHUTDOWN(timer_reset);