On Mon, Jun 3, 2013 at 1:18 AM, Lee Jones [off-list ref] wrote:

On Fri, 31 May 2013, Bryan Wu wrote:

quoted

On Wed, May 22, 2013 at 1:12 AM, Linus Walleij
[off-list ref] wrote:

quoted

From: Linus Walleij <redacted>

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 <redacted>
Cc: Bryan Wu <redacted>
Signed-off-by: Linus Walleij <redacted>
---
 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