Re: [PATCH v5 3/3] power: supply: mt6360_charger: add MT6360 charger support

From: Sebastian Reichel
Date: Fri Jul 16 2021 - 09:48:51 EST


Hi,

On Mon, May 31, 2021 at 11:39:50AM +0800, Gene Chen wrote:
> > > +static int mt6360_charger_remove(struct platform_device *pdev)
> > > +{
> > > + struct mt6360_chg_info *mci = platform_get_drvdata(pdev);
> > > +
> > > + cancel_work_sync(&mci->chrdet_work);
> > > +
> > > + return 0;
> > > +}
> >
> > Just as a note, devm_work_autocancel() should hopefully be included at
> > next cycle. Then you might be able to drop the .remove().
>
> ACK, I will replace it, thanks.

> > FWIW:
> > Reviewed-by: Matti Vaittinen <matti.vaittinen@xxxxxxxxxxxxxxxxx>

Did I miss a newer revision?

-- Sebastian

Attachment: signature.asc
Description: PGP signature