Re: [patch] power: supply: lp8788: remove an unneeded NULL check
Hi Dan, On Fri, Oct 14, 2016 at 10:33:28AM +0300, Dan Carpenter wrote: > 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> > 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) Thanks, queued. -- Sebastian signature.asc Description: PGP signature
Re: [patch] power: supply: lp8788: remove an unneeded NULL check
Hi Dan, On Fri, Oct 14, 2016 at 10:33:28AM +0300, Dan Carpenter wrote: > 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 > > 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) Thanks, queued. -- Sebastian signature.asc Description: PGP signature
Re: [patch] power: supply: lp8788: remove an unneeded NULL check
On 10/14/2016 4:33 PM, Dan Carpenter wrote: 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 CarpenterAcked-by: Milo Kim Thanks for catching this. Best regards, Milo
Re: [patch] power: supply: lp8788: remove an unneeded NULL check
On 10/14/2016 4:33 PM, Dan Carpenter wrote: 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 Acked-by: Milo Kim Thanks for catching this. Best regards, Milo
[patch] power: supply: lp8788: remove an unneeded NULL check
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 Carpenterdiff --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)
[patch] power: supply: lp8788: remove an unneeded NULL check
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 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)