cyclic: Fix rollover every 72 min on 32 bits platforms
authorPatrice Chotard <patrice.chotard@foss.st.com>
Tue, 14 Jan 2025 13:28:13 +0000 (14:28 +0100)
committerStefan Roese <sr@denx.de>
Wed, 22 Jan 2025 09:22:34 +0000 (10:22 +0100)
commit302f866e5c2671919f4900983a59476966c84156
tree27d09f556f398b1716a6d4df52da14b241bf2f27
parent82d262ae162d859d3b0bbcd40a9464e890b009da
cyclic: Fix rollover every 72 min on 32 bits platforms

On 32 bits platforms, timer_get_us() returns an unsigned long which
is a 32 bits. timer_get_us() wraps around every 72 minutes
(2 ^ 32 / 1000000 =~ 4295 sec =~ 72 min).

So the test "if time_after_eq64(now, cyclic->next_call)" is no more
true when cyclic->next_call becomes above 32 bits max value (4294967295).

At this point after 72 min, no more cyclic function are
executed included watchdog one.

Instead of using timer_get_us(), use get_timer_us() which returns a
uint64_t, this allows a rollover every 584942 years.

Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
Reviewed-by: Stefan Roese <sr@denx.de>
common/cyclic.c