Messages in this thread | | | Date | Fri, 1 May 2020 16:58:40 +0200 | From | Sebastian Reichel <> | Subject | Re: [PATCH] power: charger-manager: clarify num_properties starting value |
| |
Hi,
On Fri, May 01, 2020 at 04:30:43PM +0200, Michał Mirosław wrote: > Initialize num_properties with length of the copied array instead > of relying on previously memcpy'd value. This makes it clear how > the array and the counter are related. > > Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl> > ---
Thanks, queued.
-- Sebastian
> drivers/power/supply/charger-manager.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/power/supply/charger-manager.c b/drivers/power/supply/charger-manager.c > index 415a9efa6816..2ef53dc1f2fb 100644 > --- a/drivers/power/supply/charger-manager.c > +++ b/drivers/power/supply/charger-manager.c > @@ -1729,7 +1729,7 @@ static int charger_manager_probe(struct platform_device *pdev) > memcpy(properties, default_charger_props, > sizeof(enum power_supply_property) * > ARRAY_SIZE(default_charger_props)); > - num_properties = psy_default.num_properties; > + num_properties = ARRAY_SIZE(default_charger_props); > > /* Find which optional psy-properties are available */ > fuel_gauge = power_supply_get_by_name(desc->psy_fuel_gauge); > -- > 2.20.1 > [unhandled content-type:application/pgp-signature] | |