Re: [PATCH v5 0/9] mfd: max14577: Add support for MAX77836

From: Lee Jones
Date: Wed Apr 23 2014 - 09:04:30 EST


On Wed, 23 Apr 2014, Krzysztof Kozlowski wrote:

> On Åro, 2014-04-16 at 13:18 +0100, Lee Jones wrote:
> > > > > > > This is first part of fifth version of patchset adding support for
> > > > > > > MAX77836 device to the max14577 driver.
> > > > > > >
> > > > > > > I decided to split the patchset into smaller parts. I hope this would
> > > > > > > help in getting necessary review/acks from maintainers.
> > > > > > >
> > > > > > > The first part introduces changes in max14577 driver and adds support
> > > > > > > for MAX77836 in MFD, extcon and regulator drivers. Most of these
> > > > > > > patches already have required acks from maintainers.
> > > > > > > Full v4 patchset can be found here:
> > > > > > > http://thread.gmane.org/gmane.linux.kernel/1654267
> > > > > >
> > > > > > So what's the plan for this driver now? What's missing? Do you have
> > > > > > all the Acks you need? If so, I'm happy to apply this patch-set and
> > > > > > supply a pull-request to the other maintainers.
> > > > >
> > > > > Next patches (part 2):
> > > > > 1. charger: max14577: Add support for MAX77836 charger
> > > > > 2. power: max17040: Add ID for MAX77836 Fuel Gauge block
> > > > > 3. regulator/mfd: max14577: Export symbols for calculating charger
> > > > > current
> > > > > 4. charger: max14577: Configure battery-dependent settings from DTS
> > > > > 5. Documentation: mfd: max14577: Add device tree bindings document
> > > > > 6. regulator: max14577: Implement SUSPEND mode for MAX77836 LDO-s
> > > > >
> > > > > They are quite independent from each other so I think it would be the
> > > > > best if you could prepare an immutable branch with first part of
> > > > > patches.
> > > >
> > > > I'm only concerned with the patches in this set at the moment.
> > > >
> > > > Do all of the patches in _this_ set have all of the required Acks, or
> > > > are you still waiting on some maintainers?
> > >
> > > I have all necessary Acks for this patchset.
> > > One exception: patch 9/9 (regulator) was only reviewed by Mark Brown,
> > > not acked:
> > > https://lkml.org/lkml/2014/1/28/252
> >
> > That's okay. Still counts. :)
>
> If there are no more issues can you apply this patchset and prepare an
> immutable branch with them?

Yep, it's on my TODO list for today.

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/