Re: [PATCH v2 2/9] mfd: max77693: Store I2C device type as enum and add default unknown

From: Lee Jones
Date: Wed May 27 2015 - 08:24:31 EST


On Wed, 27 May 2015, Lee Jones wrote:

> On Wed, 27 May 2015, Krzysztof KozÅowski wrote:
>
> > 2015-05-27 19:23 GMT+09:00 Lee Jones <lee.jones@xxxxxxxxxx>:
> > > On Sun, 24 May 2015, Krzysztof Kozlowski wrote:
> > >
> > >> Store the device type (obtained from i2c_device_id) as an enum and add a
> > >> default type of unknown to distinguish from case when this is not set
> > >> at all.
> > >>
> > >> Signed-off-by: Krzysztof Kozlowski <k.kozlowski.k@xxxxxxxxx>
> > >> ---
> > >> include/linux/mfd/max77693-private.h | 11 ++++++-----
> > >> 1 file changed, 6 insertions(+), 5 deletions(-)
> > >
> > > Applied, thanks.
> >
> > Thanks.
> >
> > I was thinking about merging everything through one tree because of
> > the dependency (successive patches depend on previous) but I don't
> > mind choosing different strategy. However if you picked this one maybe
> > you can also apply the patch 1/9:
> > mfd/extcon: max77693: Remove unused extern declarations and max77693_dev members
> >
> > It already got ack from Chanwoo.
> >
> > Other patches depend on these so either everything will go through
> > your tree or you will prepare an immutable branch for e.g. Mark (as
> > everything is around regulator drivers).
> >
> > Anyway I need some input from Mark about the patches 4, 5 and 9. Rest
> > of them already have his ack.
>
> I'm happy to do that. Just let me know when you have all the Acks you
> need and I'll apply them all at once.

In fact, it's probably better to withdraw this patch too, then apply
it with all the others.

Patch unapplied (but you can add my Ack).

--
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/