[U-Boot] [PATCH] OMAP[34]: fix broken timer

Igor Grinberg grinberg at compulab.co.il
Tue Jun 28 07:43:24 CEST 2011


Sandeep, Albert,


Please, apply this patch. It fixes a real issue with timer on omap.


Thanks


On 12/28/10 02:33, John Rigby wrote:

> As implemented now the timer used to implement __udelay counts
> to 0xffffffff and then gets stuck there because the the programmed
> reload value is 0xffffffff.  This value is not only wrong but
> illegal according to the reference manual.
>
> One can reproduce the bug by leaving a board at the u-boot prompt
> for sometime then issuing a sleep command.  The sleep will hang
> forever.
>
> The timer is a count up timer that reloads as it rolls over
> from 0xffffffff so the correct load value is 0.
>
> Change TIMER_LOAD_VAL from 0xffffffff to 0 and introduce
> a new constant called TIMER_OVERFLOW_VAL set to 0xffffffff.
>
> Signed-off-by: John Rigby <john.rigby at linaro.org>

Tested-by: Igor Grinberg <grinberg at compulab.co.il>

On DM3730, AM3703, OMAP3530.

> ---
>  arch/arm/cpu/armv7/omap-common/timer.c |    7 ++++---
>  1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/omap-common/timer.c b/arch/arm/cpu/armv7/omap-common/timer.c
> index 9beebb1..59bbca8 100644
> --- a/arch/arm/cpu/armv7/omap-common/timer.c
> +++ b/arch/arm/cpu/armv7/omap-common/timer.c
> @@ -43,8 +43,9 @@ static struct gptimer *timer_base = (struct gptimer *)CONFIG_SYS_TIMERBASE;
>   * Nothing really to do with interrupts, just starts up a counter.
>   */
>  
> -#define TIMER_CLOCK	(V_SCLK / (2 << CONFIG_SYS_PTV))
> -#define TIMER_LOAD_VAL	0xffffffff
> +#define TIMER_CLOCK		(V_SCLK / (2 << CONFIG_SYS_PTV))
> +#define TIMER_OVERFLOW_VAL	0xffffffff
> +#define TIMER_LOAD_VAL		0
>  
>  int timer_init(void)
>  {
> @@ -86,7 +87,7 @@ void __udelay(unsigned long usec)
>  	while (tmo > 0) {
>  		now = readl(&timer_base->tcrr);
>  		if (last > now) /* count up timer overflow */
> -			tmo -= TIMER_LOAD_VAL - last + now;
> +			tmo -= TIMER_OVERFLOW_VAL - last + now + 1;
>  		else
>  			tmo -= now - last;
>  		last = now;

-- 
Regards,
Igor.



More information about the U-Boot mailing list