Commit 216e6274 authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Bjorn Andersson

arm64: dts: qcom: apq8016-sbc: correct GPIO LEDs node names

Individual LEDs in a GPIO LEDs device node are not addressable, thus
unit address is not correct.

dtc is also not happy:

  Warning (unit_address_vs_reg): /leds/led@5: node has a unit name, but no reg or ranges property
Reported-by: default avatarSumit Garg <sumit.garg@linaro.org>
Closes: https://lore.kernel.org/all/CAFA6WYNRwF7GqhBk2B7i-deT3aLxNQckhnOasjip2TYm4HZgAw@mail.gmail.com/Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: default avatarSumit Garg <sumit.garg@linaro.org>
Reviewed-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Link: https://lore.kernel.org/r/20240314112657.167006-1-krzysztof.kozlowski@linaro.orgSigned-off-by: default avatarBjorn Andersson <andersson@kernel.org>
parent 9f42f738
...@@ -91,7 +91,7 @@ leds { ...@@ -91,7 +91,7 @@ leds {
compatible = "gpio-leds"; compatible = "gpio-leds";
led@1 { led-1 {
label = "apq8016-sbc:green:user1"; label = "apq8016-sbc:green:user1";
function = LED_FUNCTION_HEARTBEAT; function = LED_FUNCTION_HEARTBEAT;
color = <LED_COLOR_ID_GREEN>; color = <LED_COLOR_ID_GREEN>;
...@@ -100,7 +100,7 @@ led@1 { ...@@ -100,7 +100,7 @@ led@1 {
default-state = "off"; default-state = "off";
}; };
led@2 { led-2 {
label = "apq8016-sbc:green:user2"; label = "apq8016-sbc:green:user2";
function = LED_FUNCTION_DISK_ACTIVITY; function = LED_FUNCTION_DISK_ACTIVITY;
color = <LED_COLOR_ID_GREEN>; color = <LED_COLOR_ID_GREEN>;
...@@ -109,7 +109,7 @@ led@2 { ...@@ -109,7 +109,7 @@ led@2 {
default-state = "off"; default-state = "off";
}; };
led@3 { led-3 {
label = "apq8016-sbc:green:user3"; label = "apq8016-sbc:green:user3";
function = LED_FUNCTION_DISK_ACTIVITY; function = LED_FUNCTION_DISK_ACTIVITY;
color = <LED_COLOR_ID_GREEN>; color = <LED_COLOR_ID_GREEN>;
...@@ -118,7 +118,7 @@ led@3 { ...@@ -118,7 +118,7 @@ led@3 {
default-state = "off"; default-state = "off";
}; };
led@4 { led-4 {
label = "apq8016-sbc:green:user4"; label = "apq8016-sbc:green:user4";
color = <LED_COLOR_ID_GREEN>; color = <LED_COLOR_ID_GREEN>;
gpios = <&pm8916_gpios 2 GPIO_ACTIVE_HIGH>; gpios = <&pm8916_gpios 2 GPIO_ACTIVE_HIGH>;
...@@ -127,7 +127,7 @@ led@4 { ...@@ -127,7 +127,7 @@ led@4 {
default-state = "off"; default-state = "off";
}; };
led@5 { led-5 {
label = "apq8016-sbc:yellow:wlan"; label = "apq8016-sbc:yellow:wlan";
function = LED_FUNCTION_WLAN; function = LED_FUNCTION_WLAN;
color = <LED_COLOR_ID_YELLOW>; color = <LED_COLOR_ID_YELLOW>;
...@@ -136,7 +136,7 @@ led@5 { ...@@ -136,7 +136,7 @@ led@5 {
default-state = "off"; default-state = "off";
}; };
led@6 { led-6 {
label = "apq8016-sbc:blue:bt"; label = "apq8016-sbc:blue:bt";
function = LED_FUNCTION_BLUETOOTH; function = LED_FUNCTION_BLUETOOTH;
color = <LED_COLOR_ID_BLUE>; color = <LED_COLOR_ID_BLUE>;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment