[PATCH] regulator: fixed: take regulator-boot-on into account

[PATCH] regulator: fixed: take regulator-boot-on into account

[Marco Felsch]

There are regulators which are turned on by default (hw-design) and
shouldn't be disabled during the boot. Upstream uses the
regulator-boot-on dt-property for it:

8<--------------------------------------------------
  regulator-boot-on:
    description: bootloader/firmware enabled regulator
    type: boolean
8<--------------------------------------------------

The difference between boot-on and always-on is the linux suspend
handling. Regulators marked as boot-on can be disabled during suspend.
This case isn't interesting for barebox so we can mark those as
always-on too.

Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
---
 drivers/regulator/fixed.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c
index cb5d785817..78b8290ff2 100644
--- a/drivers/regulator/fixed.c
+++ b/drivers/regulator/fixed.c
@@ -86,7 +86,8 @@ static int regulator_fixed_probe(struct device_d *dev)
 	fix->rdesc.ops = &fixed_ops;
 	fix->rdev.desc = &fix->rdesc;
 
-	if (of_find_property(dev->device_node, "regulator-always-on", NULL)) {
+	if (of_find_property(dev->device_node, "regulator-always-on", NULL) ||
+	    of_find_property(dev->device_node, "regulator-boot-on", NULL)) {
 		fix->always_on = 1;
 		regulator_fixed_enable(&fix->rdev);
 	}
-- 
2.20.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] regulator: fixed: take regulator-boot-on into account

[Sascha Hauer]

On Wed, Jan 29, 2020 at 12:38:23PM +0100, Marco Felsch wrote:
> There are regulators which are turned on by default (hw-design) and
> shouldn't be disabled during the boot. Upstream uses the
> regulator-boot-on dt-property for it:
> 
> 8<--------------------------------------------------
>   regulator-boot-on:
>     description: bootloader/firmware enabled regulator
>     type: boolean
> 8<--------------------------------------------------
> 
> The difference between boot-on and always-on is the linux suspend
> handling. Regulators marked as boot-on can be disabled during suspend.
> This case isn't interesting for barebox so we can mark those as
> always-on too.
> 
> Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
> ---
>  drivers/regulator/fixed.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Applied, thanks

Sascha


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox