led: remove support for green status led in legacy API

The last user of it was removed in a previous commit so let's remove its
support entirely.

Signed-off-by: Quentin Schulz <quentin.schulz@cherry.de>
Reviewed-by: Heiko Schocher <hs@nabladev.com>

diff --git a/cmd/legacy_led.c b/cmd/legacy_led.c
index 609a4b08e0a5b9825c3d0fa3fe10c83f1fa4e347..2c6d606f48b097e1749403a52cc2d6097e5c7033 100644 (file)
--- a/cmd/legacy_led.c
+++ b/cmd/legacy_led.c
@@ -45,9 +45,6 @@ static const led_tbl_t led_commands[] = {
        { "5", CONFIG_LED_STATUS_BIT5, NULL, NULL, NULL },
 #endif
 #endif
-#ifdef CONFIG_LED_STATUS_GREEN
-       { "green", CONFIG_LED_STATUS_GREEN, green_led_off, green_led_on, NULL },
-#endif
 #ifdef CONFIG_LED_STATUS_RED
        { "red", CONFIG_LED_STATUS_RED, red_led_off, red_led_on, NULL },
 #endif
@@ -162,9 +159,6 @@ U_BOOT_CMD(
        "5|"
 #endif
 #endif
-#ifdef CONFIG_LED_STATUS_GREEN
-       "green|"
-#endif
 #ifdef CONFIG_LED_STATUS_RED
        "red|"
 #endif

diff --git a/common/board_f.c b/common/board_f.c
index 224cc0ac561f78198a19b0c32b0bca7690f81382..74571cb6d91d3d488ba05991954ece61e421b8ed 100644 (file)
--- a/common/board_f.c
+++ b/common/board_f.c
@@ -67,8 +67,6 @@ DECLARE_GLOBAL_DATA_PTR;
  */
 __weak void red_led_on(void) {}
 __weak void red_led_off(void) {}
-__weak void green_led_on(void) {}
-__weak void green_led_off(void) {}
 
 /*
  * Why is gd allocated a register? Prior to reloc it might be better to

diff --git a/doc/api/led.rst b/doc/api/led.rst
index 6488416c8b91661c5b28a5cc06560d30a4ab26f9..6889f4878515789a18103098dff0d87deadbbb1c 100644 (file)
--- a/doc/api/led.rst
+++ b/doc/api/led.rst
@@ -58,8 +58,7 @@ CONFIG_STATUS_LED_BOOT is the LED to light when the board is booting.
 This must be a valid LED number (0-5).
 
 CONFIG_STATUS_LED_RED is the red LED. It is used to signal errors. This must be
-a valid LED number (0-5). Other similar color LED's macros are
-CONFIG_STATUS_LED_GREEN.
+a valid LED number (0-5).
 
 General LED functions
 ~~~~~~~~~~~~~~~~~~~~~
@@ -81,8 +80,6 @@ The functions names explain their purpose.
 
 - red_LED_on
 - red_LED_off
-- green_LED_on
-- green_LED_off
 
 These are weakly defined in arch/arm/lib/board.c to noops. Where applicable, define
 these functions in the board specific source.

diff --git a/drivers/led/Kconfig b/drivers/led/Kconfig
index 1077792efb056de7b7e6e98f2e3a2c231dd03d42..6b043f9a522340459c00f4e70876a540d0ddaa5e 100644 (file)
--- a/drivers/led/Kconfig
+++ b/drivers/led/Kconfig
@@ -414,20 +414,6 @@ config LED_STATUS_RED
 
 endif # LED_STATUS_RED_ENABLE
 
-config LED_STATUS_GREEN_ENABLE
-       bool "Enable green LED"
-       help
-         Enable green status LED.
-
-if LED_STATUS_GREEN_ENABLE
-
-config LED_STATUS_GREEN
-       int "Green LED identification"
-       help
-         Valid enabled LED device number (0-5).
-
-endif # LED_STATUS_GREEN_ENABLE
-
 config LED_STATUS_CMD
        bool "Enable status LED commands"
 

diff --git a/drivers/misc/gpio_led.c b/drivers/misc/gpio_led.c
index e806b1f241f717c1ce0166813f463102e3b0271b..a1432a53440e0a1754e6b436a67de3c8fbd869e4 100644 (file)
--- a/drivers/misc/gpio_led.c
+++ b/drivers/misc/gpio_led.c
@@ -67,16 +67,4 @@ void red_led_off(void)
 }
 #endif
 
-#ifdef CONFIG_LED_STATUS_GREEN
-void green_led_on(void)
-{
-       __led_set(CONFIG_LED_STATUS_GREEN, CONFIG_LED_STATUS_ON);
-}
-
-void green_led_off(void)
-{
-       __led_set(CONFIG_LED_STATUS_GREEN, CONFIG_LED_STATUS_OFF);
-}
-#endif
-
 #endif /* CONFIG_GPIO_LED_STUBS */

diff --git a/include/status_led.h b/include/status_led.h
index 85e583f18c8b9bfbd026b0b20419f9ccd0d91117..8e8b19f8c190db242972a59e30f099fa4ad0d0d0 100644 (file)
--- a/include/status_led.h
+++ b/include/status_led.h
@@ -72,13 +72,9 @@ static inline void status_led_boot_blink(void) { }
 #ifndef        __ASSEMBLY__
 void red_led_on(void);
 void red_led_off(void);
-void green_led_on(void);
-void green_led_off(void);
 #else
        .extern red_led_on
        .extern red_led_off
-       .extern green_led_on
-       .extern green_led_off
 #endif
 
 #endif /* _STATUS_LED_H_       */