Commit a9aece5d authored by Yuanjun Gong's avatar Yuanjun Gong Committed by Sebastian Reichel
Browse files

power: return the correct error code 



In POWER_SUPPLY_PROP_MODEL_NAME branch of max1720x_battery_get_property(),
program would return -ENODEV out of FIELD_GET error, but it's better also
considering the error code returned by regmap_read() in case it fails.

Signed-off-by: default avatarYuanjun Gong <ruc_gongyuanjun@163.com>
Link: https://lore.kernel.org/r/20250513123732.3041577-1-ruc_gongyuanjun@163.com


Signed-off-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
parent 22e4d29f

drivers/power/supply/max1720x_battery.c

+2 −0
Original line number Diff line number Diff line
@@ -426,6 +426,8 @@ static int max1720x_battery_get_property(struct power_supply *psy, 
		break;

	case POWER_SUPPLY_PROP_MODEL_NAME:

		ret = regmap_read(info->regmap, MAX172XX_DEV_NAME, &reg_val);

		if (ret)

			return ret;

		reg_val = FIELD_GET(MAX172XX_DEV_NAME_TYPE_MASK, reg_val);

		if (reg_val == MAX172XX_DEV_NAME_TYPE_MAX17201)

			val->strval = max17201_model;