* [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean
@ 2012-07-27 5:01 Chanwoo Choi
2012-08-23 3:10 ` Anton Vorontsov
0 siblings, 1 reply; 4+ messages in thread
From: Chanwoo Choi @ 2012-07-27 5:01 UTC (permalink / raw)
To: anton.vorontsov
Cc: cbouatmailru, jenny.tc, ramakrishna.pallala, myungjoo.ham,
kyungmin.park, linux-kernel, Chanwoo Choi
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(-)
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean
2012-07-27 5:01 [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean Chanwoo Choi
@ 2012-08-23 3:10 ` Anton Vorontsov
2012-08-23 4:27 ` Chanwoo Choi
0 siblings, 1 reply; 4+ messages in thread
From: Anton Vorontsov @ 2012-08-23 3:10 UTC (permalink / raw)
To: Chanwoo Choi
Cc: jenny.tc, ramakrishna.pallala, myungjoo.ham, kyungmin.park,
linux-kernel
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!
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean
2012-08-23 3:10 ` Anton Vorontsov
@ 2012-08-23 4:27 ` Chanwoo Choi
2012-08-23 4:33 ` Anton Vorontsov
0 siblings, 1 reply; 4+ messages in thread
From: Chanwoo Choi @ 2012-08-23 4:27 UTC (permalink / raw)
To: Anton Vorontsov
Cc: jenny.tc, ramakrishna.pallala, myungjoo.ham, kyungmin.park,
linux-kernel
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
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean
2012-08-23 4:27 ` Chanwoo Choi
@ 2012-08-23 4:33 ` Anton Vorontsov
0 siblings, 0 replies; 4+ messages in thread
From: Anton Vorontsov @ 2012-08-23 4:33 UTC (permalink / raw)
To: Chanwoo Choi
Cc: jenny.tc, ramakrishna.pallala, myungjoo.ham, kyungmin.park,
linux-kernel
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.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-08-23 4:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-27 5:01 [PATCH 0/2] charger-manager: Stop charging when charge cable is detached and code clean Chanwoo Choi
2012-08-23 3:10 ` Anton Vorontsov
2012-08-23 4:27 ` Chanwoo Choi
2012-08-23 4:33 ` Anton Vorontsov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox