From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Sun, 05 Aug 2012 12:11:35 +0200 Subject: [U-Boot] [PATCH] mx53loco: Define CONFIG_BOARD_LATE_INIT In-Reply-To: <501E220A.70801@denx.de> References: <1343762513-5574-1-git-send-email-fabio.estevam@freescale.com> <501E220A.70801@denx.de> Message-ID: <501E46D7.9050707@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 05/08/2012 09:34, Stefano Babic wrote: > On 31/07/2012 21:21, Fabio Estevam wrote: >> Define CONFIG_BOARD_LATE_INIT so that the serial console messages can be redirected to the serial port. >> >> This is needed because in mx53loco.c we have: >> >> #ifdef CONFIG_BOARD_LATE_INIT >> int board_late_init(void) >> { >> setenv("stdout", "serial"); >> >> return 0; >> } >> #endif >> >> Signed-off-by: Fabio Estevam >> --- >> include/configs/mx53loco.h | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h >> index 0a25c7d..bd23387 100644 >> --- a/include/configs/mx53loco.h >> +++ b/include/configs/mx53loco.h >> @@ -41,6 +41,7 @@ >> #define CONFIG_SYS_MALLOC_LEN (10 * 1024 * 1024) >> >> #define CONFIG_BOARD_EARLY_INIT_F >> +#define CONFIG_BOARD_LATE_INIT > > > Applied to u-boot-imx, thank. > > This is a real fix - board does not run without it. According to our > discussion in the thread, dropping setenv("stdout", "serial") in code > should be done in a separate patch. Sorry Fabio - I was too fast. I thnked about and there is a better solution. I drop your patch from u-boot-imx and I send a new on for review, fixing the same issue. Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================