regulator: core: Fix voltage change propagations to supply regulators
authorTirupathi Reddy <tirupath@codeaurora.org>
Thu, 25 May 2017 10:03:17 +0000 (15:33 +0530)
committerMark Brown <broonie@kernel.org>
Wed, 28 Jun 2017 19:39:36 +0000 (20:39 +0100)
Some regulators support get_voltage() and some support get_voltage_sel()
operations but currently we only propagate changes if the regulator has
a get_voltage() operation.  Also do this if we've got get_voltage_sel()

[Rewite commit message for clarity -- broonie]

Signed-off-by: Tirupathi Reddy <tirupath@codeaurora.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/core.c

index c0d9ae8d0860e8e9466dee6fe1e275853b1f9e3b..9fecbd4e3546aeff60f9eddc6447b2d2d7c0981d 100644 (file)
@@ -2938,7 +2938,8 @@ static int regulator_set_voltage_unlocked(struct regulator *regulator,
        if (rdev->supply &&
            regulator_ops_is_valid(rdev->supply->rdev,
                                   REGULATOR_CHANGE_VOLTAGE) &&
-           (rdev->desc->min_dropout_uV || !rdev->desc->ops->get_voltage)) {
+           (rdev->desc->min_dropout_uV || !(rdev->desc->ops->get_voltage ||
+                                          rdev->desc->ops->get_voltage_sel))) {
                int current_supply_uV;
                int selector;