Re: [PATCH 4/4] leds-lp5523: minor code style fixes

[Bryan Wu]

On Tue, Aug 21, 2012 at 5:04 PM, Kim, Milo <Milo.Kim@xxxxxx> wrote:
> (a) use LP5523_ENABLE rather than magic number 0x40
> (b) use min_t() in lp5523_mux_parse()
> (c) skip while loop and just return if invalid command
>

Thanks for this cleanup. I will merge it later.

-Bryan

> Signed-off-by: Milo(Woogyom) Kim <milo.kim@xxxxxx>
> ---
>  drivers/leds/leds-lp5523.c |   15 +++++++--------
>  1 files changed, 7 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c
> index f8231f1..64ba8b2 100644
> --- a/drivers/leds/leds-lp5523.c
> +++ b/drivers/leds/leds-lp5523.c
> @@ -177,7 +177,7 @@ static int lp5523_detect(struct i2c_client *client)
>         int ret;
>         u8 buf;
>
> -       ret = lp5523_write(client, LP5523_REG_ENABLE, 0x40);
> +       ret = lp5523_write(client, LP5523_REG_ENABLE, LP5523_ENABLE);
>         if (ret)
>                 return ret;
>         ret = lp5523_read(client, LP5523_REG_ENABLE, &buf);
> @@ -338,7 +338,8 @@ static int lp5523_mux_parse(const char *buf, u16 *mux, size_t len)
>  {
>         int i;
>         u16 tmp_mux = 0;
> -       len = len < LP5523_LEDS ? len : LP5523_LEDS;
> +
> +       len = min_t(int, len, LP5523_LEDS);
>         for (i = 0; i < len; i++) {
>                 switch (buf[i]) {
>                 case '1':
> @@ -546,6 +547,9 @@ static int lp5523_do_store_load(struct lp5523_engine *engine,
>         unsigned cmd;
>         u8 pattern[LP5523_PROGRAM_LENGTH] = {0};
>
> +       if (engine->mode != LP5523_CMD_LOAD)
> +               return -EINVAL;
> +
>         while ((offset < len - 1) && (i < LP5523_PROGRAM_LENGTH)) {
>                 /* separate sscanfs because length is working only for %s */
>                 ret = sscanf(buf + offset, "%2s%n ", c, &nrchars);
> @@ -563,12 +567,7 @@ static int lp5523_do_store_load(struct lp5523_engine *engine,
>                 goto fail;
>
>         mutex_lock(&chip->lock);
> -
> -       if (engine->mode == LP5523_CMD_LOAD)
> -               ret = lp5523_load_program(engine, pattern);
> -       else
> -               ret = -EINVAL;
> -
> +       ret = lp5523_load_program(engine, pattern);
>         mutex_unlock(&chip->lock);
>
>         if (ret) {
> --
> 1.7.2.5
>
>
> Best Regards,
> Milo
>
>



-- 
Bryan Wu <bryan.wu@xxxxxxxxxxxxx>
Kernel Developer    +86.186-168-78255 Mobile
Canonical Ltd.      www.canonical.com
Ubuntu - Linux for human beings | www.ubuntu.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/