Re: [U-Boot] [PATCH] regulator: Allow autosetting fixed regulators

31 May 2019

...
How about the following patch? not tested

diff --git a/drivers/power/regulator/regulator-uclass.c b/drivers/power/regulator/regulator-uclass.c
index 9118b8eb39..76be95bcd1 100644
--- a/drivers/power/regulator/regulator-uclass.c
+++ b/drivers/power/regulator/regulator-uclass.c
@@ -238,6 +238,9 @@ int regulator_autoset(struct udevice *dev)
       if (!uc_pdata->always_on && !uc_pdata->boot_on)
               return -EMEDIUMTYPE;

  if (uc_pdata->type == REGULATOR_TYPE_FIXED)


          return regulator_set_enable(dev, true);


 if (uc_pdata->flags & REGULATOR_FLAG_AUTOSET_UV)
         ret = regulator_set_value(dev, uc_pdata->min_uV);
 if (!ret && (uc_pdata->flags & REGULATOR_FLAG_AUTOSET_UA))



That will probably work as well. It does, however, feel a little like a hack. What if there are other regulator types that require a similar workaround?
Sven

    

Sven Schwermer