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

Groups > linux.kernel > #1519120

Re: [PATCH V3 1/9] PM / OPP: Reword binding supporting multiple regulators per device

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Thu, Nov 10, 2016 at 09:34:40AM +0530, Viresh Kumar wrote:
> On 09-11-16, 14:58, Mark Brown wrote:
> > On Wed, Oct 26, 2016 at 12:02:56PM +0530, Viresh Kumar wrote:

> > > +  Entries for multiple regulators shall be provided in the same field separated
> > > +  by angular brackets <>. The OPP binding doesn't provide any provisions to
> > > +  relate the values to their power supplies or the order in which the supplies
> > > +  need to be configured.

> > I don't understand how this works.  If we have an unordered list of
> > values to set for regulators how will we make sense of them?

> The platform driver is responsible to identify the order and pass it on to the
> OPP core. And the platform driver needs to have that hard coded.

That *really* should be in the binding.  Honestly if the binding is this
vague I'm not even clear that it's worth documenting these properties at
this level, might be better to just put the documentation in the
platform driver bindings.

> > > -			cpu-supply = <&cpu_supply0>, <&cpu_supply1>, <&cpu_supply2>;
> > > +			vcc0-supply = <&cpu_supply0>;
> > > +			vcc1-supply = <&cpu_supply1>;
> > > +			vcc2-supply = <&cpu_supply2>;

> > This change doesn't seem to correspond to the documentation change.

> This rectifies the incorrect binding previously added to the example, which I
> realized to be incorrect only while attempting to code for it. And so it brings
> the example on the same state as the documentation now.

Then that should be in a separate patch with a changelog explaining what
the change is doing.

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

Thread

Re: [PATCH V3 1/9] PM / OPP: Reword binding supporting multiple  regulators per device Mark Brown <broonie@kernel.org> - 2016-11-09 16:00 +0100
  Re: [PATCH V3 1/9] PM / OPP: Reword binding supporting multiple  regulators per device Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-10 05:10 +0100
    Re: [PATCH V3 1/9] PM / OPP: Reword binding supporting multiple  regulators per device Mark Brown <broonie@kernel.org> - 2016-11-10 17:40 +0100
      Re: [PATCH V3 1/9] PM / OPP: Reword binding supporting multiple  regulators per device Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-10 19:10 +0100
        Re: [PATCH V3 1/9] PM / OPP: Reword binding supporting multiple  regulators per device Stephen Boyd <sboyd@codeaurora.org> - 2016-11-11 00:00 +0100
          Re: [PATCH V3 1/9] PM / OPP: Reword binding supporting multiple  regulators per device Viresh Kumar <viresh.kumar@linaro.org> - 2016-11-11 04:20 +0100

csiph-web