regulator: max597x: Fix error return code in max597x_get_status
authorAxel Lin <axel.lin@ingics.com>
Mon, 20 Feb 2023 10:56:14 +0000 (18:56 +0800)
committerMark Brown <broonie@kernel.org>
Mon, 20 Feb 2023 14:32:21 +0000 (14:32 +0000)
REGULATOR_FAILED_RETRY should not be used in max597x_get_status error path.
Othewise, the regulator core will treat it as REGULATOR_STATUS_ON.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Link: https://lore.kernel.org/r/20230220105614.356187-1-axel.lin@ingics.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/max597x-regulator.c

index f0fb0f56e4207e2786360ec3b457ae7886155154..648e3641885a884b2da137d1e74efdd32a44b279 100644 (file)
@@ -193,7 +193,7 @@ static int max597x_get_status(struct regulator_dev *rdev)
 
        ret = regmap_read(rdev->regmap, MAX5970_REG_STATUS3, &val);
        if (ret)
-               return REGULATOR_FAILED_RETRY;
+               return ret;
 
        if (val & MAX5970_STATUS3_ALERT)
                return REGULATOR_STATUS_ERROR;