ARM: dts: stm32: drop "st,led1" compatible
authorDario Binacchi <dario.binacchi@amarulasolutions.com>
Mon, 24 Feb 2025 18:39:26 +0000 (19:39 +0100)
committerPatrice Chotard <patrice.chotard@foss.st.com>
Mon, 10 Mar 2025 13:15:14 +0000 (14:15 +0100)
It is pointless to use the custom compatible "st,led1" when
stm32746g-eval.dts and stm32f769-disco.dts already contain the
"gpio-leds" compatible, which is specifically used for GPIO LEDs
management.

Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
arch/arm/dts/stm32746g-eval-u-boot.dtsi
arch/arm/dts/stm32f746-disco-u-boot.dtsi
arch/arm/dts/stm32f769-disco-u-boot.dtsi
board/st/stm32f746-disco/stm32f746-disco.c

index f643292..32b5c7c 100644 (file)
                mmc0 = &sdio1;
                spi0 = &qspi;
        };
-
-       led1 {
-               compatible = "st,led1";
-               led-gpio = <&gpiof 10 0>;
-       };
 };
 
 &fmc {
index a79fca2..38d797e 100644 (file)
                mmc0 = &sdio1;
                spi0 = &qspi;
        };
-
-       led1 {
-               compatible = "st,led1";
-               led-gpio = <&gpioi 1 0>;
-       };
 };
 
 &ltdc {
index a50fba6..7c99a6e 100644 (file)
                spi0 = &qspi;
        };
 
-       led1 {
-               compatible = "st,led1";
-               led-gpio = <&gpioj 5 0>;
-       };
-
        panel: panel {
                compatible = "orisetech,otm8009a";
                reset-gpios = <&gpioj 15 1>;
index 65a39d9..72f479c 100644 (file)
@@ -78,22 +78,6 @@ u32 spl_boot_device(void)
 
 int board_late_init(void)
 {
-       struct gpio_desc gpio = {};
-       int node;
-
-       node = fdt_node_offset_by_compatible(gd->fdt_blob, 0, "st,led1");
-       if (node < 0)
-               return -1;
-
-       gpio_request_by_name_nodev(offset_to_ofnode(node), "led-gpio", 0, &gpio,
-                                  GPIOD_IS_OUT);
-
-       if (dm_gpio_is_valid(&gpio)) {
-               dm_gpio_set_value(&gpio, 0);
-               mdelay(10);
-               dm_gpio_set_value(&gpio, 1);
-       }
-
        return 0;
 }