public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@sirena.org.uk>
To: Colin Cross <ccross@google.com>
Cc: Nishanth Menon <nm@ti.com>, Len Brown <len.brown@intel.com>,
	Kevin Hilman <khilman@deeprootsystems.com>,
	Greg Kroah-Hartman <gregkh@suse.de>,
	Kyungmin Park <kyungmin.park@samsung.com>,
	MyungJoo Ham <myungjoo.ham@samsung.com>,
	linux-pm@lists.linux-foundation.org,
	Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [RFC PATCH] PM: Introduce generic DVFS framework	with device-specific OPPs
Date: Wed, 27 Apr 2011 19:34:30 +0100	[thread overview]
Message-ID: <20110427183430.GC12249@sirena.org.uk> (raw)
In-Reply-To: <BANLkTi=GVZnZuXmbK54Swi8Nv9ZZGhZcNw@mail.gmail.com>

On Wed, Apr 27, 2011 at 10:49:47AM -0700, Colin Cross wrote:

> Moreover, from a silicon perspective, there is always a simple link
> from a single frequency to a minimum voltage for a given circuit.
> There is no need to group them into OPPs, which seem to have a group
> of clocks and their frequencies that map to a single voltage.  That is
> an artifact of the way TI specifies voltages.

This isn't just a TI thing, other CPU (and general big digital) vendors
spec things similarly.  It's just that TI have more code in mainline
than most.  My understanding is that when people do this the set of
operating points aren't purely about DVFS, they're also about specifying
the relationships between the various system clock rates and potentially
other things which are supported, especially in the blocks that mediate
between multiple domains.  The voltages are just one of the parameters
here, and with multiple supplies the voltages aren't always orthogonal
to each other.

> I proposed in a different thread on LKML that DVFS be handled within
> the generic clock implementation.  Platforms would register a
> regulator and a table of voltages for each struct clock that required
> DVFS, and the voltages would be changed on normal clk_* requests.
> This maintains compatibility with existing clk_* calls.

This comes back to the thing we were discussing in the thread there
about there being n:m constraints between settings.  I've not looked at
this in any detail but it may be that for the systems which spec things
as operating points we want to root the core clocking in an operating
point block which deals with stuff like this.

  parent reply	other threads:[~2011-04-27 18:34 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-26  8:43 [RFC PATCH] PM: Introduce generic DVFS framework with device-specific OPPs MyungJoo Ham
2011-04-26 11:09 ` Rafael J. Wysocki
2011-04-27  5:22   ` MyungJoo Ham
2011-04-27 13:46     ` Rafael J. Wysocki
2011-04-28  6:54       ` MyungJoo Ham
2011-04-28 19:19         ` Rafael J. Wysocki
2011-04-27 17:49     ` Colin Cross
2011-04-27 18:07       ` Menon, Nishanth
     [not found]       ` <BANLkTin4rrniJpOheLH=JE0jq7vFpdDbrA@mail.gmail.com>
2011-04-27 18:29         ` Colin Cross
     [not found]         ` <BANLkTi=WWASxk3Fz83324k40Lbq+ccAPeA@mail.gmail.com>
2011-04-27 18:37           ` Colin Cross
2011-04-27 18:48           ` Menon, Nishanth
     [not found]           ` <BANLkTikEtTDp3Cpj0D5z34wA0_G_DWwKFg@mail.gmail.com>
2011-04-27 18:54             ` Colin Cross
2011-04-27 19:26             ` Thomas Gleixner
     [not found]             ` <alpine.LFD.2.02.1104272104390.3323@ionos>
2011-04-27 20:48               ` Colin Cross
     [not found]               ` <BANLkTinWD0ktYKzR_jnQSE76AuW+Z1PSLw@mail.gmail.com>
2011-04-28  6:12                 ` MyungJoo Ham
     [not found]                 ` <BANLkTi=Be3DrAUoPmgHpjhAVmN-Pi5Au7g@mail.gmail.com>
2011-04-28  6:44                   ` Colin Cross
     [not found]                   ` <BANLkTiniJUgTTDh7SF8yNMeSMa+JYM21wA@mail.gmail.com>
2011-04-28  6:50                     ` MyungJoo Ham
     [not found]                     ` <BANLkTimzWyvcBV2ymxaMf=HmRhGYBboB1w@mail.gmail.com>
2011-04-28  7:06                       ` Colin Cross
     [not found]                       ` <BANLkTik6L8HkJ0YjQYjh_3eEmbffyk1kRA@mail.gmail.com>
2011-04-28  7:34                         ` MyungJoo Ham
2011-04-28 12:19                 ` Mark Brown
     [not found]           ` <BANLkTinP0_qpF3sp8CzDy42i24wqqo7ohw@mail.gmail.com>
2011-04-28  5:59             ` MyungJoo Ham
     [not found]             ` <BANLkTin-OMyrz=McN6X4TAZrvv3ehVBGew@mail.gmail.com>
2011-04-28  6:43               ` Colin Cross
     [not found]               ` <BANLkTikVHs1n8fXEAZh6Q4io39eD9PG14w@mail.gmail.com>
2011-04-28  7:16                 ` MyungJoo Ham
2011-04-27 19:00         ` Thomas Gleixner
2011-04-27 18:34       ` Mark Brown [this message]
2011-04-27 18:45         ` Colin Cross
2011-04-27 18:58           ` Mark Brown
2011-04-26 13:22 ` Greg KH
2011-04-26 14:45   ` Jiejing.Zhang 
2011-04-26 20:54     ` Greg KH
2011-04-27  5:33       ` MyungJoo Ham
2011-04-27 13:49         ` Rafael J. Wysocki
2011-04-27 13:36       ` Mark Brown
2011-04-26 19:45   ` Pavel Machek
2011-04-26 15:07 ` Mark Brown
2011-04-27  5:24   ` MyungJoo Ham

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=20110427183430.GC12249@sirena.org.uk \
    --to=broonie@sirena.org.uk \
    --cc=ccross@google.com \
    --cc=gregkh@suse.de \
    --cc=khilman@deeprootsystems.com \
    --cc=kyungmin.park@samsung.com \
    --cc=len.brown@intel.com \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=myungjoo.ham@samsung.com \
    --cc=nm@ti.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox