charger: bq25713: Corrects unsigned int comparison

rriveramcrus · kartben · commit 68f300dc3148 · 2025-05-28T11:10:43.000+02:00
Removes an unnecessary comparison of an unsigned int against the value
zero.

Signed-off-by: Ricardo Rivera-Matos &lt;ricardo.rivera-matos@cirrus.com&gt;
diff --git a/drivers/charger/charger_bq25713.c b/drivers/charger/charger_bq25713.c
@@ -165,8 +165,7 @@ static int bq25713_set_minimum_system_voltage(const struct device *dev, uint32_t
 
 static int bq25713_set_constant_charge_current(const struct device *dev, uint32_t current_ua)
 {
-	if (!IN_RANGE(current_ua, BQ25713_REG_CC_CHARGE_CURRENT_MIN_UA,
-		      BQ25713_REG_CC_CHARGE_CURRENT_MAX_UA)) {
+	if (current_ua > BQ25713_REG_CC_CHARGE_CURRENT_MAX_UA) {
 		LOG_WRN("charging current out of range: %umA, "
 			"clamping to the nearest limit",
 			current_ua / BQ25713_FACTOR_U_TO_M);

Original file line number	Diff line number	Diff line change
`@@ -165,8 +165,7 @@ static int bq25713_set_minimum_system_voltage(const struct device *dev, uint32_t`
`165`	`165`
`166`	`166`	`static int bq25713_set_constant_charge_current(const struct device *dev, uint32_t current_ua)`
`167`	`167`	`{`
`168`		`- if (!IN_RANGE(current_ua, BQ25713_REG_CC_CHARGE_CURRENT_MIN_UA,`
`169`		`- BQ25713_REG_CC_CHARGE_CURRENT_MAX_UA)) {`
	`168`	`+ if (current_ua > BQ25713_REG_CC_CHARGE_CURRENT_MAX_UA) {`
`170`	`169`	`LOG_WRN("charging current out of range: %umA, "`
`171`	`170`	`"clamping to the nearest limit",`
`172`	`171`	`current_ua / BQ25713_FACTOR_U_TO_M);`