From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v6 0/7] Add initial support for DA9150 Charger & Fuel-Gauge IC Date: Thu, 22 Jan 2015 07:58:11 +0000 Message-ID: <20150122075811.GO22024@x1> References: <20150121165250.GG13715@earth.universe> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-pm-owner@vger.kernel.org To: Jonathan Cameron Cc: Sebastian Reichel , Adam Thomson , Jonathan Cameron , Samuel Ortiz , Hartmut Knaack , linux-iio@vger.kernel.org, Dmitry Eremin-Solenikov , David Woodhouse , linux-pm@vger.kernel.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Grant Likely , devicetree@vger.kernel.org, Andrew Morton , Joe Perches , linux-kernel@vger.kernel.org, support.opensource@diasemi.com List-Id: devicetree@vger.kernel.org On Wed, 21 Jan 2015, Jonathan Cameron wrote: >=20 >=20 > On 21 January 2015 16:52:50 GMT+00:00, Sebastian Reichel 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 Integrat= ed > >Charger & > >> Fuel-Gauge IC. The device also provides GPIO and GPADC functionali= ty. > >>=20 > >> 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 suppor= t > >list. > >>=20 > >> To keep patch submission from being too large, support for GPIO an= d > >Fuel-Gauge > >> will come after initial support patches are accepted. > >>=20 > >> This patch set is baselined against the v3.19-rc5 kernel version. > >>=20 > >> 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 cleane= r. > > > >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. --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog