From: Keerthy Date: Wed, 26 Oct 2016 08:12:31 +0000 (+0530) Subject: power: regulator: Add limits checking while setting voltage X-Git-Tag: v2017.01-rc1~210^2~30 X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=eaadcf38ddfe23aee9c3a85a96d4103f5398fc5e;p=u-boot power: regulator: Add limits checking while setting voltage Currently the specific set ops functions are directly called without any check for voltage limits for a regulator. Check for them and proceed. Signed-off-by: Keerthy Reviewed-by: Simon Glass Fixed checking of voltate limits: Signed-off-by: Simon Glass --- diff --git a/drivers/power/regulator/regulator-uclass.c b/drivers/power/regulator/regulator-uclass.c index d64400981f..e48d213288 100644 --- a/drivers/power/regulator/regulator-uclass.c +++ b/drivers/power/regulator/regulator-uclass.c @@ -41,6 +41,13 @@ int regulator_get_value(struct udevice *dev) int regulator_set_value(struct udevice *dev, int uV) { const struct dm_regulator_ops *ops = dev_get_driver_ops(dev); + struct dm_regulator_uclass_platdata *uc_pdata; + + uc_pdata = dev_get_uclass_platdata(dev); + if (uc_pdata->min_uV != -ENODATA && uV < uc_pdata->min_uV) + return -EINVAL; + if (uc_pdata->max_uV != -ENODATA && uV > uc_pdata->max_uV) + return -EINVAL; if (!ops || !ops->set_value) return -ENOSYS;