[U-Boot] [PATCH 2/2] mx23_olinuxino: Add support for status LED
Marek Vasut
marex at denx.de
Sun Jan 27 22:52:20 CET 2013
Dear Otavio Salvador,
> This allow user to know if the bootloader is running, even without a
> serial console.
1/2 is missing ?
> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> ---
> board/olimex/mx23_olinuxino/mx23_olinuxino.c | 7 +++++++
> board/olimex/mx23_olinuxino/spl_boot.c | 4 ++++
> include/configs/mx23_olinuxino.h | 12 ++++++++++++
> 3 files changed, 23 insertions(+)
>
> diff --git a/board/olimex/mx23_olinuxino/mx23_olinuxino.c
> b/board/olimex/mx23_olinuxino/mx23_olinuxino.c index 6a6053b..2501417
> 100644
> --- a/board/olimex/mx23_olinuxino/mx23_olinuxino.c
> +++ b/board/olimex/mx23_olinuxino/mx23_olinuxino.c
> @@ -28,6 +28,9 @@
> #include <asm/arch/imx-regs.h>
> #include <asm/arch/clock.h>
> #include <asm/arch/sys_proto.h>
> +#ifdef CONFIG_STATUS_LED
> +#include <status_led.h>
> +#endif
>
> DECLARE_GLOBAL_DATA_PTR;
>
> @@ -67,5 +70,9 @@ int board_init(void)
> /* Adress of boot parameters */
> gd->bd->bi_boot_params = PHYS_SDRAM_1 + 0x100;
>
> +#if defined(CONFIG_STATUS_LED) && defined(STATUS_LED_BOOT)
> + status_led_set(STATUS_LED_BOOT, STATUS_LED_STATE);
> +#endif
> +
> return 0;
> }
I'm missing LED value setter ... so the LED toggles only because you set the
pinmux, right ?
> diff --git a/board/olimex/mx23_olinuxino/spl_boot.c
> b/board/olimex/mx23_olinuxino/spl_boot.c index 7def8bc..ed8f8d9 100644
> --- a/board/olimex/mx23_olinuxino/spl_boot.c
> +++ b/board/olimex/mx23_olinuxino/spl_boot.c
> @@ -84,6 +84,10 @@ const iomux_cfg_t iomux_setup[] = {
> MX23_PAD_EMI_RASN__EMI_RASN | MUX_CONFIG_EMI,
> MX23_PAD_EMI_WEN__EMI_WEN | MUX_CONFIG_EMI,
>
> + /* Green LED */
> + MX23_PAD_SSP1_DETECT__GPIO_2_1 |
> + (MXS_PAD_4MA | MXS_PAD_3V3),
NOPULL ?
> /* MMC 0 */
> MX23_PAD_SSP1_CMD__SSP1_CMD | MUX_CONFIG_SSP,
> MX23_PAD_SSP1_DATA0__SSP1_DATA0 | MUX_CONFIG_SSP,
> diff --git a/include/configs/mx23_olinuxino.h
> b/include/configs/mx23_olinuxino.h index 4658aea..c57bb67 100644
> --- a/include/configs/mx23_olinuxino.h
> +++ b/include/configs/mx23_olinuxino.h
> @@ -56,6 +56,7 @@
> #define CONFIG_CMD_EXT2
> #define CONFIG_CMD_FAT
> #define CONFIG_CMD_GPIO
> +#define CONFIG_CMD_LED
> #define CONFIG_CMD_MMC
>
> /*
> @@ -112,6 +113,17 @@
> #define CONFIG_BAUDRATE 115200 /* Default baud rate */
>
> /*
> + * Status LED
> + */
> +#define CONFIG_STATUS_LED
> +#define CONFIG_GPIO_LED
> +#define CONFIG_BOARD_SPECIFIC_LED
> +#define STATUS_LED_BOOT 0
> +#define STATUS_LED_BIT 0x30a
> +#define STATUS_LED_STATE STATUS_LED_ON
> +#define STATUS_LED_PERIOD (CONFIG_SYS_HZ / 2)
Are these needed?
> +/*
> * MMC Driver
> */
> #ifdef CONFIG_CMD_MMC
Best regards,
Marek Vasut
More information about the U-Boot
mailing list