We checked that "pdata->chg_params" is non-NULL earlier in this function
so when we add "i" to it, it's still non-NULL.

Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>

diff --git a/drivers/power/supply/lp8788-charger.c 
b/drivers/power/supply/lp8788-charger.c
index 7321b72..509e2b3 100644
--- a/drivers/power/supply/lp8788-charger.c
+++ b/drivers/power/supply/lp8788-charger.c
@@ -384,9 +384,6 @@ static int lp8788_update_charger_params(struct 
platform_device *pdev,
        for (i = 0; i < pdata->num_chg_params; i++) {
                param = pdata->chg_params + i;
 
-               if (!param)
-                       continue;
-
                if (lp8788_is_valid_charger_register(param->addr)) {
                        ret = lp8788_write_byte(lp, param->addr, param->val);
                        if (ret)

Reply via email to