From: David Gibson <dwg@au1.ibm.com>
To: Daniel Walker <dwalker@mvista.com>
Cc: linuxppc-dev@ozlabs.org, Thomas Gleixner <tglx@linutronix.de>,
Paul Mackerras <paulus@samba.org>,
Realtime Kernel <linux-rt-users@vger.kernel.org>,
johnstul@us.ibm.com
Subject: Re: [PATCH 2/5] Implement generic time of day clocksource for powerpc machines.
Date: Thu, 20 Sep 2007 13:29:35 +1000 [thread overview]
Message-ID: <20070920032935.GC18991@localhost.localdomain> (raw)
In-Reply-To: <1190253281.11899.74.camel@imap.mvista.com>
On Wed, Sep 19, 2007 at 06:54:41PM -0700, Daniel Walker wrote:
> On Thu, 2007-09-20 at 11:46 +1000, Paul Mackerras wrote:
> > Daniel Walker writes:
> >
> > > On Thu, 2007-09-20 at 10:52 +1000, Paul Mackerras wrote:
> > > > Daniel Walker writes:
> > > >
> > > > > If you switch to the rtc do the shift and mult need to change?
> > > >
> > > > You can't switch; any given CPU chip will have either the RTC or the
> > > > timebase but not both.
> > >
> > > The code is switching between to clock read functions .. If they are
> >
> > What part of "you can't switch" wasn't clear? __USE_RTC() is a
> > constant on any given machine.
>
> Maybe say "It a constant" up front .. The value that is output from the
> clocksource read is converted to nanoseconds .. This clocksource is only
> designed to work with the timebase, so it doesn't make sense to output
> an rtc value.
I believe "rtc" in a 601 context means something different to "rtc" in
a general context...
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
WARNING: multiple messages have this Message-ID (diff)
From: David Gibson <dwg@au1.ibm.com>
To: Daniel Walker <dwalker@mvista.com>
Cc: Paul Mackerras <paulus@samba.org>,
linuxppc-dev@ozlabs.org, Thomas Gleixner <tglx@linutronix.de>,
johnstul@us.ibm.com,
Realtime Kernel <linux-rt-users@vger.kernel.org>
Subject: Re: [PATCH 2/5] Implement generic time of day clocksource for powerpc machines.
Date: Thu, 20 Sep 2007 13:29:35 +1000 [thread overview]
Message-ID: <20070920032935.GC18991@localhost.localdomain> (raw)
In-Reply-To: <1190253281.11899.74.camel@imap.mvista.com>
On Wed, Sep 19, 2007 at 06:54:41PM -0700, Daniel Walker wrote:
> On Thu, 2007-09-20 at 11:46 +1000, Paul Mackerras wrote:
> > Daniel Walker writes:
> >
> > > On Thu, 2007-09-20 at 10:52 +1000, Paul Mackerras wrote:
> > > > Daniel Walker writes:
> > > >
> > > > > If you switch to the rtc do the shift and mult need to change?
> > > >
> > > > You can't switch; any given CPU chip will have either the RTC or the
> > > > timebase but not both.
> > >
> > > The code is switching between to clock read functions .. If they are
> >
> > What part of "you can't switch" wasn't clear? __USE_RTC() is a
> > constant on any given machine.
>
> Maybe say "It a constant" up front .. The value that is output from the
> clocksource read is converted to nanoseconds .. This clocksource is only
> designed to work with the timebase, so it doesn't make sense to output
> an rtc value.
I believe "rtc" in a 601 context means something different to "rtc" in
a general context...
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
next prev parent reply other threads:[~2007-09-20 3:29 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-19 6:49 [PATCH 1/5] Implement {read,update}_persistent_clock Tony Breeds
2007-09-19 6:49 ` [PATCH 2/5] Implement generic time of day clocksource for powerpc machines Tony Breeds
2007-09-19 15:43 ` Daniel Walker
2007-09-19 15:43 ` Daniel Walker
2007-09-20 0:52 ` Paul Mackerras
2007-09-20 0:52 ` Paul Mackerras
2007-09-20 1:35 ` Daniel Walker
2007-09-20 1:35 ` Daniel Walker
2007-09-20 1:46 ` Paul Mackerras
2007-09-20 1:46 ` Paul Mackerras
2007-09-20 1:54 ` Daniel Walker
2007-09-20 1:54 ` Daniel Walker
2007-09-20 3:29 ` David Gibson [this message]
2007-09-20 3:29 ` David Gibson
2007-09-20 3:38 ` Daniel Walker
2007-09-20 3:38 ` Daniel Walker
2007-09-20 16:35 ` john stultz
2007-09-20 16:35 ` john stultz
2007-09-20 23:46 ` Tony Breeds
2007-09-20 23:46 ` Tony Breeds
2007-09-19 6:49 ` [PATCH 3/5] Implement clockevents driver for powerpc Tony Breeds
2007-09-19 6:49 ` [PATCH 4/5] Include hrtimer.h in tick.h Tony Breeds
2007-09-19 6:49 ` [PATCH 5/5] Enable tickless idle and high res timers for powerpc Tony Breeds
2007-09-19 6:53 ` [PATCH 1/5] Implement {read,update}_persistent_clock Tony Breeds
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=20070920032935.GC18991@localhost.localdomain \
--to=dwg@au1.ibm.com \
--cc=dwalker@mvista.com \
--cc=johnstul@us.ibm.com \
--cc=linux-rt-users@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=paulus@samba.org \
--cc=tglx@linutronix.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.