RE: [PATCH RESEND] max17042: missing assignment

From: Pallala, Ramakrishna
Date: Thu Oct 25 2012 - 10:28:00 EST



Thanks Alan.

> The second error check is unreachable because the lock function isn't assigned
> to ret
>
> Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx>
Acked-by: Ramakrishna Pallala <ramakrishna.pallala@xxxxxxxxx>

> ---
>
> drivers/power/max17042_battery.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/power/max17042_battery.c
> b/drivers/power/max17042_battery.c
> index 74abc6c..73b432d 100644
> --- a/drivers/power/max17042_battery.c
> +++ b/drivers/power/max17042_battery.c
> @@ -572,7 +572,7 @@ static int max17042_init_chip(struct max17042_chip
> *chip)
> __func__);
> return -EIO;
> }
> - max17042_verify_model_lock(chip);
> + ret = max17042_verify_model_lock(chip);
> if (ret) {
> dev_err(&chip->client->dev, "%s lock verify failed\n",
> __func__);

¢éì®&Þ~º&¶¬–+-±éÝ¥Šw®žË±Êâmébžìdz¹Þ)í…æèw*jg¬±¨¶‰šŽŠÝj/êäz¹ÞŠà2ŠÞ¨è­Ú&¢)ß«a¶Úþø®G«éh®æj:+v‰¨Šwè†Ù>Wš±êÞiÛaxPjØm¶Ÿÿà -»+ƒùdš_