On Thu, Aug 8, 2013 at 12:59 AM, Milo Kim <woogyom....@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....@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
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
