On Wed, Jul 18, 2012 at 08:55:30AM +0100, Graeme Gregory wrote:
> On 18/07/12 04:30, Axel Lin wrote:
> > This patch fixes below issues when choosing selector:

> > 1. Current code returns negative selector if min_uV < 900000 which is wrong.
> >    For example, it is possible to satisfy the request with selector = 1 if
> >    the requested min_uV is 850000.

> Isnt this defending against bugs in the regulator core? I am normally
> against repeating the same checks at multiple levels in a stack.

This isn't a bug in the core, a negative selector is an error code (and
in general whatever selector is returned by the driver ought to agree
with selectors the driver actually supports).  It's probably at least a
questionable decision in the machine constraints to allow a request for
less than the regulator can deliver to get through though.