ARM: dts: ux500: Fix LED probing
authorLinus Walleij <linus.walleij@linaro.org>
Sun, 13 Jun 2021 12:33:56 +0000 (14:33 +0200)
committerOlof Johansson <olof@lixom.net>
Tue, 15 Jun 2021 15:12:29 +0000 (08:12 -0700)
The Ux500 HREF LEDs have not been probing properly for a
while as this was introduce:

     ret = of_property_read_u32(np, "color", &led_color);
     if (ret)
             return ret;

Since the device tree did not define the new invented color
attribute, probe was failing.

Define color attributes for the LEDs so they work again.

Link: https://lore.kernel.org/r/20210613123356.880933-1-linus.walleij@linaro.org
Fixes: 92a81562e695 ("leds: lp55xx: Add multicolor framework support to lp55xx")
Cc: stable@vger.kernel.org
Cc: Dan Murphy <dmurphy@ti.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Olof Johansson <olof@lixom.net>
arch/arm/boot/dts/ste-href.dtsi

index 83b1796..13d2161 100644 (file)
@@ -4,6 +4,7 @@
  */
 
 #include <dt-bindings/interrupt-controller/irq.h>
+#include <dt-bindings/leds/common.h>
 #include "ste-href-family-pinctrl.dtsi"
 
 / {
                                        reg = <0>;
                                        led-cur = /bits/ 8 <0x2f>;
                                        max-cur = /bits/ 8 <0x5f>;
+                                       color = <LED_COLOR_ID_BLUE>;
                                        linux,default-trigger = "heartbeat";
                                };
                                chan@1 {
                                        reg = <1>;
                                        led-cur = /bits/ 8 <0x2f>;
                                        max-cur = /bits/ 8 <0x5f>;
+                                       color = <LED_COLOR_ID_BLUE>;
                                };
                                chan@2 {
                                        reg = <2>;
                                        led-cur = /bits/ 8 <0x2f>;
                                        max-cur = /bits/ 8 <0x5f>;
+                                       color = <LED_COLOR_ID_BLUE>;
                                };
                        };
                        lp5521@34 {
                                        reg = <0>;
                                        led-cur = /bits/ 8 <0x2f>;
                                        max-cur = /bits/ 8 <0x5f>;
+                                       color = <LED_COLOR_ID_BLUE>;
                                };
                                chan@1 {
                                        reg = <1>;
                                        led-cur = /bits/ 8 <0x2f>;
                                        max-cur = /bits/ 8 <0x5f>;
+                                       color = <LED_COLOR_ID_BLUE>;
                                };
                                chan@2 {
                                        reg = <2>;
                                        led-cur = /bits/ 8 <0x2f>;
                                        max-cur = /bits/ 8 <0x5f>;
+                                       color = <LED_COLOR_ID_BLUE>;
                                };
                        };
                        bh1780@29 {