2012-08-23 03:13:04

by Anton Vorontsov

[permalink] [raw]
Subject: Re: [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean

On Fri, Jul 27, 2012 at 02:01:25PM +0900, Chanwoo Choi wrote:
> This patchset fix bug related to stop charging when charger cable is
> detached and check return value of regulator_enable/disable() function
> to confirm correct operation of enabled or disabling regulator for
> charging. Second patch is code clean which remove unnecessary variable
> type.
>
> Chanwoo Choi (2):
> charger-manager: Disable regulator for charging when charger cable is
> detached
> charger-manager: Use replacement variable to check state of battery
>
> drivers/power/charger-manager.c | 21 ++++++++++++++++++---
> include/linux/power/charger-manager.h | 3 ---
> 2 files changed, 18 insertions(+), 6 deletions(-)

Both applied, much thanks!


2012-08-23 04:27:43

by Chanwoo Choi

[permalink] [raw]
Subject: Re: [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean

On 08/23/2012 12:10 PM, Anton Vorontsov wrote:

> On Fri, Jul 27, 2012 at 02:01:25PM +0900, Chanwoo Choi wrote:
>> This patchset fix bug related to stop charging when charger cable is
>> detached and check return value of regulator_enable/disable() function
>> to confirm correct operation of enabled or disabling regulator for
>> charging. Second patch is code clean which remove unnecessary variable
>> type.
>>
>> Chanwoo Choi (2):
>> charger-manager: Disable regulator for charging when charger cable is
>> detached
>> charger-manager: Use replacement variable to check state of battery
>>
>> drivers/power/charger-manager.c | 21 ++++++++++++++++++---
>> include/linux/power/charger-manager.h | 3 ---
>> 2 files changed, 18 insertions(+), 6 deletions(-)
>
> Both applied, much thanks!
>

Thanks for your reply and applied it.

I did send additional patchset which has 5 patch and also include below
patch list,
if you may confuse due to duplicate patch, so I inform to you about
duplicate patch.

Chanwoo Choi (2):
charger-manager: Disable regulator for charging when charger cable is
detached
charger-manager: Use replacement variable to check state of battery

You can check additional patchset for charger-manager:
https://lkml.org/lkml/2012/8/21/68

Thanks and Best Regards,
Chanwoo Choi

2012-08-23 04:35:47

by Anton Vorontsov

[permalink] [raw]
Subject: Re: [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean

On Thu, Aug 23, 2012 at 01:27:27PM +0900, Chanwoo Choi wrote:
[...]
> I did send additional patchset which has 5 patch and also include below
> patch list,
> if you may confuse due to duplicate patch, so I inform to you about
> duplicate patch.
>
> Chanwoo Choi (2):
> charger-manager: Disable regulator for charging when charger cable is
> detached
> charger-manager: Use replacement variable to check state of battery

Understood, thanks for the heads up!

The third patch of 0/5 patch set needs a thoughtful review,
I'll surely get to it, but not today.

Anton.