regulator: core: Use local ops variable in _regulator_do_set_voltage()
authorMatthias Kaehlcke <mka@chromium.org>
Wed, 14 Sep 2016 16:52:05 +0000 (09:52 -0700)
committerMark Brown <broonie@kernel.org>
Wed, 14 Sep 2016 17:12:20 +0000 (18:12 +0100)
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/core.c

index db320e8..b059e83 100644 (file)
@@ -2751,6 +2751,7 @@ static int _regulator_do_set_voltage(struct regulator_dev *rdev,
        int best_val = 0;
        unsigned int selector;
        int old_selector = -1;
+       const struct regulator_ops *ops = rdev->desc->ops;
 
        trace_regulator_set_voltage(rdev_get_name(rdev), min_uV, max_uV);
 
@@ -2762,29 +2763,28 @@ static int _regulator_do_set_voltage(struct regulator_dev *rdev,
         * info to call set_voltage_time_sel().
         */
        if (_regulator_is_enabled(rdev) &&
-           rdev->desc->ops->set_voltage_time_sel &&
-           rdev->desc->ops->get_voltage_sel) {
-               old_selector = rdev->desc->ops->get_voltage_sel(rdev);
+           ops->set_voltage_time_sel && ops->get_voltage_sel) {
+               old_selector = ops->get_voltage_sel(rdev);
                if (old_selector < 0)
                        return old_selector;
        }
 
-       if (rdev->desc->ops->set_voltage) {
+       if (ops->set_voltage) {
                ret = _regulator_call_set_voltage(rdev, min_uV, max_uV,
                                                  &selector);
 
                if (ret >= 0) {
-                       if (rdev->desc->ops->list_voltage)
-                               best_val = rdev->desc->ops->list_voltage(rdev,
-                                                                        selector);
+                       if (ops->list_voltage)
+                               best_val = ops->list_voltage(rdev,
+                                                            selector);
                        else
                                best_val = _regulator_get_voltage(rdev);
                }
 
-       } else if (rdev->desc->ops->set_voltage_sel) {
+       } else if (ops->set_voltage_sel) {
                ret = regulator_map_voltage(rdev, min_uV, max_uV);
                if (ret >= 0) {
-                       best_val = rdev->desc->ops->list_voltage(rdev, ret);
+                       best_val = ops->list_voltage(rdev, ret);
                        if (min_uV <= best_val && max_uV >= best_val) {
                                selector = ret;
                                if (old_selector == selector)
@@ -2804,7 +2804,7 @@ static int _regulator_do_set_voltage(struct regulator_dev *rdev,
        if (ret == 0 && !rdev->constraints->ramp_disable && old_selector >= 0
                && old_selector != selector) {
 
-               delay = rdev->desc->ops->set_voltage_time_sel(rdev,
+               delay = ops->set_voltage_time_sel(rdev,
                                                old_selector, selector);
                if (delay < 0) {
                        rdev_warn(rdev, "set_voltage_time_sel() failed: %d\n",