Message ID | 1344179121-17738-4-git-send-email-shawn.guo@linaro.org (mailing list archive) |
---|---|
State | Superseded, archived |
Headers | show |
On Sun, Aug 05, 2012 at 11:05:20PM +0800, Shawn Guo wrote: > There are some use cases where a voltage range could be reasonably > specified by a target voltage and tolerance. Add a new API > regulator_set_voltage_tol() wrapping regulator_set_voltage() call > to ease the users. Applied, thanks! It's on a separate branch so can easily be either applied somewhere else or merged into another tree if required for dependencies, just let me know. -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Aug 08, 2012 at 02:35:40PM +0100, Mark Brown wrote: > Applied, thanks! It's on a separate branch so can easily be either > applied somewhere else or merged into another tree if required for > dependencies, just let me know. I intended to have the whole series go via cpufreq tree with your ack on this patch. But I could also ask Rafael to merge your branch to resolve the dependency.
On Wed, Aug 08, 2012 at 10:30:07PM +0800, Shawn Guo wrote: > On Wed, Aug 08, 2012 at 02:35:40PM +0100, Mark Brown wrote: > > Applied, thanks! It's on a separate branch so can easily be either > > applied somewhere else or merged into another tree if required for > > dependencies, just let me know. > I intended to have the whole series go via cpufreq tree with your ack > on this patch. But I could also ask Rafael to merge your branch to > resolve the dependency. I anticipate wanting the new API in other trees too (I have one likely use in mind) so having it only in the cpufreq tree would be inconvenient. The whole reason to have this in the regulator API is that it's something that's useful outside your cpufreq code! -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/include/linux/regulator/consumer.h b/include/linux/regulator/consumer.h index da339fd..2160352 100644 --- a/include/linux/regulator/consumer.h +++ b/include/linux/regulator/consumer.h @@ -352,4 +352,11 @@ static inline void regulator_set_drvdata(struct regulator *regulator, #endif +static inline int regulator_set_voltage_tol(struct regulator *regulator, + int new_uV, int tol_uV) +{ + return regulator_set_voltage(regulator, + new_uV - tol_uV, new_uV + tol_uV); +} + #endif
There are some use cases where a voltage range could be reasonably specified by a target voltage and tolerance. Add a new API regulator_set_voltage_tol() wrapping regulator_set_voltage() call to ease the users. Signed-off-by: Shawn Guo <shawn.guo@linaro.org> --- include/linux/regulator/consumer.h | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-)