Skip to content

Commit 74be161

Browse files
ABOSTMMaureenHelm
authored andcommitted
tests: kernel: timer: timer_api: reorder test_timer_remaining step
Move init_timer_data() out of k_usleep() tick alignment. Compute rem_ticks just after busy_wait_ms() to avoid slew due to 'now' and 'rem_ms' computations. With slow CPU 32MHz: -2 Ticks. Signed-off-by: Alexandre Bourdiol <[email protected]>
1 parent 4133cb5 commit 74be161

File tree

1 file changed

+2
-2
lines changed
  • tests/kernel/timer/timer_api/src

1 file changed

+2
-2
lines changed

tests/kernel/timer/timer_api/src/main.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -610,14 +610,14 @@ void test_timer_remaining(void)
610610
uint32_t slew_ticks;
611611
uint64_t now;
612612

613-
k_usleep(1); /* align to tick */
614613

615614
init_timer_data();
615+
k_usleep(1); /* align to tick */
616616
k_timer_start(&remain_timer, K_MSEC(DURATION), K_NO_WAIT);
617617
busy_wait_ms(DURATION / 2);
618+
rem_ticks = k_timer_remaining_ticks(&remain_timer);
618619
now = k_uptime_ticks();
619620
rem_ms = k_timer_remaining_get(&remain_timer);
620-
rem_ticks = k_timer_remaining_ticks(&remain_timer);
621621
exp_ticks = k_timer_expires_ticks(&remain_timer);
622622
k_timer_stop(&remain_timer);
623623
TIMER_ASSERT(tdata.expire_cnt == 0, &remain_timer);

0 commit comments

Comments
 (0)