All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jon Hunter <jon-hunter@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Fix bootdelay timeout calculation when SYS_HZ!=1000
Date: Thu, 21 Mar 2013 10:25:03 -0500	[thread overview]
Message-ID: <514B264F.2000306@ti.com> (raw)
In-Reply-To: <20130321144145.GG26945@bill-the-cat>


On 03/21/2013 09:41 AM, Tom Rini wrote:
> On Thu, Mar 21, 2013 at 03:35:40PM +0100, Wolfgang Denk wrote:
>> Dear Tom,
>>
>> In message <514B1036.3090400@ti.com> you wrote:
>>>
>>>> A system with CONFIG_SYS_HZ != 1000 is _broken_.
>>>
>>> So, RPi is going higher, and Jon hit this on I suspect omap2420h4
>>> which is also higher (after mathing that all out).  If we no longer
>>> support CONFIG_SYS_HZ != 1000 we need to make that clear (and explain
>>> why).
>>
>> It has never been supported, so this is not a case of "no longer".
>> There are several longish threads (about the timer code, especially
>> on ARM) in the archives.
> 
> OK, then we need to do something about these platforms today.  I'm
> guessing RPi can just be tuned down to 1000 but for omap2420h4 it's an
> interesting value and I don't know about the platform well enough to say
> what we'd need to do to adapt it.  Jon?

For OMAP2420, we have a choice of using either a 12MHz or 32kHz clock to
drive the timer. With the 32kHz clock we can get close to 1000Hz tick
but we cannot get it dead on. That's why OMAP has been using 128Hz tick
in the kernel as opposed to 100Hz tick in the kernel for years (although
that was changed recently).

Cheers
Jon

  reply	other threads:[~2013-03-21 15:25 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-21  5:14 [U-Boot] [PATCH] Fix bootdelay timeout calculation when SYS_HZ!=1000 Stephen Warren
2013-03-21  6:55 ` Wolfgang Denk
2013-03-21 13:50   ` Tom Rini
2013-03-21 14:35     ` Wolfgang Denk
2013-03-21 14:41       ` Tom Rini
2013-03-21 15:25         ` Jon Hunter [this message]
2013-03-21 15:28           ` Tom Rini
2013-03-21 18:07             ` Jon Hunter
2013-03-21 18:26               ` Tom Rini
2013-03-21 16:04   ` Stephen Warren
2013-03-21 19:17     ` 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=514B264F.2000306@ti.com \
    --to=jon-hunter@ti.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.