commit 16e56a75deb30fb18c43a94762a830fc94c71a16
author Nicolas Le Bayon <nicolas.le.bayon@st.com> Thu Sep 19 11:24:50 2019 +0200
committer Yann Gautier <yann.gautier@st.com> Wed Dec 22 14:04:32 2021 +0100
tree 60ba32dc0b5ebd086ba6e8b3c9e8c221aae628b8
parent 0c16e7d2fbb99fb546efd5303ec863742d99500e

refactor(stpmic1): set stpmic1_is_regulator_enabled() as boolean

Improve use and readability.

Change-Id: Ia99fc38287f36c9dd12bfe51352afa5da68c0e47
Signed-off-by: Nicolas Le Bayon <nicolas.le.bayon@st.com>

drivers/st/pmic/stpmic1.c

diff --git a/drivers/st/pmic/stpmic1.c b/drivers/st/pmic/stpmic1.c
index 0a35df3..8199b7a 100644
--- a/drivers/st/pmic/stpmic1.c
+++ b/drivers/st/pmic/stpmic1.c
@@ -606,7 +606,7 @@
 				       regul->enable_mask);
 }
 
-uint8_t stpmic1_is_regulator_enabled(const char *name)
+bool stpmic1_is_regulator_enabled(const char *name)
 {
 	uint8_t val;
 	const struct regul_struct *regul = get_regulator_data(name);
@@ -615,7 +615,7 @@
 		panic();
 	}
 
-	return (val & regul->enable_mask);
+	return (val & regul->enable_mask) == regul->enable_mask;
 }
 
 int stpmic1_regulator_voltage_set(const char *name, uint16_t millivolts)