Date: Wed, 27 Apr 2016 16:29:37 +0100
From: Lee Jones <lee.jones@linaro.org>
To: Thierry Reding <treding@nvidia.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>, linus.walleij@linaro.org,
        robh+dt@kernel.org, mark.rutland@arm.com, gnurou@gmail.com,
        linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org,
        devicetree@vger.kernel.org, ijc+devicetree@hellion.org.uk,
        swarren@nvidia.com, Mallikarjun Kasoju <mkasoju@nvidia.com>
Subject: Re: [PATCH V10 2/6] mfd: max77620: add core driver for
 MAX77620/MAX20024
Message-ID: <20160427152937.GE4892@dell>
References: <1459348188-11726-1-git-send-email-ldewangan@nvidia.com>
 <1459348188-11726-3-git-send-email-ldewangan@nvidia.com>
 <20160427150559.GD4176@ulmo.ba.sec>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20160427150559.GD4176@ulmo.ba.sec>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2828
Lines: 77

On Wed, 27 Apr 2016, Thierry Reding wrote:

> On Wed, Mar 30, 2016 at 07:59:44PM +0530, Laxman Dewangan wrote:
> > MAX77620/MAX20024 are Power Management IC from the MAXIM.
> > It supports RTC, multiple GPIOs, multiple DCDC and LDOs,
> > watchdog, clock etc.
> > 
> > Add MFD drier to provides common support for accessing the
> > device; additional drivers is developed on respected subsystem
> > in order to use the functionality of the device.
> > 
> > Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
> > Signed-off-by: Mallikarjun Kasoju <mkasoju@nvidia.com>
> > Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> > 
> > ---
> > Changes from V1:
> > - Code cleanups per review from V1.
> > - Move register acccess APIs from header to c file.
> > - Remove some of non required variable, remove duplication in error message
> >  and simplify some of function implementation.
> > - Register RTC driver such that it can get the regmap handle form parent device
> > 
> > Changes from V2:
> > - Run coccicheck and checkpatch in strict mode for the alignment.
> > - Drop RTC driver and its i2c client registration.
> > 
> > Changes from V3:
> > - Change all sys initcall to module driver.
> > - change the max77620_read argument to unisgned int from u8.
> > 
> > Changes from V4:
> > - Take care of fps nodes.
> > - Drop the battery charger and low battery binding and related code as
> >   it need to go on power driver.
> > 
> > Changes from V5:
> > -None
> > 
> > Changes from V6:
> > - Taken care of Lee's comment like used defines for irqs, remove max77620
> >   register accesss abstractions, remove DTof module and use ID table only,
> >   reduce the copyright lines.
> > - Drop configuration for hard power off time chnage as it will be in
> >   power driver.
> > - Use direct regmap from all drivers instead of using abstractions.
> > - This depends on patch
> > 
> > Change from V7:
> > - Use MFD defines for making mfd cells.
> > - Use new property name.
> > 
> > Changes from V8:
> > - Remove the usage of MFD defines. Remove mutex_config as not needed.
> > 
> > Changes from V9:
> > - Use the devm_regmap_add_irq_chip() for irq registration.
> > 
> >  drivers/mfd/Kconfig          |  15 ++
> >  drivers/mfd/Makefile         |   1 +
> >  drivers/mfd/max77620.c       | 544 +++++++++++++++++++++++++++++++++++++++++++
> >  include/linux/mfd/max77620.h | 337 +++++++++++++++++++++++++++
> >  4 files changed, 897 insertions(+)
> >  create mode 100644 drivers/mfd/max77620.c
> >  create mode 100644 include/linux/mfd/max77620.h
> 
> Lee,
> 
> Were you going to pick this up along with the corresponding DT bindings?

I will, once I'm happy with the driver.

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