Messages in this thread | | | Date | Mon, 3 Nov 2014 11:00:12 +0100 | From | Markus Pargmann <> | Subject | Re: [PATCH] regulator: adding disable options for regulator-always-on and regulator-boots-on |
| |
Hi,
On Mon, Nov 03, 2014 at 10:26:43AM +0900, Hugh Kang wrote: > From: "hugh.kang" <hugh.kang@lge.com> > > If a regulator is set by always-on option, the regulator will be set forever. > For example, suppose LDO1 is set to always-on at RevA.dts with including of a.dtsi. After that > RevB.dts may wants to include the same a.dtsi but override the LDO1 always-on option. However, > currently there is no way to delete the always-on option, even when we change the LDO1 option value, > the always-on setting is still remains.
Why don't you split your devicetrees according to the real hardware and set the always-on property only for revA?
Regards,
Markus
> > Signed-off-by: hugh.kang <hugh.kang@lge.com> > --- > drivers/regulator/of_regulator.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/regulator/of_regulator.c b/drivers/regulator/of_regulator.c > index ee5e67b..4178dbd 100644 > --- a/drivers/regulator/of_regulator.c > +++ b/drivers/regulator/of_regulator.c > @@ -52,7 +52,13 @@ static void of_get_regulation_constraints(struct device_node *np, > constraints->valid_ops_mask |= REGULATOR_CHANGE_CURRENT; > > constraints->boot_on = of_property_read_bool(np, "regulator-boot-on"); > + if (of_property_read_bool(np, "regulator-disable-boot-on")) > + constraints->boot_on = false; > + > constraints->always_on = of_property_read_bool(np, "regulator-always-on"); > + if (of_property_read_bool(np, "regulator-disable-always-on")) > + constraints->always_on = false; > + > if (!constraints->always_on) /* status change should be possible. */ > constraints->valid_ops_mask |= REGULATOR_CHANGE_STATUS; > > -- > 1.8.3.2 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ >
-- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | [unhandled content-type:application/pgp-signature] | |