linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/13] 64-bit sched_clock
Date: Sun, 19 Dec 2010 23:39:10 +0000	[thread overview]
Message-ID: <20101219233910.GA816@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <AANLkTik0zkpUUvrv8YmxGd_=vMaKR42KbJnd4BqHqias@mail.gmail.com>

On Sun, Dec 19, 2010 at 11:23:02PM +0100, Linus Walleij wrote:
> 2010/12/16 Russell King - ARM Linux <linux@arm.linux.org.uk>:
> 
> > I can provide an init_early() method in the machine description which
> > setup_arch() can call as the last thing it does - this will happen
> > after the ->reserve and ->map_io callbacks, but before interrupts and
> > timers have been setup. ?This is the ideal time to setup the
> > sched_clock(). ?However, that means digging through lots of arch code
> > to sort out what can be moved where, and that's going to be inherently
> > unreliable.
> >
> > I think init_early() has merit in other ways - it'll allow us to remove
> > crap from the ->map_io callbacks which is doing stuff like registering
> > clk structures and so forth.
> 
> I agree. I was contemplating a clock_init() or similar a way back
> for setting up clock trees early. We would have to move clock tree
> registration in front of the init_sched_clock() call in the init_early()
> handler in many cases since the rate often comes from the clock
> framework, but this is indeed where it belongs.

Yes, that was indeed one of the subsystems that I thought platforms
should initialize there - one of my motivations is in connection with
6433/1 and 6434/1.

If we have everyone setting up their clock subsystems in init_early,
then we can have both the TWD and SP804 always getting their clock
rates from the clk API, rather than having the conditional approach
in those two patches.

      reply	other threads:[~2010-12-19 23:39 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-16  9:25 [PATCH 0/13] 64-bit sched_clock Russell King - ARM Linux
2010-12-16  9:27 ` [PATCH 01/13] ARM: sched_clock: provide common infrastructure for sched_clock() Russell King - ARM Linux
2010-12-16 12:52   ` Russell King - ARM Linux
2010-12-16  9:28 ` [PATCH 02/13] ARM: ixp4xx: convert sched_clock() to use new infrastructure Russell King - ARM Linux
2010-12-16  9:28 ` [PATCH 03/13] ARM: mmp: " Russell King - ARM Linux
2010-12-16  9:28 ` [PATCH 04/13] ARM: pxa: " Russell King - ARM Linux
2010-12-16 10:05   ` Uwe Kleine-König
2010-12-16 10:14     ` Russell King - ARM Linux
2010-12-16  9:29 ` [PATCH 05/13] ARM: sa1100: " Russell King - ARM Linux
2010-12-16  9:29 ` [PATCH 06/13] ARM: tegra: " Russell King - ARM Linux
2010-12-16  9:29 ` [PATCH 07/13] ARM: u300: " Russell King - ARM Linux
2010-12-16  9:30 ` [PATCH 08/13] ARM: iop: " Russell King - ARM Linux
2010-12-16  9:30 ` [PATCH 09/13] ARM: nomadik: " Russell King - ARM Linux
2010-12-16  9:31 ` [PATCH 10/13] ARM: omap: " Russell King - ARM Linux
2010-12-16  9:31 ` [PATCH 11/13] ARM: orion: " Russell King - ARM Linux
2010-12-16  9:31 ` [PATCH 12/13] ARM: versatile: " Russell King - ARM Linux
2010-12-16  9:32 ` [PATCH 13/13] ARM: vexpress: add sched_clock() for Versatile Express Russell King - ARM Linux
2010-12-16 13:04 ` [PATCH 0/13] 64-bit sched_clock Russell King - ARM Linux
2010-12-19 22:23   ` Linus Walleij
2010-12-19 23:39     ` Russell King - ARM Linux [this message]

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=20101219233910.GA816@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).