提交 43fc99f2 编写于 作者: M Mark Brown

regulator: core: Only propagate voltage changes to if it can change voltages

When we are propagating voltage changes to parent regulators don't
bother if the parent does not have permission to change voltages.  This
simplifies error checking in the function for cases where the regulator
lacks some of the voltage operations.
Reported-by: NDong Aisheng <aisheng.dong@nxp.com>
Tested-by: NDong Aisheng <aisheng.dong@nxp.com>
Reviewed-by: NDong Aisheng <aisheng.dong@nxp.com>
Signed-off-by: NMark Brown <broonie@kernel.org>
上级 fd086045
...@@ -2939,8 +2939,10 @@ static int regulator_set_voltage_unlocked(struct regulator *regulator, ...@@ -2939,8 +2939,10 @@ static int regulator_set_voltage_unlocked(struct regulator *regulator,
if (ret < 0) if (ret < 0)
goto out2; goto out2;
if (rdev->supply && (rdev->desc->min_dropout_uV || if (rdev->supply &&
!rdev->desc->ops->get_voltage)) { regulator_ops_is_valid(rdev->supply->rdev,
REGULATOR_CHANGE_VOLTAGE) &&
(rdev->desc->min_dropout_uV || !rdev->desc->ops->get_voltage)) {
int current_supply_uV; int current_supply_uV;
int selector; int selector;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册