On 25/08/15 22:36, Stephen Boyd wrote:

On 08/18/2015 06:10 AM, Srinivas Kandagatla wrote:

quoted

diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts

b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
index b1f9ddb..08daafe 100644

--- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts

@@ -221,6 +221,18 @@
              status = "okay";
          };
+        leds {
+            compatible = "gpio-leds";
+            pinctrl-names = "default";
+            pinctrl-0 = <&notify_led>;
+
+            led@1 {
+                label = "apq8064:green:user1";
+                gpios = <&pm8921_gpio 18 GPIO_ACTIVE_HIGH>;
+                default-state = "on";
+            };
+        };

Wrong place. Should be in root, not soc node.

yep, Will move it to the root.