power: regulator: denied disable on always-on regulator
authorPatrick Delaunay <patrick.delaunay@st.com>
Thu, 15 Nov 2018 12:45:31 +0000 (13:45 +0100)
committerSimon Glass <sjg@chromium.org>
Wed, 5 Dec 2018 13:06:44 +0000 (06:06 -0700)
Don't disable regulator which are tagged as "regulator-always-on" in DT.

Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jack Mitchell <jack@embed.me.uk>
Tested-by: Jack Mitchell <jack@embed.me.uk>
Signed-off-by: Patrice Chotard <patrice.chotard@st.com>
Reviewed-by: Richard Röjfors <richard@puffinpack.se>
Tested-by: Richard Röjfors <richard@puffinpack.se>
Reviewed-by: Felix Brack <fb@ltec.ch>
Tested-by: Felix Brack <fb@ltec.ch>
drivers/power/regulator/regulator-uclass.c

index 4da8e43..4511625 100644 (file)
@@ -106,10 +106,15 @@ int regulator_get_enable(struct udevice *dev)
 int regulator_set_enable(struct udevice *dev, bool enable)
 {
        const struct dm_regulator_ops *ops = dev_get_driver_ops(dev);
+       struct dm_regulator_uclass_platdata *uc_pdata;
 
        if (!ops || !ops->set_enable)
                return -ENOSYS;
 
+       uc_pdata = dev_get_uclass_platdata(dev);
+       if (!enable && uc_pdata->always_on)
+               return -EACCES;
+
        return ops->set_enable(dev, enable);
 }