lkml.org 
[lkml]   [2019]   [Nov]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v5 01/16] dt-bindings: regulator: Document ROHM BD71282 regulator bindings
Date
Hello Mark,

It's nice to hear from you again :) I hope you're enjoying all the new
things :)

On Mon, 2019-11-18 at 16:25 +0000, Mark Brown wrote:
> On Mon, Nov 18, 2019 at 08:53:57AM +0200, Matti Vaittinen wrote:
>
> > +#Supported default DVS states:
> > +#buck | run | idle | suspend
> > | lpsr
> > +#-----------------------------------------------------------------
> > -----------
> > +#1, 2, 6, and 7 | supported | supported | supported
> > (*)
> > +#-----------------------------------------------------------------
> > -----------
> > +#3, 4, and 5 | supported (**)
> > +#-----------------------------------------------------------------
> > -----------
> > +#(*) LPSR and SUSPEND states use same voltage but both states
> > have own enable /
> > +# disable settings. Voltage 0 can be specified for a state to
> > make regulator
> > +# disabled on that state.
> > +#(**) All states use same voltage but have own enable / disable
> > settings.
> > +# Voltage 0 can be specified for a state to make regulator
> > disabled on that
> > +# state.
> > +
> > + rohm,dvs-runlvl-ctrl:
> > + description: |
> > + buck control is done based on run-level. Regulator is
> > not
> > + individually controllable. See ../mfd/rohm,bd71828-
> > pmic.yaml for
> > + how to specify run-level control mechanism. Only bucks
> > 1, 2, 6
> > + and 7 support this.
> > + type: boolean
>
> I don't think I saw this having the effect on set_voltage() that I'd
> have expected in the driver?

The support for this is added in patch 12. I should've ordered the
patch series so that all regulator patches were one after another.
Sorry for that.
The patch 12 adds the run-level support. Please see the functions
get_runcontrolled_bucks_dt(),
mark_regulator_runlvl_controlled() (sets the g->runlvl)
and set_buck_runlvl_controlled() (called based on g->runlvl)
which changes the ops to disallow setters and to get voltage based on
current runlevel - and different ops depending on if runlevels are
controlled by GPIO or I2C. Additionally set_buck_runlvl_controlled()
adds DT parsing call-back for setting the initial voltages.

Complex, I know. I'm open to hints,tips and suggestions :)

> > + rohm,dvs-runlevel-microvolts:
> > + minimum: 0
> > + maximum: 2000000
> > + maxItems: 4
> > + description:
> > + Array of voltages for run-levels. First value is for
> > run-level 0,
> > + second for run-level 1 etc. Microvolts.
>
> What's the mapping from array indexes to the names used elsewhere to
> support runlevels?

Hmm. Sorry Mark, I don't think I follow your question. Do you mean
names like LPSR, SUSPEND, IDLE, RUN? If so, then I might need to
rephrase this. The runlevels referred here are different from LPSR,
SUSPEND, IDLE etc. They are actually 'sub-levels' for PMIC's RUN state.
Eg, kind of a 'fast way' to change voltages for multiple power rails
when SoC is at RUN state. The names I have seen are RUN0, RUN1, RUN2
and RUN3. That mapping is described in description above.

Br,
Matti Vaittinen
\
 
 \ /
  Last update: 2019-11-18 19:04    [W:0.129 / U:0.748 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site