Skip to contentSkip to conversation

ARM: dts: nxp: imx: Fix mc13xxx LED node names

PATCH Applied Rob Herring (Arm) 3 participants 3 messages Nov 18, 2025 – Dec 29, 2025

Subsystems (2)
Device tree bindings Arm i.MX and Layerscape
lore
Share & tools
lore

Rob Herring (Arm)

author maintainer
#0 lore raw

PATCH 1/1 ARM: dts: nxp: imx: Fix mc13xxx LED node names

Node names are supposed to be generic and use hexadecimal unit-addresses.

Signed-off-by Rob Herring (Arm) <[email protected]>
---
 arch/arm/boot/dts/nxp/imx/imx27-phytec-phycore-rdk.dts | 8 ++++----
 arch/arm/boot/dts/nxp/imx/imx51-zii-rdu1.dts           | 4 ++--
 arch/arm/boot/dts/nxp/imx/imx51-zii-scu2-mezz.dts      | 4 ++--
 arch/arm/boot/dts/nxp/imx/imx51-zii-scu3-esb.dts       | 4 ++--
 4 files changed, 10 insertions(+), 10 deletions(-)
arch/arm/boot/dts/nxp/imx/imx27-phytec-phycore-rdk.dts +4 -4
index b8048e12e3d9..5398e9067e60 100644
@@ -248,14 +248,14 @@ ledr3: led@9 {
 		linux,default-trigger = "nand-disk";
 	};
 
-	ledg3: led@10 {
-		reg = <10>;
+	ledg3: led@a {
+		reg = <0xa>;
 		label = "system:green3:live";
 		linux,default-trigger = "heartbeat";
 	};
 
-	ledb3: led@11 {
-		reg = <11>;
+	ledb3: led@b {
+		reg = <0xb>;
 		label = "system:blue3:cpu";
 		linux,default-trigger = "cpu0";
 	};
arch/arm/boot/dts/nxp/imx/imx51-zii-rdu1.dts +2 -2
index 06545a6052f7..238138c4ee72 100644
@@ -398,13 +398,13 @@ leds {
 			#size-cells = <0>;
 			led-control = <0x0 0x0 0x3f83f8 0x0>;
 
-			sysled0@3 {
+			led@3 {
 				reg = <3>;
 				label = "system:green:status";
 				linux,default-trigger = "default-on";
 			};
 
-			sysled1@4 {
+			led@4 {
 				reg = <4>;
 				label = "system:green:act";
 				linux,default-trigger = "heartbeat";
arch/arm/boot/dts/nxp/imx/imx51-zii-scu2-mezz.dts +2 -2
index 26eb7a9506e4..1598bf4f4991 100644
@@ -225,13 +225,13 @@ leds {
 			#size-cells = <0>;
 			led-control = <0x0 0x0 0x3f83f8 0x0>;
 
-			sysled3: led3@3 {
+			sysled3: led@3 {
 				reg = <3>;
 				label = "system:red:power";
 				linux,default-trigger = "default-on";
 			};
 
-			sysled4: led4@4 {
+			sysled4: led@4 {
 				reg = <4>;
 				label = "system:green:act";
 				linux,default-trigger = "heartbeat";
arch/arm/boot/dts/nxp/imx/imx51-zii-scu3-esb.dts +2 -2
index 19a3b142c964..c2dcfd44c445 100644
@@ -153,13 +153,13 @@ leds {
 			#size-cells = <0>;
 			led-control = <0x0 0x0 0x3f83f8 0x0>;
 
-			sysled3: led3@3 {
+			sysled3: led@3 {
 				reg = <3>;
 				label = "system:red:power";
 				linux,default-trigger = "default-on";
 			};
 
-			sysled4: led4@4 {
+			sysled4: led@4 {
 				reg = <4>;
 				label = "system:green:act";
 				linux,default-trigger = "heartbeat";
Signature · 1 line
2.51.0
Reviewed-by on v1 1/1
#1 lore raw
Quoted 9 lines · Rob Herring (Arm)

Node names are supposed to be generic and use hexadecimal unit-addresses.

Signed-off-by Rob Herring (Arm) <[email protected]>
---
 arch/arm/boot/dts/nxp/imx/imx27-phytec-phycore-rdk.dts | 8 ++++----
 arch/arm/boot/dts/nxp/imx/imx51-zii-rdu1.dts           | 4 ++--
 arch/arm/boot/dts/nxp/imx/imx51-zii-scu2-mezz.dts      | 4 ++--
 arch/arm/boot/dts/nxp/imx/imx51-zii-scu3-esb.dts       | 4 ++--
 4 files changed, 10 insertions(+), 10 deletions(-)
Reviewed-by Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof