From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ig0-f169.google.com ([209.85.213.169]:65340 "EHLO mail-ig0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751398AbbAVH6S (ORCPT ); Thu, 22 Jan 2015 02:58:18 -0500 Received: by mail-ig0-f169.google.com with SMTP id hl2so17427192igb.0 for ; Wed, 21 Jan 2015 23:58:18 -0800 (PST) Date: Thu, 22 Jan 2015 07:58:11 +0000 From: Lee Jones 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 Subject: Re: [PATCH v6 0/7] Add initial support for DA9150 Charger & Fuel-Gauge IC Message-ID: <20150122075811.GO22024@x1> References: <20150121165250.GG13715@earth.universe> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On Wed, 21 Jan 2015, Jonathan Cameron wrote: > > > 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 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