Re: DVS regulator drivers

Guennadi Liakhovetski
2012-11-20T10:03:09Z

Sorry, one more point, that I raised in the original mail yesterday and
that I forgot about, when replying today:

On Tue, 20 Nov 2012, Mark Brown wrote:

> On Mon, Nov 19, 2012 at 12:52:09PM +0100, Guennadi Liakhovetski wrote:

[snip]

> > /* Always set the ON status to the minimum voltage */
> >
> > but I actually don't see, where the minimum is selected. It seems instead
> > in this case the "ON" value is just set:

In other words, I don't see where voltages are compared to select the
minimum to be used for .on_vsel. Instead, it seems, .on_vsel is always set
to the new value, and, if it is also higher then the old .dvs_vsel value,
.dvs_vsel is _also_ set to the new voltage, in which case they become
equal. Is this the intended behaviour?