OSDN Git Service

ARM: dts: at91: align LED node names with dtschema
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Fri, 25 Nov 2022 14:41:30 +0000 (15:41 +0100)
committerKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Fri, 13 Jan 2023 10:38:33 +0000 (11:38 +0100)
The node names should be generic and DT schema expects certain pattern:

  at91-sama5d2_ptc_ek.dtb: leds: 'blue', 'green', 'red' do not match any of the regexes: '(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+'

Link: https://lore.kernel.org/r/20221125144131.477089-1-krzysztof.kozlowski@linaro.org
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
15 files changed:
arch/arm/boot/dts/at91-gatwick.dts
arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
arch/arm/boot/dts/at91-sama5d27_wlsom1_ek.dts
arch/arm/boot/dts/at91-sama5d2_icp.dts
arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
arch/arm/boot/dts/at91-sama5d2_xplained.dts
arch/arm/boot/dts/at91-sama5d3_xplained.dts
arch/arm/boot/dts/at91-sama5d4_ma5d4evk.dts
arch/arm/boot/dts/at91-sama5d4_xplained.dts
arch/arm/boot/dts/at91-sama5d4ek.dts
arch/arm/boot/dts/at91-tse850-3.dts
arch/arm/boot/dts/sama5d31ek.dts
arch/arm/boot/dts/sama5d34ek.dts
arch/arm/boot/dts/sama5d3xcm.dtsi
arch/arm/boot/dts/sama5d3xcm_cmp.dtsi

index 2c718cf..551300f 100644 (file)
        leds {
                compatible = "gpio-leds";
 
-               ethernet {
+               led-ethernet {
                        label = "gatwick:yellow:ethernet";
                        gpios = <&pioA 10 GPIO_ACTIVE_LOW>;
                        default-state = "off";
                };
 
-               wifi {
+               led-wifi {
                        label = "gatwick:green:wifi";
                        gpios = <&pioA 28 GPIO_ACTIVE_LOW>;
                        default-state = "off";
                };
 
-               ble {
+               led-ble {
                        label = "gatwick:blue:ble";
                        gpios = <&pioA 22 GPIO_ACTIVE_LOW>;
                        default-state = "off";
                };
 
-               lora {
+               led-lora {
                        label = "gatwick:orange:lora";
                        gpios = <&pioA 26 GPIO_ACTIVE_LOW>;
                        default-state = "off";
                };
 
-               blank {
+               led-blank {
                        label = "gatwick:green:blank";
                        gpios = <&pioA 24 GPIO_ACTIVE_LOW>;
                        default-state = "off";
                };
 
-               user {
+               led-user {
                        label = "gatwick:yellow:user";
                        gpios = <&pioA 12 GPIO_ACTIVE_LOW>;
                        default-state = "off";
index 0dc6ca3..52ddd05 100644 (file)
                pinctrl-0 = <&pinctrl_led_gpio_default>;
                status = "okay"; /* Conflict with pwm0. */
 
-               red {
+               led-red {
                        label = "red";
                        gpios = <&pioA PIN_PA10 GPIO_ACTIVE_HIGH>;
                };
 
-               green {
+               led-green {
                        label = "green";
                        gpios = <&pioA PIN_PB1 GPIO_ACTIVE_HIGH>;
                };
 
-               blue {
+               led-blue {
                        label = "blue";
                        gpios = <&pioA PIN_PA31 GPIO_ACTIVE_HIGH>;
                        linux,default-trigger = "heartbeat";
index b665ddc..e055b9e 100644 (file)
                pinctrl-0 = <&pinctrl_led_gpio_default>;
                status = "okay";
 
-               red {
+               led-red {
                        label = "red";
                        gpios = <&pioA PIN_PA6 GPIO_ACTIVE_HIGH>;
                };
 
-               green {
+               led-green {
                        label = "green";
                        gpios = <&pioA PIN_PA7 GPIO_ACTIVE_HIGH>;
                };
 
-               blue {
+               led-blue {
                        label = "blue";
                        gpios = <&pioA PIN_PA8 GPIO_ACTIVE_HIGH>;
                        linux,default-trigger = "heartbeat";
index dd1dec9..1346b8f 100644 (file)
                pinctrl-0 = <&pinctrl_led_gpio_default>;
                status = "okay"; /* conflict with pwm0 */
 
-               red {
+               led-red {
                        label = "red";
                        gpios = <&pioA PIN_PB0 GPIO_ACTIVE_HIGH>;
                };
 
-               green {
+               led-green {
                        label = "green";
                        gpios = <&pioA PIN_PB1 GPIO_ACTIVE_HIGH>;
                };
 
-               blue {
+               led-blue {
                        label = "blue";
                        gpios = <&pioA PIN_PA31 GPIO_ACTIVE_HIGH>;
                        linux,default-trigger = "heartbeat";
index 76a711b..bf1c9ca 100644 (file)
                pinctrl-0 = <&pinctrl_led_gpio_default>;
                status = "okay";
 
-               red {
+               led-red {
                        label = "red";
                        gpios = <&pioA PIN_PB10 GPIO_ACTIVE_HIGH>;
                };
 
-               green {
+               led-green {
                        label = "green";
                        gpios = <&pioA PIN_PB8 GPIO_ACTIVE_HIGH>;
                };
 
-               blue {
+               led-blue {
                        label = "blue";
                        gpios = <&pioA PIN_PB6 GPIO_ACTIVE_HIGH>;
                        linux,default-trigger = "heartbeat";
index 85949c2..2d53c47 100644 (file)
                pinctrl-0 = <&pinctrl_led_gpio_default>;
                status = "okay"; /* conflict with pwm0 */
 
-               red {
+               led-red {
                        label = "red";
                        gpios = <&pioA PIN_PB6 GPIO_ACTIVE_LOW>;
                };
 
 
-               green {
+               led-green {
                        label = "green";
                        gpios = <&pioA PIN_PB5 GPIO_ACTIVE_LOW>;
                };
 
-               blue {
+               led-blue {
                        label = "blue";
                        gpios = <&pioA PIN_PB0 GPIO_ACTIVE_LOW>;
                        linux,default-trigger = "heartbeat";
index 1f42a6a..8200337 100644 (file)
                pinctrl-0 = <&pinctrl_gpio_leds>;
                status = "okay";
 
-               d2 {
+               led-d2 {
                        label = "d2";
                        gpios = <&pioE 23 GPIO_ACTIVE_LOW>;     /* PE23, conflicts with A23, CTS2 */
                        linux,default-trigger = "heartbeat";
                };
 
-               d3 {
+               led-d3 {
                        label = "d3"; /* Conflict with EBI CS0, USART2 CTS. */
                        gpios = <&pioE 24 GPIO_ACTIVE_HIGH>;
                };
index 4d7cee5..8adf567 100644 (file)
                compatible = "gpio-leds";
                status = "okay";
 
-               user1 {
+               led-user1 {
                        label = "user1";
                        gpios = <&pioD 28 GPIO_ACTIVE_HIGH>;
                        linux,default-trigger = "heartbeat";
                };
 
-               user2 {
+               led-user2 {
                        label = "user2";
                        gpios = <&pioD 29 GPIO_ACTIVE_HIGH>;
                        linux,default-trigger = "heartbeat";
                };
 
-               user3 {
+               led-user3 {
                        label = "user3";
                        gpios = <&pioD 30 GPIO_ACTIVE_HIGH>;
                        linux,default-trigger = "heartbeat";
index f122f30..95d701d 100644 (file)
                pinctrl-0 = <&pinctrl_gpio_leds>;
                status = "okay";
 
-               d8 {
+               led-d8 {
                        label = "d8";
                        gpios = <&pioD 30 GPIO_ACTIVE_HIGH>;
                        default-state = "on";
                };
 
-               d10 {
+               led-d10 {
                        label = "d10";
                        gpios = <&pioE 15 GPIO_ACTIVE_LOW>;
                        linux,default-trigger = "heartbeat";
index fce4e93..20ac775 100644 (file)
                compatible = "gpio-leds";
                status = "okay";
 
-               d8 {
+               led-d8 {
                        label = "d8";
                        /* PE28, conflicts with usart4 rts pin */
                        gpios = <&pioE 28 GPIO_ACTIVE_LOW>;
                };
 
-               d9 {
+               led-d9 {
                        label = "d9";
                        gpios = <&pioE 9 GPIO_ACTIVE_HIGH>;
                };
 
-               d10 {
+               led-d10 {
                        label = "d10";
                        gpios = <&pioE 8 GPIO_ACTIVE_LOW>;
                        linux,default-trigger = "heartbeat";
index 7e5c598..b99a4fb 100644 (file)
        leds {
                compatible = "gpio-leds";
 
-               ch1-red {
+               led-ch1-red {
                        label = "ch-1:red";
                        gpios = <&pioA 23 GPIO_ACTIVE_LOW>;
                };
-               ch1-green {
+               led-ch1-green {
                        label = "ch-1:green";
                        gpios = <&pioA 22 GPIO_ACTIVE_LOW>;
                };
-               ch2-red {
+               led-ch2-red {
                        label = "ch-2:red";
                        gpios = <&pioA 21 GPIO_ACTIVE_LOW>;
                };
-               ch2-green {
+               led-ch2-green {
                        label = "ch-2:green";
                        gpios = <&pioA 20 GPIO_ACTIVE_LOW>;
                };
-               data-red {
+               led-data-red {
                        label = "data:red";
                        gpios = <&pioA 19 GPIO_ACTIVE_LOW>;
                };
-               data-green {
+               led-data-green {
                        label = "data:green";
                        gpios = <&pioA 18 GPIO_ACTIVE_LOW>;
                };
-               alarm-red {
+               led-alarm-red {
                        label = "alarm:red";
                        gpios = <&pioA 17 GPIO_ACTIVE_LOW>;
                };
-               alarm-green {
+               led-alarm-green {
                        label = "alarm:green";
                        gpios = <&pioA 16 GPIO_ACTIVE_LOW>;
                };
index 10fc80d..1f2dfb3 100644 (file)
@@ -40,7 +40,7 @@
        };
 
        leds {
-               d3 {
+               led-d3 {
                        label = "d3";
                        gpios = <&pioE 24 GPIO_ACTIVE_HIGH>;
                };
index 2335bf9..bffd613 100644 (file)
@@ -50,7 +50,7 @@
        };
 
        leds {
-               d3 {
+               led-d3 {
                        label = "d3";
                        gpios = <&pioE 24 GPIO_ACTIVE_HIGH>;
                };
index 3843356..7d1d785 100644 (file)
        leds {
                compatible = "gpio-leds";
 
-               d2 {
+               led-d2 {
                        label = "d2";
                        gpios = <&pioE 25 GPIO_ACTIVE_LOW>;     /* PE25, conflicts with A25, RXD2 */
                        linux,default-trigger = "heartbeat";
index 5579c95..830a095 100644 (file)
        leds {
                compatible = "gpio-leds";
 
-               d2 {
+               led-d2 {
                        label = "d2";
                        gpios = <&pioE 25 GPIO_ACTIVE_LOW>;
                        linux,default-trigger = "heartbeat";