[PATCH 00/22] lp8727_charger: clean up code

[PATCH 00/22] lp8727_charger: clean up code

[Kim, Milo]

LP8727 driver should be fixed for several reasons.

(a) Need to clean up _probe() and _remove()
(b) Not secure code when the platform data is NULL
(c) Improved the interrupt handling
(d) Lots of definitions should be unique.
(e) Others...

Patch v2.
These patches are submitted more separately.

[PATCH 01/22] lp8727_charger: use devm_kzalloc()
[PATCH 02/22] lp8727_charger: cleanup _probe() and _remove()
[PATCH 03/22] lp8727_charger: fix buggy code of NULL pdata
[PATCH 04/22] lp8727_charger: add configurable debouce timer
[PATCH 05/22] lp8727_charger: remove unnecessary workqueue thread
[PATCH 06/22] lp8727_charger: clean up the interrupt handler
[PATCH 07/22] lp8727_charger: clear interrrupts at inital time
[PATCH 08/22] lp8727_charger: fix code for getting battery temp
[PATCH 09/22] lp8727_charger: use the definition rather than enum
[PATCH 10/22] lp8727_charger: clean up lp8727 definitions
[PATCH 11/22] lp8727_charger: use specific definition
[PATCH 12/22] lp8727_charger: clean up lp8727_is_charger_attached()
[PATCH 13/22] lp8727_charger: make lp8727_init_device() shorter
[PATCH 14/22] lp8727_charger: make lp8727_ctrl_switch() inline
[PATCH 15/22] lp8727_charger: make lp8727_charger_get_propery() simpler
[PATCH 16/22] lp8727_charger: return if the battery is discharging
[PATCH 17/22] lp8727_charger: clean up lp8727_charger_changed()
[PATCH 18/22] lp8727_charger: make cosmetic code on lp8727_delayed_func()
[PATCH 19/22] lp8727_charger: fix typo - chg_parm to chg_param
[PATCH 20/22] lp8727_charger: add description in the private data
[PATCH 21/22] lp8727_charger: fix checkpatch warning
[PATCH 22/22] lp8727_charger: make cosmetic code


Best Regards,
Milo

Re: [PATCH 00/22] lp8727_charger: clean up code

[Anton Vorontsov]

On Fri, Aug 31, 2012 at 09:22:39AM +0000, Kim, Milo wrote:
> LP8727 driver should be fixed for several reasons.
> 
> (a) Need to clean up _probe() and _remove()
> (b) Not secure code when the platform data is NULL
> (c) Improved the interrupt handling
> (d) Lots of definitions should be unique.
> (e) Others...
> 
> Patch v2.
> These patches are submitted more separately.

Nice! All 22 applied, thank you so much!

Anton.