mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Brownell <david-b@pacbell.net>
To: Mark Brown <broonie@sirena.org.uk>
Cc: Liam Girdwood <lrg@slimlogic.co.uk>,
	lkml <linux-kernel@vger.kernel.org>,
	davej@redhat.com
Subject: Re: [patch 2.6.28-rc7] regulator: catch some registration errors
Date: Mon, 5 Jan 2009 15:45:24 -0800	[thread overview]
Message-ID: <200901051545.24605.david-b@pacbell.net> (raw)
In-Reply-To: <20081204111206.GA8019@sirena.org.uk>

On Thursday 04 December 2008, Mark Brown wrote:
> [ Re Liam's comments about those non-existent cpufreq+regulator drivers ]
>
> On the bright side, looking at the situation with the clock API there
> don't seem to be any other substantial offenders here.  Most of the
> other users are part of the platform code and get to peer inside the
> clocking structure, so if cpufreq were fixed there shouldn't be much
> problem here.

Last time I looked, no cpufreq driver tried to use <linux/clk.h>
calls.  The reason was simple:  the clock framework code sits in
DRAM, so it can't be executed while changing DRAM clocks.

Those bits of cpufreq had to live in on-chip SRAM (there's usually
a few pages of it) and directly update PLL and other clock config
registers ... then wait for things to stabilize before they returned
and the CPU executed from DRAM again.

- Dave
 



  reply	other threads:[~2009-01-06  0:15 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-01 21:35 [patch 2.6.28-rc6+] regulator: bugfixes and messaging cleanup David Brownell
2008-12-01 22:53 ` Mark Brown
2008-12-01 23:58   ` David Brownell
2008-12-02  1:22     ` Mark Brown
2008-12-02  5:50   ` [patch 2.6.28-rc7] regulator: init/link earlier David Brownell
2008-12-02 10:51     ` Mark Brown
2008-12-03 21:45     ` Liam Girdwood
2008-12-02  5:50   ` [patch 2.6.28-rc7] regulator: catch some registration errors David Brownell
2008-12-02 13:32     ` Mark Brown
2009-01-05 23:45       ` David Brownell
2009-01-06 10:10         ` Mark Brown
2009-01-06 10:21           ` David Brownell
2009-01-06 12:58             ` Mark Brown
2008-12-03 21:48     ` Liam Girdwood
2008-12-04 11:12       ` Mark Brown
2009-01-05 23:45         ` David Brownell [this message]
2009-01-06 10:33           ` Mark Brown

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=200901051545.24605.david-b@pacbell.net \
    --to=david-b@pacbell.net \
    --cc=broonie@sirena.org.uk \
    --cc=davej@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lrg@slimlogic.co.uk \
    /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

Powered by JetHome