lkml.org 
[lkml]   [2013]   [Aug]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH 01/10] leds: lp55xx: add common data structure for program
On Thu, Aug 8, 2013 at 12:59 AM, Milo Kim <woogyom.kim@gmail.com> wrote:
> LP55xx family devices have internal three program engines which are used for
> loading LED patterns.
> To maintain legacy device attributes, specific data structure is used, 'mode'
> and 'led_mux'.
> The mode is used for showing/storing current engine mode such like disabled,
> load and run.
> Then led_mux is used for showing/storing current output LED selection.
> This is only for LP5523/55231.
>

This patch looks good to me, but the commit message format is little
bit odd to me. I will fix that and merge into my tree.

Thanks,
-Bryan


> Signed-off-by: Milo Kim <milo.kim@ti.com>
> ---
> drivers/leds/leds-lp55xx-common.h | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/drivers/leds/leds-lp55xx-common.h b/drivers/leds/leds-lp55xx-common.h
> index dbbf86d..04c1d4f 100644
> --- a/drivers/leds/leds-lp55xx-common.h
> +++ b/drivers/leds/leds-lp55xx-common.h
> @@ -20,6 +20,13 @@ enum lp55xx_engine_index {
> LP55XX_ENGINE_1,
> LP55XX_ENGINE_2,
> LP55XX_ENGINE_3,
> + LP55XX_ENGINE_MAX = LP55XX_ENGINE_3,
> +};
> +
> +enum lp55xx_engine_mode {
> + LP55XX_ENGINE_DISABLED,
> + LP55XX_ENGINE_LOAD,
> + LP55XX_ENGINE_RUN,
> };
>
> struct lp55xx_led;
> @@ -72,6 +79,16 @@ struct lp55xx_device_config {
> };
>
> /*
> + * struct lp55xx_engine
> + * @mode : Engine mode
> + * @led_mux : Mux bits for LED selection. Only used in LP5523
> + */
> +struct lp55xx_engine {
> + enum lp55xx_engine_mode mode;
> + u16 led_mux;
> +};
> +
> +/*
> * struct lp55xx_chip
> * @cl : I2C communication for access registers
> * @pdata : Platform specific data
> @@ -79,6 +96,7 @@ struct lp55xx_device_config {
> * @num_leds : Number of registered LEDs
> * @cfg : Device specific configuration data
> * @engine_idx : Selected engine number
> + * @engines : Engine structure for the device attribute R/W interface
> * @fw : Firmware data for running a LED pattern
> */
> struct lp55xx_chip {
> @@ -89,6 +107,7 @@ struct lp55xx_chip {
> int num_leds;
> struct lp55xx_device_config *cfg;
> enum lp55xx_engine_index engine_idx;
> + struct lp55xx_engine engines[LP55XX_ENGINE_MAX];
> const struct firmware *fw;
> };
>
> --
> 1.7.9.5
>


\
 
 \ /
  Last update: 2013-08-13 21:21    [W:0.373 / U:0.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site