[PATCH] ARM: ux500: register LP5521 LEDs in the device tree

[Bryan Wu]

On Mon, Jun 3, 2013 at 1:18 AM, Lee Jones <lee.jones at linaro.org> wrote:
> On Fri, 31 May 2013, Bryan Wu wrote:
>
>> On Wed, May 22, 2013 at 1:12 AM, Linus Walleij
>> <linus.walleij at stericsson.com> wrote:
>> > From: Linus Walleij <linus.walleij at linaro.org>
>> >
>> > Based on pending device tree support in the LP55xx drivers
>> > we can add the correct LED and channel configuration from the
>> > ux500 device tree for all HREF variants.
>> >
>> > Cc: Milo Kim <Milo.Kim at ti.com>
>> > Cc: Bryan Wu <cooloney at gmail.com>
>> > Signed-off-by: Linus Walleij <linus.walleij at linaro.org>
>> > ---
>> >  arch/arm/boot/dts/href.dtsi | 38 ++++++++++++++++++++++++++++++++------
>> >  1 file changed, 32 insertions(+), 6 deletions(-)
>> >
>> > diff --git a/arch/arm/boot/dts/href.dtsi b/arch/arm/boot/dts/href.dtsi
>> > index 62523f6..e315429 100644
>> > --- a/arch/arm/boot/dts/href.dtsi
>> > +++ b/arch/arm/boot/dts/href.dtsi
>> > @@ -63,16 +63,42 @@
>> >                 };
>> >
>> >                 i2c at 80128000 {
>> > -                       lp5521 at 0x33 {
>> > -                               compatible = "lp5521";
>> > +                       lp5521 at 33 {
>>
>> Is this right, I guess it should be still "lp5521 at 0x33", although it's
>> just a name.
>
> What makes you say that Bryan?
>
> All other nodes drop the '0x', as it's implied.
>

But I still got this in this patch
"  bh1780 at 0x29 {"

which is not changed. So which one should be the right?

-Bryan