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

[Lee Jones <lee.jones@linaro.org>]

On Wed, 21 Jan 2015, Jonathan Cameron wrote:

> 
> 
> On 21 January 2015 16:52:50 GMT+00:00, Sebastian Reichel <sre@kernel.org> 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.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog