ARM: dts: stm32: drop "st,button1" compatible
authorDario Binacchi <dario.binacchi@amarulasolutions.com>
Mon, 24 Feb 2025 18:39:25 +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,button1" when
stm32746g-eval.dts and stm32f769-disco.dts already contain the
"gpio-keys" compatible, which is specifically used for button
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 1c288ac..f643292 100644 (file)
                spi0 = &qspi;
        };
 
-       button1 {
-               compatible = "st,button1";
-               button-gpio = <&gpioc 13 0>;
-       };
-
        led1 {
                compatible = "st,led1";
                led-gpio = <&gpiof 10 0>;
index 1b42d6c..a79fca2 100644 (file)
                spi0 = &qspi;
        };
 
-       button1 {
-               compatible = "st,button1";
-               button-gpio = <&gpioi 11 0>;
-       };
-
        led1 {
                compatible = "st,led1";
                led-gpio = <&gpioi 1 0>;
index add55c9..a50fba6 100644 (file)
                spi0 = &qspi;
        };
 
-       button1 {
-               compatible = "st,button1";
-               button-gpio = <&gpioa 0 0>;
-       };
-
        led1 {
                compatible = "st,led1";
                led-gpio = <&gpioj 5 0>;
index 8966a09..65a39d9 100644 (file)
@@ -94,21 +94,6 @@ int board_late_init(void)
                dm_gpio_set_value(&gpio, 1);
        }
 
-       /* read button 1*/
-       node = fdt_node_offset_by_compatible(gd->fdt_blob, 0, "st,button1");
-       if (node < 0)
-               return -1;
-
-       gpio_request_by_name_nodev(offset_to_ofnode(node), "button-gpio", 0,
-                                  &gpio, GPIOD_IS_IN);
-
-       if (dm_gpio_is_valid(&gpio)) {
-               if (dm_gpio_get_value(&gpio))
-                       puts("usr button is at HIGH LEVEL\n");
-               else
-                       puts("usr button is at LOW LEVEL\n");
-       }
-
        return 0;
 }