linux-kernel - Re: [PATCH v2 1/2] leds-lp5523: add channel name in the platform data

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] [day] [month] [year] [list]

Message-ID: <CAK5ve-+umR2-8P46bZaDzqQBKotLTNb-Jg8xYBAP_kjWCK04LA@mail.gmail.com>
Date:	Wed, 22 Aug 2012 22:54:16 +0800
From:	Bryan Wu <bryan.wu@...onical.com>
To:	"Kim, Milo" <Milo.Kim@...com>
Cc:	Richard Purdie <rpurdie@...ys.net>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-leds@...r.kernel.org" <linux-leds@...r.kernel.org>
Subject: Re: [PATCH v2 1/2] leds-lp5523: add channel name in the platform data

On Wed, Aug 22, 2012 at 3:32 PM, Kim, Milo <Milo.Kim@...com> wrote:
> The name of each led channel is configurable.
> If the name is NULL, just use the channel id for making the channel name
>

OK, merged to my for-next branch.

-Bryan

> Signed-off-by: Milo(Woogyom) Kim <milo.kim@...com>
> ---
>  Documentation/leds/leds-lp5523.txt |   21 ++++++++++++++++++---
>  drivers/leds/leds-lp5523.c         |   10 +++++++---
>  include/linux/leds-lp5523.h        |    1 +
>  3 files changed, 26 insertions(+), 6 deletions(-)
>
> diff --git a/Documentation/leds/leds-lp5523.txt b/Documentation/leds/leds-lp5523.txt
> index fad2feb..c2743f5 100644
> --- a/Documentation/leds/leds-lp5523.txt
> +++ b/Documentation/leds/leds-lp5523.txt
> @@ -10,8 +10,22 @@ Contact: Samu Onkalo (samu.p.onkalo-at-nokia.com)
>  Description
>  -----------
>  LP5523 can drive up to 9 channels. Leds can be controlled directly via
> -the led class control interface. Channels have generic names:
> -lp5523:channelx where x is 0...8
> +the led class control interface.
> +The name of each channel is configurable in the platform data - name and label.
> +There are three options to make the channel name.
> +
> +a) Define the 'name' in the platform data
> +To make specific channel name, then use 'name' platform data.
> +/sys/class/leds/R1               (name: 'R1')
> +/sys/class/leds/B1               (name: 'B1')
> +
> +b) Use the 'label' with no 'name' field
> +For one device name with channel number, then use 'label'.
> +/sys/class/leds/RGB:channelN     (label: 'RGB', N: 0 ~ 8)
> +
> +c) Default
> +If both fields are NULL, 'lp5523' is used by default.
> +/sys/class/leds/lp5523:channelN  (N: 0 ~ 8)
>
>  The chip provides 3 engines. Each engine can control channels without
>  interaction from the main CPU. Details of the micro engine code can be found
> @@ -46,12 +60,13 @@ Note - chan_nr can have values between 0 and 8.
>
>  static struct lp5523_led_config lp5523_led_config[] = {
>          {
> +               .name           = "D1",
>                  .chan_nr        = 0,
>                  .led_current    = 50,
>                 .max_current    = 130,
>          },
>  ...
> -        }, {
> +        {
>                  .chan_nr        = 8,
>                  .led_current    = 50,
>                 .max_current    = 130,
> diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c
> index fbc12ac..9fd9a92 100644
> --- a/drivers/leds/leds-lp5523.c
> +++ b/drivers/leds/leds-lp5523.c
> @@ -846,10 +846,14 @@ static int __devinit lp5523_init_led(struct lp5523_led *led, struct device *dev,
>                         return -EINVAL;
>                 }
>
> -               snprintf(name, sizeof(name), "%s:channel%d",
> -                       pdata->label ?: "lp5523", chan);
> +               if (pdata->led_config[chan].name) {
> +                       led->cdev.name = pdata->led_config[chan].name;
> +               } else {
> +                       snprintf(name, sizeof(name), "%s:channel%d",
> +                               pdata->label ?: "lp5523", chan);
> +                       led->cdev.name = name;
> +               }
>
> -               led->cdev.name = name;
>                 led->cdev.brightness_set = lp5523_set_brightness;
>                 res = led_classdev_register(dev, &led->cdev);
>                 if (res < 0) {
> diff --git a/include/linux/leds-lp5523.h b/include/linux/leds-lp5523.h
> index 2694289..727877f 100644
> --- a/include/linux/leds-lp5523.h
> +++ b/include/linux/leds-lp5523.h
> @@ -26,6 +26,7 @@
>  /* See Documentation/leds/leds-lp5523.txt */
>
>  struct lp5523_led_config {
> +       const char      *name;
>         u8              chan_nr;
>         u8              led_current; /* mA x10, 0 if led is not connected */
>         u8              max_current;
> --
> 1.7.2.5
>
>
> Best Regards,
> Milo
>
>



-- 
Bryan Wu <bryan.wu@...onical.com>
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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/