Hi Marek,
On Thu, 26 Sept 2024 at 23:40, Marek Vasut marex@denx.de wrote:
On 9/26/24 10:44 PM, Simon Glass wrote:
This is not used anymore, so drop it.
Signed-off-by: Simon Glass sjg@chromium.org
board/olimex/mx23_olinuxino/mx23_olinuxino.c | 7 ------- 1 file changed, 7 deletions(-)
diff --git a/board/olimex/mx23_olinuxino/mx23_olinuxino.c b/board/olimex/mx23_olinuxino/mx23_olinuxino.c index b2bb6678c23..78136c1620a 100644 --- a/board/olimex/mx23_olinuxino/mx23_olinuxino.c +++ b/board/olimex/mx23_olinuxino/mx23_olinuxino.c @@ -13,9 +13,6 @@ #include <asm/arch/imx-regs.h> #include <asm/arch/clock.h> #include <asm/arch/sys_proto.h> -#ifdef CONFIG_LED_STATUS -#include <status_led.h> -#endif #include <linux/delay.h>
DECLARE_GLOBAL_DATA_PTR; @@ -61,9 +58,5 @@ int board_init(void) /* Adress of boot parameters */ gd->bd->bi_boot_params = PHYS_SDRAM_1 + 0x100;
-#if defined(CONFIG_LED_STATUS) && defined(CONFIG_LED_STATUS_BOOT_ENABLE)
status_led_set(CONFIG_LED_STATUS_BOOT, CONFIG_LED_STATUS_STATE);-#endif
Why not define the GPIO LED in DT, it might even already be defined there.
Yes I see it in the DT for quite a few boards, so that's what we should use.
Regards, Simon