[U-Boot] [PATCH] am33xx, spl, siemens: enable debug uart output again
Simon Glass
sjg at chromium.org
Tue Feb 24 14:31:41 CET 2015
Hi Heiko,
On 23 February 2015 at 23:18, Heiko Schocher <hs at denx.de> wrote:
> a6b541b090: TI ARMv7: Don't use GD before crt0.S has set it
>
> moves the init of the debug uart at the very end of SPL code.
> Enable it for the siemens board earlier, as they print
> ddr settings ... all debug output before board_init_r()
> is here currently useless. Maybe we must rework this
> globally?
Assuming we are talking about U-Boot proper, the DDR init should
happen in board_init_f(), specifically dram_init(). so I think this
code should be updated.
If it is SPL, then DDR init should happen in SPL's board_init_f().
I sent a series a few weeks ago (available at u-boot-dm branch
spl-working) related to this topic:
http://patchwork.ozlabs.org/patch/438581/
>
> Signed-off-by: Heiko Schocher <hs at denx.de>
> ---
>
> board/siemens/common/board.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/board/siemens/common/board.c b/board/siemens/common/board.c
> index cc0ac6b..fb2de48 100644
> --- a/board/siemens/common/board.c
> +++ b/board/siemens/common/board.c
> @@ -43,6 +43,11 @@ void set_mux_conf_regs(void)
> /* Initalize the board header */
> enable_i2c0_pin_mux();
> i2c_set_bus_num(0);
> +
> + /* enable early the console */
> + gd->baudrate = CONFIG_BAUDRATE;
> + serial_init();
> + gd->have_console = 1;
> if (read_eeprom() < 0)
> puts("Could not get board ID.\n");
>
> --
> 2.1.0
>
Regards,
Simon
More information about the U-Boot
mailing list