Message ID | 20201001214034.5925-1-r-rivera-matos@ti.com |
---|---|
Headers | show |
Series | Introduce the BQ256XX family of chargers | expand |
Hi Ricardo, On Thu, Oct 01, 2020 at 04:40:34PM -0500, Ricardo Rivera-Matos wrote: > The BQ256XX family of devices are highly integrated buck chargers > for single cell batteries. > > Signed-off-by: Ricardo Rivera-Matos <r-rivera-matos@ti.com> > > v5 - adds power_supply_put_battery_info() and devm_add_action_or_reset() calls > --- changes look good, but you are still missing usb_unregister_notifier() calls on drival removal. -- Sebastian
Sebastian, On 10/3/20 5:54 AM, Sebastian Reichel wrote: > Hi Ricardo, > > On Thu, Oct 01, 2020 at 04:40:34PM -0500, Ricardo Rivera-Matos wrote: >> The BQ256XX family of devices are highly integrated buck chargers >> for single cell batteries. >> >> Signed-off-by: Ricardo Rivera-Matos <r-rivera-matos@ti.com> >> >> v5 - adds power_supply_put_battery_info() and devm_add_action_or_reset() calls >> --- > changes look good, but you are still missing > usb_unregister_notifier() calls on drival removal. ACK, I will add a driver removal function and make the appropriate usb_unregister_notifier() calls. > > -- Sebastian Ricardo