Messages in this thread | | | Date | Sun, 11 Sep 2022 14:34:43 +0200 | From | Sebastian Reichel <> | Subject | Re: [PATCH] power: supply: ab8500: remove unused static local variable |
| |
Hi,
On Mon, Jul 18, 2022 at 10:27:43PM -0400, Tom Rix wrote: > cpp_check reports > [drivers/power/supply/ab8500_chargalg.c:493]: (style) Variable 'ab8500_chargalg_ex_ac_enable_toggle' is assigned a value that is never used. > > From inspection, this variable is never used. So remove it. > > Fixes: 6c50a08d9dd3 ("power: supply: ab8500: Drop external charger leftovers") > Signed-off-by: Tom Rix <trix@redhat.com> > ---
Thanks, queued into power-supply's fixes branch. Sorry for the delay.
-- Sebastian
> drivers/power/supply/ab8500_chargalg.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/power/supply/ab8500_chargalg.c b/drivers/power/supply/ab8500_chargalg.c > index ae4be553f424..05146d436a6a 100644 > --- a/drivers/power/supply/ab8500_chargalg.c > +++ b/drivers/power/supply/ab8500_chargalg.c > @@ -490,8 +490,6 @@ static int ab8500_chargalg_kick_watchdog(struct ab8500_chargalg *di) > static int ab8500_chargalg_ac_en(struct ab8500_chargalg *di, int enable, > int vset_uv, int iset_ua) > { > - static int ab8500_chargalg_ex_ac_enable_toggle; > - > if (!di->ac_chg || !di->ac_chg->ops.enable) > return -ENXIO; > > -- > 2.27.0 > [unhandled content-type:application/pgp-signature] | |