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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxx>
> > > Acked-by: Jonghwa Lee <jonghwa3.lee@xxxxxxxxxxx>
> > > Acked-by: Pavel Machek <pavel@xxxxxx>
> > > Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>
> > > Reviewed-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx>
> > > Reviewed-by: Sebastian Reichel <sre@xxxxxxxxxx>
> > > ---
> > >  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
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux