diff mbox series

ARM: dts: logicpd: align LED node names with dtschema

Message ID 20221125144122.476962-1-krzysztof.kozlowski@linaro.org
State New
Headers show
Series ARM: dts: logicpd: align LED node names with dtschema | expand

Commit Message

Krzysztof Kozlowski Nov. 25, 2022, 2:41 p.m. UTC
The node names should be generic and DT schema expects certain pattern:

  logicpd-torpedo-37xx-devkit.dtb: leds: 'user0' does not match any of the regexes: '(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+'

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/logicpd-torpedo-som.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tony Lindgren Nov. 28, 2022, 6:47 a.m. UTC | #1
* Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [221125 14:41]:
> The node names should be generic and DT schema expects certain pattern:
> 
>   logicpd-torpedo-37xx-devkit.dtb: leds: 'user0' does not match any of the regexes: '(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+'

Feel free to apply this along with your other clean up patches:

Acked-by: Tony Lindgren <tony@atomide.com>
Krzysztof Kozlowski Dec. 1, 2022, 5:06 p.m. UTC | #2
On Fri, 25 Nov 2022 15:41:22 +0100, Krzysztof Kozlowski wrote:
> The node names should be generic and DT schema expects certain pattern:
> 
>   logicpd-torpedo-37xx-devkit.dtb: leds: 'user0' does not match any of the regexes: '(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+'
> 
> 

Applied, thanks!

[1/1] ARM: dts: logicpd: align LED node names with dtschema
      https://git.kernel.org/krzk/linux-dt/c/0cd4cab6ac67b5ae88511b24d6c1a8c0e61f9f76

Best regards,
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/logicpd-torpedo-som.dtsi b/arch/arm/boot/dts/logicpd-torpedo-som.dtsi
index 3a5228562b0d..72b5af475d09 100644
--- a/arch/arm/boot/dts/logicpd-torpedo-som.dtsi
+++ b/arch/arm/boot/dts/logicpd-torpedo-som.dtsi
@@ -20,7 +20,7 @@  memory@80000000 {
 
 	leds {
 		compatible = "gpio-leds";
-		user0 {
+		led-user0 {
 			label = "user0";
 			gpios = <&twl_gpio 18 GPIO_ACTIVE_LOW>;	/* LEDA */
 			linux,default-trigger = "none";