All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dirk Behme <dirk.behme@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] DaVinci DM644x timer problem (divided by 16)
Date: Fri, 21 Mar 2008 09:54:25 +0100	[thread overview]
Message-ID: <47E377C1.4000200@googlemail.com> (raw)
In-Reply-To: <4CD35CD1F8085945B597F80EEC8942138A2496@exc01.bk.prodrive.nl>

Pieter Voorthuijsen wrote:
> Hi Dirk,
> 
> If I understand correctly: When the timer wraps around, flash writes or
> other long lasting actions get timed-out. 

Yes. I never observed it myself. The inital report is

http://article.gmane.org/gmane.comp.boot-loaders.u-boot/32563/match=patch+fix+timer+overflow+davinci

but the patch was rejected by Wolfgang.

I then tried to fix it with a better patch, but seems that I failed ;) 
Sorry for that.

> Currently the timer is in 32 bit unchained mode, why not place it 64 bit
> mode?

Good idea! I'm not so familiar with DaVinci timer, but sounds like an 
option. This will need some changes to davinci/timer.c to handle 64bit 
timer, but this shouldn't be to hard, right?

Do you like to send a patch?

Many thanks

Dirk

> -----Original Message-----
> From: Dirk Behme [mailto:dirk.behme at googlemail.com] 
> Sent: donderdag 20 maart 2008 15:25
> To: Pieter Voorthuijsen
> Cc: u-boot-users at lists.sourceforge.net
> Subject: Re: [U-Boot-Users] DaVinci DM644x timer problem (divided by 16)
> 
> Pieter Voorthuijsen wrote:
> 
>>Hello,
>>
>>Some months ago, a patch was submitted ([ARM] fix timer overflow in 
>>DaVinci). The result was that in /cpu/arm926ejs/davinci/timer.c 
>>read_timer was divided by 16.
>>
>>http://www.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=commit;h=ea686f52
>>e4
>>5b3df2938866d3f5a98bb2556dfe2b
>>
>>Right now, when I have my bootdelay variable (on a DVEVM) set to 2, 
>>and countdown takes 32 seconds... Also iprobe got very very  very
> 
> slow.
> 
>>Maybe more items are affected by this timer division.
>>
>>Is anybody else seeing this? Clearly when the timer is used by several
> 
> 
>>blocks, it cannot be easily divided by 16. I think we need something 
>>else to work with the wrap around of the timer...
> 
> 
> Thanks for the report!
> 
> Any idea what might be better?
> 
> Thanks
> 
> Dirk
> 

  reply	other threads:[~2008-03-21  8:54 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-20 12:18 [U-Boot-Users] DaVinci DM644x timer problem (divided by 16) Pieter Voorthuijsen
2008-03-20 14:24 ` Dirk Behme
2008-03-21  8:19   ` Pieter Voorthuijsen
2008-03-21  8:54     ` Dirk Behme [this message]
     [not found]   ` <47E39D32.6090601@googlemail.com>
2008-03-21 15:33     ` Pieter Voorthuijsen
2008-03-21 23:20       ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47E377C1.4000200@googlemail.com \
    --to=dirk.behme@googlemail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.