power: qcom_smbb: test the correct variable
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 24 Sep 2015 18:41:37 +0000 (21:41 +0300)
committerSebastian Reichel <sre@kernel.org>
Thu, 24 Sep 2015 20:35:24 +0000 (22:35 +0200)
"state" is a bool so it's never less than zero.  The intent was to test
"ret" instead.

Fixes: 56d7df8716b2 ('power: Add Qualcomm SMBB driver')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
drivers/power/qcom_smbb.c

index 0dabfe8..5eb1e9e 100644 (file)
@@ -351,7 +351,7 @@ static void smbb_set_line_flag(struct smbb_charger *chg, int irq, int flag)
        int ret;
 
        ret = irq_get_irqchip_state(irq, IRQCHIP_STATE_LINE_LEVEL, &state);
-       if (state < 0) {
+       if (ret < 0) {
                dev_err(chg->dev, "failed to read irq line\n");
                return;
        }