Re: [RESEND PATCH v6 09/22] mfd: ab8500: Use power_supply_*() API for accessing function attrs

[Lee Jones <lee.jones@linaro.org>]

On Thu, 12 Mar 2015, Krzysztof Kozlowski wrote:
> On czw, 2015-03-12 at 09:01 +0000, Lee Jones wrote:
> > On Thu, 12 Mar 2015, Krzysztof Kozlowski wrote:
> > 
> > > Replace direct calls to power supply function attributes with wrappers.
> > > Wrappers provide safe access in case of unregistering the power
> > > supply (e.g. by removing the driver). Replace:
> > >  - get_property -> power_supply_get_property
> > > 
> > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> > > Acked-by: Jonghwa Lee <jonghwa3.lee@samsung.com>
> > > Acked-by: Pavel Machek <pavel@ucw.cz>
> > > Acked-by: Lee Jones <lee.jones@linaro.org>
> > > Reviewed-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > > Reviewed-by: Sebastian Reichel <sre@kernel.org>
> > > ---
> > >  drivers/mfd/ab8500-sysctrl.c | 7 ++++---
> > >  1 file changed, 4 insertions(+), 3 deletions(-)
> > 
> > Has the infrastructure for these been merged yet?
> > 
> > I.e. Can I just take them?
> 
> Previous patches were not merged yet so you cannot take it. This whole
> patchset can be applied incrementally but cherry picking won't work.

So I only usually receive patches I've already Acked when a) the bulk
of the set has been merged and I can apply the remaining lingerers, or
b) the whole set is due to be applied through the MFD tree.

Receiving 2 patches of a 22 patch set, which I have already Acked and
that I can't pick is unwelcome noise.

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