public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [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