RE: [PATCH v6 0/7] Add initial support for DA9150 Charger & Fuel-Gauge IC

From: Opensource [Adam Thomson]
Date: Mon Feb 16 2015 - 16:46:18 EST


On January 22, 2015 07:58, Lee Jones wrote:

> On Wed, 21 Jan 2015, Jonathan Cameron wrote:
>
> >
> >
> > On 21 January 2015 16:52:50 GMT+00:00, Sebastian Reichel <sre@xxxxxxxxxx>
> wrote:
> > >Hi,
> > >
> > >On Wed, Jan 21, 2015 at 03:46:25PM +0000, Adam Thomson wrote:
> > >> This patch set adds initial support for the Dialog DA9150 Integrated
> > >Charger &
> > >> Fuel-Gauge IC. The device also provides GPIO and GPADC functionality.
> > >>
> > >> In this patch set the following is provided:
> > >> - MFD Core support and DT bindings documentation.
> > >> - IIO GPADC support and DT bindings documentation.
> > >> - Power Supply Charger support and DT bindings documentation.
> > >> - Update to MAINTAINERS file to add DA9150 files to Dialog support
> > >list.
> > >>
> > >> To keep patch submission from being too large, support for GPIO and
> > >Fuel-Gauge
> > >> will come after initial support patches are accepted.
> > >>
> > >> This patch set is baselined against the v3.19-rc5 kernel version.
> > >>
> > >> Changes in v6:
> > >> - For GPADC driver, Use lower case extended names for IIO GPADC
> > >channels, and
> > >> remove extended names for GPIO related channels.
> > >> - For charger driver, revert to no devm IRQ request/free functions as
> > >ordering
> > >> is important, and probe failure scenario was not being correctly
> > >covered. A
> > >> helper function for free IRQs has been added to make code cleaner.
> > >
> > >Probably a devm_ variant of power_supply_register should be created,
> > >but for now this is ok for me.
> > >
> > >Jonathan, Lee: I assume you are ok with the whole patchset going
> > >through the power supply subsystem?
> > >
> > Fine by me if either power supply or MFD.
>
> I am, but there is a lot of MFD code here, so I'd like to take it in
> as well. I'd like a tagged pull-request from Sebastian.

Hi Sebastien,

Just a quick one to see what the status is on this patch set? Just interested
as to when it'll be pulled in.

Thanks
Adam