From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrice CHOTARD Date: Tue, 10 Sep 2019 09:54:05 +0000 Subject: [U-Boot] [PATCH] regulator: fixed: Modify enable-active-high behavior In-Reply-To: <20190725071239.31398-1-patrice.chotard@st.com> References: <20190725071239.31398-1-patrice.chotard@st.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, It's a gentle reminder to not forgot this patch Thanks Patrice On 7/25/19 9:12 AM, Patrice Chotard wrote: > Regulator should not be enabled at probe time if regulator-boot-on > property is not in the dt node. > > "enable-active-high" property is only used to indicate the GPIO > polarity. > > See kernel documentation : > - Documentation/devicetree/bindings/regulator/fixed-regulator.yaml > - Documentation/devicetree/bindings/regulator/gpio-regulator.yaml > > Signed-off-by: Christophe Kerello > Signed-off-by: Patrice Chotard > --- > > drivers/power/regulator/regulator_common.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/power/regulator/regulator_common.c b/drivers/power/regulator/regulator_common.c > index 3dabbe2a85..2e8a083c75 100644 > --- a/drivers/power/regulator/regulator_common.c > +++ b/drivers/power/regulator/regulator_common.c > @@ -15,7 +15,9 @@ int regulator_common_ofdata_to_platdata(struct udevice *dev, > int flags = GPIOD_IS_OUT; > int ret; > > - if (dev_read_bool(dev, "enable-active-high")) > + if (!dev_read_bool(dev, "enable-active-high")) > + flags |= GPIOD_ACTIVE_LOW; > + if (uc_pdata->boot_on) > flags |= GPIOD_IS_OUT_ACTIVE; > > /* Get optional enable GPIO desc */