diff options
| author | Axel Lin <axel.lin@gmail.com> | 2012-04-10 14:20:03 +0800 |
|---|---|---|
| committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-04-10 10:14:01 +0100 |
| commit | 2358b7763ee6673c0d08ddf9dcfe96e982e9b26f (patch) | |
| tree | 33c69d226ba54a28222071360e4f4cd17d5fce2b /drivers/regulator/max8997.c | |
| parent | 0fa310ccabfe9034017e1b6780052d36fbd9f38f (diff) | |
| download | linux-2358b7763ee6673c0d08ddf9dcfe96e982e9b26f.tar.bz2 | |
regulator: max8997: Use simple equation to get selector
It's more efficient to get the best selector by simple equation.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'drivers/regulator/max8997.c')
| -rw-r--r-- | drivers/regulator/max8997.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/drivers/regulator/max8997.c b/drivers/regulator/max8997.c index 89c93d891140..db09244bb3ed 100644 --- a/drivers/regulator/max8997.c +++ b/drivers/regulator/max8997.c @@ -415,7 +415,7 @@ static inline int max8997_get_voltage_proper_val( const struct voltage_map_desc *desc, int min_vol, int max_vol) { - int i = 0; + int i; if (desc == NULL) return -EINVAL; @@ -423,9 +423,10 @@ static inline int max8997_get_voltage_proper_val( if (max_vol < desc->min || min_vol > desc->max) return -EINVAL; - while (desc->min + desc->step * i < min_vol && - desc->min + desc->step * i < desc->max) - i++; + if (min_vol < desc->min) + min_vol = desc->min; + + i = DIV_ROUND_UP(min_vol - desc->min, desc->step); if (desc->min + desc->step * i > max_vol) return -EINVAL; |