linux-kernel - Re: [PATCH 01/10] leds: lp55xx: add common data structure for program

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 13 Aug 2013 11:56:14 -0700
From:	Bryan Wu <cooloney@...il.com>
To:	Milo Kim <woogyom.kim@...il.com>
Cc:	Pali Rohár <pali.rohar@...il.com>,
	Linux LED Subsystem <linux-leds@...r.kernel.org>,
	lkml <linux-kernel@...r.kernel.org>, Milo Kim <milo.kim@...com>
Subject: Re: [PATCH 01/10] leds: lp55xx: add common data structure for program

On Thu, Aug 8, 2013 at 12:59 AM, Milo Kim <woogyom.kim@...il.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@...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 majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/