diff mbox

[U-Boot,11/12] power: regulator: lp873x: get_enable should return integer

Message ID 1497327836-20900-12-git-send-email-j-keerthy@ti.com
State Accepted
Commit 43d0247e3ef8e2f6551026d298dea84d9c7fb2d7
Delegated to: Jaehoon Chung
Headers show

Commit Message

J, KEERTHY June 13, 2017, 4:23 a.m. UTC
get_enable should be able to return error values. Hence change
the return type to integer.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 drivers/power/regulator/lp873x_regulator.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Simon Glass June 17, 2017, 3:41 a.m. UTC | #1
On 12 June 2017 at 22:23, Keerthy <j-keerthy@ti.com> wrote:
> get_enable should be able to return error values. Hence change
> the return type to integer.
>
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> ---
>  drivers/power/regulator/lp873x_regulator.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox

Patch

diff --git a/drivers/power/regulator/lp873x_regulator.c b/drivers/power/regulator/lp873x_regulator.c
index dcb19ff..11371a7 100644
--- a/drivers/power/regulator/lp873x_regulator.c
+++ b/drivers/power/regulator/lp873x_regulator.c
@@ -256,7 +256,7 @@  static int ldo_set_value(struct udevice *dev, int uV)
 	return lp873x_ldo_val(dev, PMIC_OP_SET, &uV);
 }
 
-static bool ldo_get_enable(struct udevice *dev)
+static int ldo_get_enable(struct udevice *dev)
 {
 	bool enable = false;
 	int ret;
@@ -310,7 +310,7 @@  static int buck_set_value(struct udevice *dev, int uV)
 	return lp873x_buck_val(dev, PMIC_OP_SET, &uV);
 }
 
-static bool buck_get_enable(struct udevice *dev)
+static int buck_get_enable(struct udevice *dev)
 {
 	bool enable = false;
 	int ret;