[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAK5ve-+cXRR7UHOypoA7bTLd0_K7Y5mNgmH3PadTUAW7TNOmdA@mail.gmail.com>
Date: Mon, 25 Jun 2012 01:20:47 +0800
From: Bryan Wu <bryan.wu@...onical.com>
To: Rob Landley <rob@...dley.net>
Cc: "G.Shark Jeong" <gshark.jeong@...il.com>,
Richard Purdie <rpurdie@...ys.net>,
Daniel Jeong <daniel.jeong@...com>,
linux-kernel@...r.kernel.org, milo kim <milo.kim@...com>,
linux-doc@...r.kernel.org
Subject: Re: [PATCH 1/1 v4] leds: Add LED driver for lm3556 chip
On Sat, Jun 23, 2012 at 3:47 AM, Rob Landley <rob@...dley.net> wrote:
> On 06/21/2012 07:12 PM, G.Shark Jeong wrote:
>> From: "G.Shark Jeong" <gshark.jeong@...il.com>
>>
>> LM3556 :
>> The LM3556 is a 4 MHz fixed-frequency synchronous boost
>> converter plus 1.5A constant current driver for a high-current white LED.
>> Datasheet: www.national.com/ds/LM/LM3556.pdf
>>
>> Tested on OMAP4430
> ...
>> +Indicator pattern example:
>> +pattern 0: #echo 0 > sys/class/leds/indicator/pattern
>> +....
>> +pattern 3: #echo 3 > sys/class/leds/indicator/pattern
>> +
>> +Indicater brightness can be controlled through
>
> Indicator
>
>> +sys/class/leds/indicator/brightness file.
>> +
>> +Example:
>> +OFF : #echo 0 > sys/class/leds/indicator/brightness
>> +5.86 mA : #echo 1 > sys/class/leds/indicator/brightness
>> +........
>> +46.875mA : #echo 8 > sys/class/leds/indicator/brightness
>> +
>> +Notes
>> +-----
>> +Driver expects it is registered using the i2c_board_info mechanism.
>> +Toregister the chip at address 0x63 on specific adapter, set the platform data
>
> To register
>
>> --- a/drivers/leds/Kconfig
>> +++ b/drivers/leds/Kconfig
>> @@ -415,6 +415,14 @@ config LEDS_MAX8997
>> This option enables support for on-chip LED drivers on
>> MAXIM MAX8997 PMIC.
>>
>> +config LEDS_LM3556
>> + tristate "LED support for LM3556 Chip"
>> + depends on LEDS_CLASS && I2C
>> + select REGMAP_I2C
>> + help
>> + This option enables support for LEDs connected LM3556.
>
> connected to
>
> Other than that, documentation bits Acked-by me.
>
No problem. fixed and applied to my for-next branch.
Thanks,
--
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/
Powered by blists - more mailing lists