aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/power/max17042_battery.c
diff options
context:
space:
mode:
authorAlan Cox <alan@lxorguk.ukuu.org.uk>2012-11-18 17:59:47 -0500
committerAnton Vorontsov <anton.vorontsov@linaro.org>2012-11-18 17:59:47 -0500
commita879f19fe7decb9a53d13a63edd2eced8c2895a2 (patch)
treecdd8722a6cd84aa7d27d79aded0da18552c97865 /drivers/power/max17042_battery.c
parent8511748ba1ab83a7d89d802a50335115358a8cfc (diff)
max17042_battery: Fix missing verify_model_lock() return value check
The second error check is unreachable because the lock function isn't assigned to ret. Signed-off-by: Alan Cox <alan@linux.intel.com> Acked-by: Ramakrishna Pallala <ramakrishna.pallala@intel.com> Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
Diffstat (limited to 'drivers/power/max17042_battery.c')
-rw-r--r--drivers/power/max17042_battery.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/power/max17042_battery.c b/drivers/power/max17042_battery.c
index 74abc6c755b4..66b2c7b50914 100644
--- a/drivers/power/max17042_battery.c
+++ b/drivers/power/max17042_battery.c
@@ -572,7 +572,8 @@ static int max17042_init_chip(struct max17042_chip *chip)
572 __func__); 572 __func__);
573 return -EIO; 573 return -EIO;
574 } 574 }
575 max17042_verify_model_lock(chip); 575
576 ret = max17042_verify_model_lock(chip);
576 if (ret) { 577 if (ret) {
577 dev_err(&chip->client->dev, "%s lock verify failed\n", 578 dev_err(&chip->client->dev, "%s lock verify failed\n",
578 __func__); 579 __func__);