Devicetree
 help / color / mirror / Atom feed
* [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision
@ 2026-01-19 10:34 Francesco Dolcini
  2026-01-19 12:46 ` Daniel Baluta
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Francesco Dolcini @ 2026-01-19 10:34 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam
  Cc: Francesco Dolcini, devicetree, imx, linux-arm-kernel,
	linux-kernel

From: Francesco Dolcini <francesco.dolcini@toradex.com>

Ixora boards have multiple instances of status leds, to avoid a name
collision add the function-enumerator property.

This fixes the following Linux kernel warnings:

  leds-gpio leds: Led green:status renamed to green:status_1 due to name collision
  leds-gpio leds: Led red:status renamed to red:status_1 due to name collision

Fixes: c083131c9021 ("arm64: dts: freescale: add apalis imx8 aka quadmax carrier board support")
Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
---
 arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi | 4 ++++
 arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi
index 7022de46b8bf..abb131d247c3 100644
--- a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi
@@ -21,6 +21,7 @@ led-1 {
 			color = <LED_COLOR_ID_GREEN>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <1>;
 			gpios = <&lsio_gpio5 27 GPIO_ACTIVE_HIGH>;
 		};
 
@@ -29,6 +30,7 @@ led-2 {
 			color = <LED_COLOR_ID_RED>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <1>;
 			gpios = <&lsio_gpio5 29 GPIO_ACTIVE_HIGH>;
 		};
 
@@ -37,6 +39,7 @@ led-3 {
 			color = <LED_COLOR_ID_GREEN>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <2>;
 			gpios = <&lsio_gpio5 20 GPIO_ACTIVE_HIGH>;
 		};
 
@@ -45,6 +48,7 @@ led-4 {
 			color = <LED_COLOR_ID_RED>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <2>;
 			gpios = <&lsio_gpio5 21 GPIO_ACTIVE_HIGH>;
 		};
 	};
diff --git a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi
index 12732ed7f811..a0b452b92b3e 100644
--- a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi
@@ -21,6 +21,7 @@ led-1 {
 			color = <LED_COLOR_ID_GREEN>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <1>;
 			gpios = <&lsio_gpio5 27 GPIO_ACTIVE_HIGH>;
 		};
 
@@ -29,6 +30,7 @@ led-2 {
 			color = <LED_COLOR_ID_RED>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <1>;
 			gpios = <&lsio_gpio5 29 GPIO_ACTIVE_HIGH>;
 		};
 
@@ -37,6 +39,7 @@ led-3 {
 			color = <LED_COLOR_ID_GREEN>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <2>;
 			gpios = <&lsio_gpio5 20 GPIO_ACTIVE_HIGH>;
 		};
 
@@ -45,6 +48,7 @@ led-4 {
 			color = <LED_COLOR_ID_RED>;
 			default-state = "off";
 			function = LED_FUNCTION_STATUS;
+			function-enumerator = <2>;
 			gpios = <&lsio_gpio5 21 GPIO_ACTIVE_HIGH>;
 		};
 	};
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision
  2026-01-19 10:34 [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision Francesco Dolcini
@ 2026-01-19 12:46 ` Daniel Baluta
  2026-01-21 20:05 ` Frank Li
  2026-02-24 17:00 ` Frank Li
  2 siblings, 0 replies; 4+ messages in thread
From: Daniel Baluta @ 2026-01-19 12:46 UTC (permalink / raw)
  To: Francesco Dolcini
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	Francesco Dolcini, devicetree, imx, linux-arm-kernel,
	linux-kernel

On Mon, Jan 19, 2026 at 12:35 PM Francesco Dolcini <francesco@dolcini.it> wrote:
>
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
>
> Ixora boards have multiple instances of status leds, to avoid a name
> collision add the function-enumerator property.
>
> This fixes the following Linux kernel warnings:
>
>   leds-gpio leds: Led green:status renamed to green:status_1 due to name collision
>   leds-gpio leds: Led red:status renamed to red:status_1 due to name collision
>
> Fixes: c083131c9021 ("arm64: dts: freescale: add apalis imx8 aka quadmax carrier board support")
> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>

Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision
  2026-01-19 10:34 [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision Francesco Dolcini
  2026-01-19 12:46 ` Daniel Baluta
@ 2026-01-21 20:05 ` Frank Li
  2026-02-24 17:00 ` Frank Li
  2 siblings, 0 replies; 4+ messages in thread
From: Frank Li @ 2026-01-21 20:05 UTC (permalink / raw)
  To: Francesco Dolcini
  Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	Francesco Dolcini, devicetree, imx, linux-arm-kernel,
	linux-kernel

On Mon, Jan 19, 2026 at 11:34:09AM +0100, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
>
> Ixora boards have multiple instances of status leds, to avoid a name
> collision add the function-enumerator property.
>
> This fixes the following Linux kernel warnings:
>
>   leds-gpio leds: Led green:status renamed to green:status_1 due to name collision
>   leds-gpio leds: Led red:status renamed to red:status_1 due to name collision
>
> Fixes: c083131c9021 ("arm64: dts: freescale: add apalis imx8 aka quadmax carrier board support")
> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>

Reviewed-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi | 4 ++++
>  arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi | 4 ++++
>  2 files changed, 8 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi
> index 7022de46b8bf..abb131d247c3 100644
> --- a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.1.dtsi
> @@ -21,6 +21,7 @@ led-1 {
>  			color = <LED_COLOR_ID_GREEN>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <1>;
>  			gpios = <&lsio_gpio5 27 GPIO_ACTIVE_HIGH>;
>  		};
>
> @@ -29,6 +30,7 @@ led-2 {
>  			color = <LED_COLOR_ID_RED>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <1>;
>  			gpios = <&lsio_gpio5 29 GPIO_ACTIVE_HIGH>;
>  		};
>
> @@ -37,6 +39,7 @@ led-3 {
>  			color = <LED_COLOR_ID_GREEN>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <2>;
>  			gpios = <&lsio_gpio5 20 GPIO_ACTIVE_HIGH>;
>  		};
>
> @@ -45,6 +48,7 @@ led-4 {
>  			color = <LED_COLOR_ID_RED>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <2>;
>  			gpios = <&lsio_gpio5 21 GPIO_ACTIVE_HIGH>;
>  		};
>  	};
> diff --git a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi
> index 12732ed7f811..a0b452b92b3e 100644
> --- a/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8-apalis-ixora-v1.2.dtsi
> @@ -21,6 +21,7 @@ led-1 {
>  			color = <LED_COLOR_ID_GREEN>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <1>;
>  			gpios = <&lsio_gpio5 27 GPIO_ACTIVE_HIGH>;
>  		};
>
> @@ -29,6 +30,7 @@ led-2 {
>  			color = <LED_COLOR_ID_RED>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <1>;
>  			gpios = <&lsio_gpio5 29 GPIO_ACTIVE_HIGH>;
>  		};
>
> @@ -37,6 +39,7 @@ led-3 {
>  			color = <LED_COLOR_ID_GREEN>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <2>;
>  			gpios = <&lsio_gpio5 20 GPIO_ACTIVE_HIGH>;
>  		};
>
> @@ -45,6 +48,7 @@ led-4 {
>  			color = <LED_COLOR_ID_RED>;
>  			default-state = "off";
>  			function = LED_FUNCTION_STATUS;
> +			function-enumerator = <2>;
>  			gpios = <&lsio_gpio5 21 GPIO_ACTIVE_HIGH>;
>  		};
>  	};
> --
> 2.47.3
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision
  2026-01-19 10:34 [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision Francesco Dolcini
  2026-01-19 12:46 ` Daniel Baluta
  2026-01-21 20:05 ` Frank Li
@ 2026-02-24 17:00 ` Frank Li
  2 siblings, 0 replies; 4+ messages in thread
From: Frank Li @ 2026-02-24 17:00 UTC (permalink / raw)
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Shawn Guo,
	Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
	Francesco Dolcini
  Cc: Frank Li, Francesco Dolcini, devicetree, imx, linux-arm-kernel,
	linux-kernel


On Mon, 19 Jan 2026 11:34:09 +0100, Francesco Dolcini wrote:
> Ixora boards have multiple instances of status leds, to avoid a name
> collision add the function-enumerator property.
>
> This fixes the following Linux kernel warnings:
>
>   leds-gpio leds: Led green:status renamed to green:status_1 due to name collision
>   leds-gpio leds: Led red:status renamed to red:status_1 due to name collision
>
> [...]

Applied, thanks!

[1/1] arm64: dts: imx8-apalis: Fix LEDs name collision

Best regards,
--
Frank Li <Frank.Li@nxp.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-02-24 17:01 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-19 10:34 [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision Francesco Dolcini
2026-01-19 12:46 ` Daniel Baluta
2026-01-21 20:05 ` Frank Li
2026-02-24 17:00 ` Frank Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox