On Mon, Nov 02, 2015 at 03:18:00PM +0530, Saurabh Sengar wrote:
> replacing the of_get_property function with of_property_read_u32 function
> as its help removing two variables.

> I have put these check back, please let me know if this v2 patch is worth

It's a bit marginal TBH and...

>       /* Only one voltage?  Then make sure it's set. */
> -     if (min_uV && max_uV && constraints->min_uV == constraints->max_uV)
> +     if (constraints->min_uV && constraints->max_uV
> +                     && constraints->min_uV == constraints->max_uV)
>               constraints->apply_uV = true;

...this does disallow a constraint of 0-XuV which was previously legal
and potentially meaningful.  Hrm.

Attachment: signature.asc
Description: PGP signature

Reply via email to