With the commit 4fcba5d556b4 ("regulator: implement basic reference
counter") the return value of regulator_set_enable may be EALREADY or
EBUSY for fixed/gpio regulators and may be further expanded on all
regulators.

Change to use the more relaxed regulator_set_enable_if_allowed to
continue if regulator already was enabled or disabled.

Signed-off-by: Svyatoslav Ryhel <clamo...@gmail.com>
---
 drivers/video/pwm_backlight.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/drivers/video/pwm_backlight.c b/drivers/video/pwm_backlight.c
index 46c16a8f44..4b35e2b120 100644
--- a/drivers/video/pwm_backlight.c
+++ b/drivers/video/pwm_backlight.c
@@ -97,7 +97,7 @@ static int enable_sequence(struct udevice *dev, int seq)
                        plat = dev_get_uclass_plat(priv->reg);
                        log_debug("Enable '%s', regulator '%s'/'%s'\n",
                                  dev->name, priv->reg->name, plat->name);
-                       ret = regulator_set_enable(priv->reg, true);
+                       ret = regulator_set_enable_if_allowed(priv->reg, true);
                        if (ret) {
                                log_debug("Cannot enable regulator for PWM 
'%s'\n",
                                          dev->name);
@@ -178,11 +178,10 @@ static int pwm_backlight_set_brightness(struct udevice 
*dev, int percent)
        }
        if (disable) {
                dm_gpio_set_value(&priv->enable, 0);
-               if (priv->reg) {
-                       ret = regulator_set_enable(priv->reg, false);
-                       if (ret)
-                               return log_ret(ret);
-               }
+               ret = regulator_set_enable_if_allowed(priv->reg, false);
+               if (ret)
+                       return log_ret(ret);
+
                priv->enabled = false;
        }
 
-- 
2.39.2

Reply via email to