Hi Marek, On Thu, 26 Sept 2024 at 23:40, Marek Vasut <[email protected]> wrote: > > On 9/26/24 10:44 PM, Simon Glass wrote: > > This is not used anymore, so drop it. > > > > Signed-off-by: Simon Glass <[email protected]> > > --- > > > > 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
