arm64: dts: imx8-apalis: Fix LEDs name collision

[PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision

Posted by Francesco Dolcini 2 weeks, 5 days ago

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

Re: [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision

Posted by Frank Li 2 weeks, 3 days ago

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
>

Re: [PATCH v1] arm64: dts: imx8-apalis: Fix LEDs name collision

Posted by Daniel Baluta 2 weeks, 5 days ago

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>