Messages in this thread | | | Date | Tue, 25 Aug 2015 15:53:45 -0700 | From | Stephen Boyd <> | Subject | Re: [PATCH V3] regulator: core: Define regulator_set_voltage_triplet() |
| |
On 08/17, Viresh Kumar wrote: > Voltage tolerance isn't necessarily same on both sides of the target > voltage and regulator_set_voltage_tol() wouldn't be suitable in such > cases. > > Add another routine regulator_set_voltage_triplet(), which accepts > target, min and max voltages as arguments. > > This first tries to set the voltage between the target voltage and the > upper limit, then fall back on the full range. The idea behind this is > to set regulator's voltage as close to the target voltage, as possible. > > Based on regulator_set_voltage_tol(). > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > ---
Looks good to me.
Reviewed-by: Stephen Boyd <sboyd@codeaurora.org>
-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
| |