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] [PATCH] OMAP3: Fix timer handling to 1ms and	CONFIG_SYS_HZ to 1000.
Date: Mon, 20 Apr 2009 19:27:29 +0200	[thread overview]
Message-ID: <49ECB081.9050709@googlemail.com> (raw)
In-Reply-To: <20090420190717.GA10270@localhost.localdomain>

Dear Ladis,

Ladislav Michl wrote:
> On Thu, Apr 02, 2009 at 12:43:53AM +0200, Wolfgang Denk wrote:
>> In message <20090331202939.GI24923@game.jcrosoft.org> you wrote:
>>> On 11:19 Tue 17 Mar     , Dirk Behme wrote:
>>>> Fix OMAP3 timer handling to 1ms tick and CONFIG_SYS_HZ to 1000.
>>>> Clean up macros and comments.
>>>>
>>>> Signed-off-by: Dirk Behme <dirk.behme@googlemail.com>
>>>> Signed-off-by: Manikandan Pillai <mani.pillai@ti.com>
>> What is the status of this patch?
>>
>> I see no reason to delay it.
> 
> After messing with OMAP1 timer code (which is copied again and again over
> tree) I found such a reason. It is called 'void udelay(unsigned long usec)'
> implementation and I hereby leaving its bugfixing as an excercise for a
> reader [1]. 

Would you mind giving some more details? This would avoid wasting time 
to re-think about it (and maybe doing it wrong again) and would let us 
better use the time for fixing the issues you see.

Many thanks

Dirk

> Patch 'arm925t: Fix CONFIG_SYS_HZ to 1000', which is on its way
> to the list, can be used as a reference. Oh well, since it seems unlikely
> proper timer API will step in before 2009.06 leaves the door, let's fix it
> one by one...
> 
> 	ladis
> 
> [1] Diligent reader is welcome to base his work on the updated patch
>     http://lists.denx.de/pipermail/u-boot/2009-April/050170.html
> 

  reply	other threads:[~2009-04-20 17:27 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-17 10:19 [U-Boot] [PATCH] OMAP3: Fix timer handling to 1ms and CONFIG_SYS_HZ to 1000 Dirk Behme
2009-03-31 20:29 ` Jean-Christophe PLAGNIOL-VILLARD
2009-04-01 18:03   ` Dirk Behme
2009-04-01 22:43   ` Wolfgang Denk
2009-04-02 10:42     ` Ladislav Michl
2009-04-02 10:48       ` [U-Boot] [PATCH v2] " Ladislav Michl
2009-04-20 19:07     ` [U-Boot] [PATCH] " Ladislav Michl
2009-04-20 17:27       ` Dirk Behme [this message]
2009-04-20 22:10         ` Ladislav Michl

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=49ECB081.9050709@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.