Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1528101

Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state binding

Show all headers | View raw

Thanks for explaining on my behalf Vincent :)

On 22-11-16, 19:34, Vincent Guittot wrote:
> On 22 November 2016 at 19:12, Kevin Hilman <khilman@baylibre.com> wrote:
> > I think the question is: what does the performance-level of a domain
> > actually mean?  Or, what are the units?

There is no unit. If we have units like Hz and volts etc, then we can actually
use the existing clk/regulator frameworks straight away.

The whole problem here is that the regulator (and maybe the clock on a different
platform) for a power domain are hidden from the kernel and handled by a black
box (An M3 core in my case). All we can ask is for a performance state, a simple
positive integer value.

> > Depending on the SoC, there's probably a few things this could mean.  It
> > might mean is that an underlying bus/interconnect can be configured to
> > guarantee a specific bandwidth or throughput.

We are talking in terms of power domains here and so if the bus/interconnect has
a power domain for itself, then yes we can very much have that situation. But if
the kernel have the capability of configuring clk and voltages directly, then we
don't need this new infrastructure at all.

> > That in turn might mean
> > that that bus/interconnect might have to be set at a specific
> > frequency/voltage.
> >
> > In your case, IIUC, you're just passing some magic value to some
> > firmware running on a micro-controller, but under the hood that uC is
> > probably configuring a frequency/voltage someplace.
> 
> In the case described by Viresh, it's only about setting the voltage
> of a power domain that is shared between different devices. these
> devices wants to run at different frequency (set by the devices) but
> we have to select a Volateg value that will match with the constraint
> of all devices (in this case the highest voltage)

That's right.

> > So, if we're going to have a generic DT binding for this, it needs to be
> > something that's useful on platforms that are not using magic numbers
> > managed by a uC as well.

What suggestions do you have for this and I am not sure what all cases we want
to solve by this ?

-- 
viresh

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] PM / Domains: Introduce domain-performance-state binding Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-18 10:30 +0100
  Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state  binding Rob Herring <robh@kernel.org> - 2016-11-21 16:10 +0100
    Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state  binding Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-22 04:20 +0100
      Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state binding Kevin Hilman <khilman@baylibre.com> - 2016-11-22 19:20 +0100
        Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state binding Vincent Guittot <vincent.guittot@linaro.org> - 2016-11-22 19:50 +0100
          Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state  binding Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-23 04:30 +0100
          Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state binding Vincent Guittot <vincent.guittot@linaro.org> - 2016-11-23 17:00 +0100
            Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state binding Kevin Hilman <khilman@baylibre.com> - 2016-11-23 23:40 +0100
              Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state  binding Stephen Boyd <sboyd@codeaurora.org> - 2016-11-24 03:10 +0100
                Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state  binding Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-24 05:50 +0100
          Re: [PATCH 1/2] PM / Domains: Introduce domain-performance-state binding Kevin Hilman <khilman@baylibre.com> - 2016-11-23 17:00 +0100

csiph-web