Re: [PATCH v2] power: supply: bq25890: Add support for setting IINLIM

Sebastian Reichel
2022-09-11T12:44:01Z

[-- Attachment #1: Type: text/plain, Size: 1300 bytes --]

Hi,

On Tue, Aug 23, 2022 at 05:59:33PM +0200, Marek Vasut wrote:
> On 8/1/22 09:15, Hans de Goede wrote:
> > On 8/1/22 04:57, Marek Vasut wrote:
> > > Let user set input current limit via sysfs. This is useful in case there
> > > are multiple chargers connected to the device, each of which with its own
> > > arbitrary maximum current which it can provide, some of which may provide
> > > more than the default 500mA. In that case, userspace can listen for plug
> > > events generated by each charger and adjust the current limit accordingly,
> > > e.g. to permit battery to charge faster.
> > >
> > > Note that the IINLIM is reset every time the bq25890 is disconnected from
> > > a charger, so the userspace must adjust the limit repeatly on every plug
> > > event.
> > >
> > > Signed-off-by: Marek Vasut <marex@denx.de>
> >
> > Thanks, patch looks good to me:
> >
> > Reviewed-by: Hans de Goede <hdegoede@redhat.com>
>
> Can this be applied now ?

Thanks, queued to power-supply's for-next branch.

Please make sure you are sending patches based on recent for-next
branch next time. Your patch was from before eab25b4f93aa ("power:
supply: bq25890: On the bq25892 set the IINLIM based on external
charger detection") which was added in 5.18 :(

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]