public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: J. William Campbell <jwilliamcampbell@comcast.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC][Timer API] Revised Specification - Implementation details
Date: Fri, 27 May 2011 07:16:54 -0700	[thread overview]
Message-ID: <4DDFB256.2060306@comcast.net> (raw)
In-Reply-To: <20110527073324.C574014EF7F@gemini.denx.de>

On 5/27/2011 12:33 AM, Wolfgang Denk wrote:
> Dear "J. William Campbell",
>
> In message<4DDF2072.5090802@comcast.net>  you wrote:
> ...
>> The problem is that the way we previously detected wrapping does not
>> work if the interrupt rate is == to the counter wrap time, which it
>> essentially always is. If get_ticks is trying to update the wrap count
> You ignore the fact that this is only ever a problem when the rollover
> cannot signal through an interrupt or similar.  Also, some processors
> allow daisy-chaning of timers, etc.
>
> Again, I would really like to know about how many exotic systems we
> are talking that fulfil your worst-case expectations.  I bet the
> overwhelming majority behaves absolutely harmless.
Hi Wolfgang,
     I think that in fact the opposite is true. The problems occur if 
both the main program and the interrupt routine are trying to update the 
timer msb using the same code, as we were originally talking about. 
There is no problem if only the interrupt routine detects the rollover. 
That is the correct way to go if your interrupts work. There was nothing 
particularly exotic required. It was the "normal" case. Take a look at 
what would happen on the PPC is the main program was reading the 
decrementer, detecting wraps and increasing the timestamp while the 
interrupt routine was also incrementing the timestamp. Every so often 
you get a double increment. Why were we doing this? Because I was trying 
to re-use exactly the same code in the interrupt case and the 
non-interrupt case. Not a good idea, in fact a bad idea as it turns out.

Best Regards,
Bill Campbell
> Best regards,
>
> Wolfgang Denk
>

  reply	other threads:[~2011-05-27 14:16 UTC|newest]

Thread overview: 87+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-26 13:27 [U-Boot] [RFC][Timer API] Revised Specification - Implementation details Graeme Russ
2011-05-26 15:57 ` Simon Glass
2011-05-26 17:28   ` Wolfgang Denk
2011-05-26 22:44     ` Graeme Russ
2011-05-27  5:23       ` Simon Glass
2011-05-27  7:40         ` Wolfgang Denk
2011-05-27 14:46           ` Simon Glass
2011-05-26 16:56 ` J. William Campbell
2011-05-26 17:53   ` Wolfgang Denk
2011-05-26 18:52     ` J. William Campbell
2011-05-26 19:16       ` Wolfgang Denk
2011-05-26 19:54         ` J. William Campbell
2011-05-26 20:27           ` Wolfgang Denk
2011-05-26 20:39             ` J. William Campbell
2011-05-26 22:59       ` Graeme Russ
2011-05-26 23:28   ` Graeme Russ
2011-05-27  1:26     ` J. William Campbell
2011-05-27  1:51       ` Graeme Russ
2011-05-27  3:54         ` J. William Campbell
2011-05-27  4:33           ` Graeme Russ
2011-05-27  6:33             ` J. William Campbell
2011-05-27  6:54               ` Graeme Russ
2011-05-27 15:49                 ` J. William Campbell
2011-05-28  0:32                   ` Graeme Russ
2011-05-27  7:33           ` Wolfgang Denk
2011-05-27 14:16             ` J. William Campbell [this message]
2011-05-27  7:28       ` Wolfgang Denk
2011-05-27 14:04         ` J. William Campbell
2011-05-27  7:13     ` Wolfgang Denk
2011-05-27  7:35       ` Graeme Russ
2011-05-27  7:48         ` Wolfgang Denk
2011-05-27  7:57           ` Graeme Russ
2011-05-27  8:01             ` Graeme Russ
2011-05-27 11:27               ` Wolfgang Denk
2011-05-27 12:43                 ` Graeme Russ
2011-05-27 13:07                   ` Scott McNutt
2011-05-27 15:00                     ` J. William Campbell
2011-05-27 15:13                       ` Simon Glass
2011-05-27 17:11                         ` J. William Campbell
2011-05-27 15:44                       ` Scott McNutt
2011-05-27 15:59                         ` J. William Campbell
2011-05-29 15:55                     ` Wolfgang Denk
2011-05-29 19:12                       ` Simon Glass
2011-05-30 10:57                         ` Wolfgang Denk
2011-05-30 11:47                           ` Graeme Russ
2011-05-30 12:31                             ` Wolfgang Denk
2011-05-30 12:46                               ` Graeme Russ
2011-05-30 18:57                           ` Reinhard Meyer
2011-05-31  0:24                             ` Graeme Russ
2011-05-31  4:07                               ` Reinhard Meyer
2011-05-31  4:24                                 ` Graeme Russ
2011-05-31  4:36                                   ` Reinhard Meyer
2011-05-31  4:53                                     ` Graeme Russ
2011-05-31  5:56                                     ` Wolfgang Denk
2011-05-31  4:45                               ` Simon Glass
2011-05-31  4:53                                 ` Reinhard Meyer
2011-05-31  5:03                                   ` Graeme Russ
2011-05-31  5:16                                     ` Reinhard Meyer
2011-05-31  6:03                                     ` Wolfgang Denk
2011-05-31  6:23                                       ` Graeme Russ
2011-05-31  5:18                                   ` Wolfgang Denk
2011-05-31  5:37                                     ` Reinhard Meyer
2011-05-31  6:10                                       ` Wolfgang Denk
2011-05-31  4:56                           ` Simon Glass
2011-05-31  5:49                             ` Wolfgang Denk
2011-05-31  6:28                               ` Graeme Russ
2011-05-31  6:29                                 ` Graeme Russ
2011-06-15 13:17                           ` Graeme Russ
2011-06-15 16:03                             ` Simon Glass
2011-06-15 20:38                               ` Graeme Russ
2011-06-15 21:58                                 ` Simon Glass
2011-06-15 23:09                                   ` Graeme Russ
2011-06-16  5:53                                     ` Simon Glass
2011-06-16  6:27                                       ` Graeme Russ
2011-06-16 13:58                                         ` Simon Glass
2011-05-27 11:26             ` Wolfgang Denk
2011-05-27 14:23         ` J. William Campbell
2011-05-28  5:53           ` Graeme Russ
2011-05-28  6:18             ` Reinhard Meyer
2011-05-28  8:59               ` Graeme Russ
2011-05-29  1:41             ` J. William Campbell
2011-05-26 17:49 ` Wolfgang Denk
2011-05-26 22:51   ` Graeme Russ
2011-05-27  7:17     ` Wolfgang Denk
2011-05-27  7:33       ` Graeme Russ
2011-05-27  7:45         ` Wolfgang Denk
2011-05-27 14:58           ` Simon Glass

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=4DDFB256.2060306@comcast.net \
    --to=jwilliamcampbell@comcast.net \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox