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

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