RE: [PATCH 0/8] lp8727_charger: cleanup code

2012-08-30 Thread Kim, Milo
> -Original Message- > From: Anton Vorontsov [mailto:anton.voront...@linaro.org] > Sent: Thursday, August 30, 2012 9:15 PM > To: Kim, Milo > Cc: linux-kernel@vger.kernel.org; David Woodhouse > Subject: Re: [PATCH 0/8] lp8727_charger: cleanup code > > On Thu, Au

Re: [PATCH 0/8] lp8727_charger: cleanup code

2012-08-30 Thread Anton Vorontsov
On Thu, Aug 30, 2012 at 11:37:16AM +, Kim, Milo wrote: > LP8727 driver should be patched for several reasons. > > (a) Need to clean up _probe()/_remove() > (b) Not secure code when the platform data is NULL > (c) Interrupt handling > Two threads are running for handling one IRQ. > One

[PATCH 0/8] lp8727_charger: cleanup code

2012-08-30 Thread Kim, Milo
LP8727 driver should be patched for several reasons. (a) Need to clean up _probe()/_remove() (b) Not secure code when the platform data is NULL (c) Interrupt handling Two threads are running for handling one IRQ. One is for the IRQ pin, the other is used for delayed processing. This

[PATCH 0/8] lp8727_charger: cleanup code

2012-08-30 Thread Kim, Milo
LP8727 driver should be patched for several reasons. (a) Need to clean up _probe()/_remove() (b) Not secure code when the platform data is NULL (c) Interrupt handling Two threads are running for handling one IRQ. One is for the IRQ pin, the other is used for delayed processing. This

Re: [PATCH 0/8] lp8727_charger: cleanup code

2012-08-30 Thread Anton Vorontsov
On Thu, Aug 30, 2012 at 11:37:16AM +, Kim, Milo wrote: LP8727 driver should be patched for several reasons. (a) Need to clean up _probe()/_remove() (b) Not secure code when the platform data is NULL (c) Interrupt handling Two threads are running for handling one IRQ. One is for

RE: [PATCH 0/8] lp8727_charger: cleanup code

2012-08-30 Thread Kim, Milo
-Original Message- From: Anton Vorontsov [mailto:anton.voront...@linaro.org] Sent: Thursday, August 30, 2012 9:15 PM To: Kim, Milo Cc: linux-kernel@vger.kernel.org; David Woodhouse Subject: Re: [PATCH 0/8] lp8727_charger: cleanup code On Thu, Aug 30, 2012 at 11:37:16AM +, Kim