From: Jun Sun <jsun@mvista.com>
To: "Maciej W. Rozycki" <macro@ds2.pg.gda.pl>
Cc: Ralf Baechle <ralf@linux-mips.org>,
linux-mips@linux-mips.org, jsun@mvista.com
Subject: Re: [patch] Generic time trailing clean-ups
Date: Thu, 14 Aug 2003 09:52:46 -0700 [thread overview]
Message-ID: <20030814095246.C1203@mvista.com> (raw)
In-Reply-To: <Pine.GSO.3.96.1030814182619.17768B-100000@delta.ds2.pg.gda.pl>; from macro@ds2.pg.gda.pl on Thu, Aug 14, 2003 at 06:35:28PM +0200
On Thu, Aug 14, 2003 at 06:35:28PM +0200, Maciej W. Rozycki wrote:
> On Thu, 14 Aug 2003, Jun Sun wrote:
>
> > I am completely lost in your arguments. Let us keep it to the basic.
> >
> > Tell me what is wrong with the following, and why your proposal
> > is better than this:
> >
> > 1) get rid of calibrate_*() function
> > 2) introduce a generic counter frequence calibration routine, which
> > is only invoked when mips_counter_frequency is 0.
> > 3) If any board is not happy with this calibration, it is free to
> > do its calibration in board_timer_init(), which would set
> > mips_counter_frequency to be non-zero.
>
> So I am lost, too. What I proposed with the patch is exactly what you
> describe above. So what's wrong with it?
>
Oh, really? :)
1) I don't see you " get rid of calibrate_*() function"
2) oh, why? because your "generic counter frequence" is not generic -
it requires board-specific routines. I was referring to using
jiffies to calibrate frequency.
3) I also don't see picky boards "do its calibration in board_timer_init()".
Your proposal differs in every count. :)
Jun
next prev parent reply other threads:[~2003-08-14 16:52 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-08-11 13:02 [patch] Generic time trailing clean-ups Maciej W. Rozycki
2003-08-11 18:34 ` Jun Sun
2003-08-12 13:34 ` Maciej W. Rozycki
2003-08-12 17:05 ` Jun Sun
2003-08-13 12:25 ` Maciej W. Rozycki
2003-08-13 16:45 ` Jun Sun
2003-08-14 11:31 ` Maciej W. Rozycki
2003-08-14 16:13 ` Jun Sun
2003-08-14 16:35 ` Maciej W. Rozycki
2003-08-14 16:52 ` Jun Sun [this message]
2003-08-16 12:32 ` Maciej W. Rozycki
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=20030814095246.C1203@mvista.com \
--to=jsun@mvista.com \
--cc=linux-mips@linux-mips.org \
--cc=macro@ds2.pg.gda.pl \
--cc=ralf@linux-mips.org \
/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.