commit f55205f4d4a8823a11bb8b37ef2ecbd78fb09463
author Axel Lin <axel.lin@gmail.com> Wed Apr 11 20:53:58 2012 +0800
committer Mark Brown <broonie@opensource.wolfsonmicro.com> Tue Apr 17 15:17:11 2012 +0100
tree acb5c8624f1af12cc9bb5235c5bc586b92efa832
parent e816b57a337ea3b755de72bec38c10c864f23015

regulator: Fix the logic to ensure new voltage setting in valid range

I think this is a typo.
To ensure new voltage setting won't greater than desc->max,
the equation should be desc->min + desc->step * new_val <= desc->max.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: stable@vger.kernel.org

drivers/regulator/max8997.c

diff --git a/drivers/regulator/max8997.c b/drivers/regulator/max8997.c
index 9657929..17a58c5 100644
--- a/drivers/regulator/max8997.c
+++ b/drivers/regulator/max8997.c
@@ -684,7 +684,7 @@
 		}
 
 		new_val++;
-	} while (desc->min + desc->step + new_val <= desc->max);
+	} while (desc->min + desc->step * new_val <= desc->max);
 
 	new_idx = tmp_idx;
 	new_val = tmp_val;