Commit fa93fd4e authored by Mark Brown's avatar Mark Brown

regulator: core: Ensure we are at least in bounds for our constraints

Currently we only attempt to set the voltage during constraints
application if an exact voltage is specified.  Extend this so that if
the currently set voltage for the regulator is outside the bounds set in
constraints we will move the voltage to the nearest constraint, raising
to the minimum or lowering to the maximum as needed.  This ensures that
drivers can probe without the hardware being driven out of spec.
Reported-by: default avatarIvaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
Tested-by: default avatarIvaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 895fe232
...@@ -906,7 +906,8 @@ static int machine_constraints_voltage(struct regulator_dev *rdev, ...@@ -906,7 +906,8 @@ static int machine_constraints_voltage(struct regulator_dev *rdev,
/* do we need to apply the constraint voltage */ /* do we need to apply the constraint voltage */
if (rdev->constraints->apply_uV && if (rdev->constraints->apply_uV &&
rdev->constraints->min_uV == rdev->constraints->max_uV) { rdev->constraints->min_uV && rdev->constraints->max_uV) {
int target_min, target_max;
int current_uV = _regulator_get_voltage(rdev); int current_uV = _regulator_get_voltage(rdev);
if (current_uV < 0) { if (current_uV < 0) {
rdev_err(rdev, rdev_err(rdev,
...@@ -914,15 +915,32 @@ static int machine_constraints_voltage(struct regulator_dev *rdev, ...@@ -914,15 +915,32 @@ static int machine_constraints_voltage(struct regulator_dev *rdev,
current_uV); current_uV);
return current_uV; return current_uV;
} }
if (current_uV < rdev->constraints->min_uV ||
current_uV > rdev->constraints->max_uV) { /*
* If we're below the minimum voltage move up to the
* minimum voltage, if we're above the maximum voltage
* then move down to the maximum.
*/
target_min = current_uV;
target_max = current_uV;
if (current_uV < rdev->constraints->min_uV) {
target_min = rdev->constraints->min_uV;
target_max = rdev->constraints->min_uV;
}
if (current_uV > rdev->constraints->max_uV) {
target_min = rdev->constraints->max_uV;
target_max = rdev->constraints->max_uV;
}
if (target_min != current_uV || target_max != current_uV) {
ret = _regulator_do_set_voltage( ret = _regulator_do_set_voltage(
rdev, rdev->constraints->min_uV, rdev, target_min, target_max);
rdev->constraints->max_uV);
if (ret < 0) { if (ret < 0) {
rdev_err(rdev, rdev_err(rdev,
"failed to apply %duV constraint(%d)\n", "failed to apply %d-%duV constraint(%d)\n",
rdev->constraints->min_uV, ret); target_min, target_max, ret);
return ret; return ret;
} }
} }
......
...@@ -43,7 +43,7 @@ static void of_get_regulation_constraints(struct device_node *np, ...@@ -43,7 +43,7 @@ static void of_get_regulation_constraints(struct device_node *np,
constraints->max_uV = pval; constraints->max_uV = pval;
/* Voltage change possible? */ /* Voltage change possible? */
if (constraints->min_uV != constraints->max_uV) { if (constraints->min_uV && constraints->max_uV) {
constraints->valid_ops_mask |= REGULATOR_CHANGE_VOLTAGE; constraints->valid_ops_mask |= REGULATOR_CHANGE_VOLTAGE;
constraints->apply_uV = true; constraints->apply_uV = true;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment