RE: [PATCH v3 00/39] leds: cleanup LP5521/5523 driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Kim, Milo
> Sent: Sunday, January 06, 2013 6:53 PM
> To: Bryan Wu
> Cc: linux-leds@xxxxxxxxxxxxxxx
> Subject: [PATCH v3 00/39] leds: cleanup LP5521/5523 driver
> 
> LP5521, LP5523 and L55231 have common features as below.
> 
>  Register access via the I2C.
>  Device initialization/de-initialization.
>  Create LED class devices for multiple output channels.
>  Device attributes for user-space interface.
> 
> Therefore, this patch-set removes duplicate code for LP5521/5523 driver
> and
> makes coherent driver structure, named lp55xx.
> 
> All patches have been tested in LP5521, LP5523 and LP55231 evaluation
> boards.
> 
> You can also pull from the location below.
> This branch is based on latest 'for-next' branch of linux-leds.
> 
>  git://github.com/milokim/lp55xx.git lp55xx-for-next
> 
> Milo(Woogyom) Kim (39):
> 
> Cleanups before adding new driver architecture
>   leds-lp55xx: cleanup init_device() in lp5521/5523
>   leds-lp55xx: cleanup deinit_device() in lp5521/5523
>   leds-lp55xx: cleanup init leds in lp5521/5523
>   leds-lp55xx: cleanup deinit leds in lp5521/5523
>   leds-lp55xx: add device reset function in lp5521/5523
>   leds-lp55xx: do chip specific configuration on device init
> 
> Replace lp5521/5523 with new lp55xx driver architecture
>   leds-lp55xx: add new common driver for lp5521/5523
>   leds-lp55xx: replace name of data structure
>   leds-lp55xx: use common lp55xx data structure in _probe()
>   leds-lp5521: cleanup lp5521_configure()
>   leds-lp5523: cleanup lp5523_configure()
>   leds-lp55xx: use lp55xx common init function - plaform data
>   leds-lp55xx: use lp55xx common init function - reset
>   leds-lp55xx: use lp55xx common init function - detect
>   leds-lp55xx: use lp55xx common init function - post init
>   leds-lp55xx: cleanup init function
>   leds-lp55xx: use lp55xx common deinit function
>   leds-lp55xx: use lp55xx common led registration function
>   leds-lp55xx: use lp55xx_init_led() common function
>   leds-lp55xx: use lp55xx_set_brightness()
>   leds-lp55xx: provide common LED current setting
>   leds-lp55xx: use lp55xx_unregister_leds()
>   leds-lp55xx: fix error condition in lp55xx_register_leds()
> 
> Adding new firmware interface for LED engines
>   leds-lp55xx: add new lp55xx_register_sysfs() for FW interface
>   leds-lp55xx: support firmware interface
>   leds-lp5521: use generic firmware interface
>   leds-lp5523: use generic firmware interface
> 
> Cleanups lp5521/5523 device attribtues
>   leds-lp55xx: support device specific attibutes
>   leds-lp55xx: use common dev attr driver function
>   leds-lp55xx: code refactoring on selftest function
>   leds-lp55xx: add unregister dev attr function
> 
> Others
>   leds-lp55xx: clean up _remove()
>   leds-lp55xx: clean up unused data and functions
>   leds-lp55xx: clean up definitions
>   leds-lp55xx: cleanup headers
>   leds-lp5521/5523: use new lp55xx common header
>   leds-lp5521/5523: add author and copyright description
>   leds-lp55xx: fix problem on removing LED attributes
>   Documentation: leds: update LP55xx family devices
> 
>  Documentation/leds/00-INDEX                  |    2 +
>  Documentation/leds/leds-lp5521.txt           |   63 +-
>  Documentation/leds/leds-lp5523.txt           |   27 +-
>  Documentation/leds/leds-lp55xx.txt           |  111 +++
>  arch/arm/mach-omap2/board-rx51-peripherals.c |    8 +-
>  arch/arm/mach-ux500/board-mop500.c           |   14 +-
>  drivers/leds/Kconfig                         |   15 +-
>  drivers/leds/Makefile                        |    1 +
>  drivers/leds/leds-lp5521.c                   |  944 +++++++-----------
> -----
>  drivers/leds/leds-lp5523.c                   | 1050 +++++++-----------
> --------
>  drivers/leds/leds-lp55xx-common.c            |  519 +++++++++++++
>  drivers/leds/leds-lp55xx-common.h            |  134 ++++
>  include/linux/leds-lp5521.h                  |   73 --
>  include/linux/leds-lp5523.h                  |   49 --
>  include/linux/platform_data/leds-lp55xx.h    |   87 +++
>  15 files changed, 1410 insertions(+), 1687 deletions(-)
>  create mode 100644 Documentation/leds/leds-lp55xx.txt
>  create mode 100644 drivers/leds/leds-lp55xx-common.c
>  create mode 100644 drivers/leds/leds-lp55xx-common.h
>  delete mode 100644 include/linux/leds-lp5521.h
>  delete mode 100644 include/linux/leds-lp5523.h
>  create mode 100644 include/linux/platform_data/leds-lp55xx.h

I need to mention what's the difference between v2 and v3.

Patch v3: 39 patches
 Do more detailed patches for patch-v2.
 Rebased on latest 'for-next' of linux-leds.
 (git://github.com/milokim/lp55xx.git lp55xx-for-next)

Patch v2: 33 patches
 Publish git tree based on 'for-next' of linux-leds.
 (git://github.com/milokim/lp55xx.git lp55xx)

Patch v1.
 Initial patch

Best Regards,
Milo

--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux