diff mbox series

[04/23] ARM: dts: qcom: apq8064-ifc6410: fix user1 LED node name

Message ID 20220930185236.867655-5-dmitry.baryshkov@linaro.org
State Accepted
Commit 5564d5337f839a24439dda4dbd9477e52a0eeaf5
Headers show
Series ARM: dts: qcom: rework DT for apq8064, msm8960 and msm8660 | expand

Commit Message

Dmitry Baryshkov Sept. 30, 2022, 6:52 p.m. UTC 
There are no address-cells for the gpio-leds child nodes, so rename
led@1 node to led-user1.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm/boot/dts/qcom-apq8064-ifc6410.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Neil Armstrong Oct. 3, 2022, 4:33 p.m. UTC | #1 
On 30/09/2022 20:52, Dmitry Baryshkov wrote:
> There are no address-cells for the gpio-leds child nodes, so rename
> led@1 node to led-user1.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>   arch/arm/boot/dts/qcom-apq8064-ifc6410.dts | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
> index f718b37192e9..610fb8d652c3 100644
> --- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
> +++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
> @@ -38,7 +38,7 @@ leds {
>   		pinctrl-names = "default";
>   		pinctrl-0 = <&notify_led>;
>   
> -		led@1 {
> +		led-user1 {
>   			label = "apq8064:green:user1";
>   			color = <LED_COLOR_ID_GREEN>;
>   			gpios = <&pm8921_gpio 18 GPIO_ACTIVE_HIGH>;

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
index f718b37192e9..610fb8d652c3 100644
--- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
+++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts
@@ -38,7 +38,7 @@  leds {
 		pinctrl-names = "default";
 		pinctrl-0 = <&notify_led>;
 
-		led@1 {
+		led-user1 {
 			label = "apq8064:green:user1";
 			color = <LED_COLOR_ID_GREEN>;
 			gpios = <&pm8921_gpio 18 GPIO_ACTIVE_HIGH>;