From: Graf Yang <graf.y...@analog.com> The timer_init() function was not using the right csync instruction, nor was it doing it right after disabling the core timer.
The timer_reset() function would reset the timestamp, but not the actual timer, so there was a common edge case where get_timer() return a jump of one timestamp (couple milliseconds) right after resetting. This caused many functions to improperly timeout right away. Signed-off-by: Graf Yang <graf.y...@analog.com> Signed-off-by: Mike Frysinger <vap...@gentoo.org> --- cpu/blackfin/interrupts.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/cpu/blackfin/interrupts.c b/cpu/blackfin/interrupts.c index d4dd636..bf6fb4b 100644 --- a/cpu/blackfin/interrupts.c +++ b/cpu/blackfin/interrupts.c @@ -95,11 +95,12 @@ void udelay(unsigned long usec) int timer_init(void) { *pTCNTL = 0x1; + CSYNC(); *pTSCALE = 0x0; *pTCOUNT = MAX_TIM_LOAD; *pTPERIOD = MAX_TIM_LOAD; *pTCNTL = 0x7; - asm("CSYNC;"); + CSYNC(); timestamp = 0; last_time = 0; @@ -151,5 +152,5 @@ ulong get_timer(ulong base) void reset_timer(void) { - timestamp = 0; + timer_init(); } -- 1.6.3.1 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot