[U-Boot] [PATCH 2/4] mx53loco: Add a comment about local print_cpuinfo method in board file

Wolfgang Denk wd at denx.de
Sun Jun 24 09:55:48 CEST 2012


Dear Otavio Salvador,

In message <1340482108-18468-3-git-send-email-otavio at ossystems.com.br> you wrote:
> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> Cc: Marek Vasut <marex at denx.de>
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Fabio Estevam <fabio.estevam at freescale.com>
> ---
>  board/freescale/mx53loco/mx53loco.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/board/freescale/mx53loco/mx53loco.c b/board/freescale/mx53loco/mx53loco.c
> index 445d022..5df1b18 100644
> --- a/board/freescale/mx53loco/mx53loco.c
> +++ b/board/freescale/mx53loco/mx53loco.c
> @@ -413,6 +413,11 @@ int board_early_init_f(void)
>  	return 0;
>  }
>  
> +/*
> + * Usually this is called too early printing a wrong information about
> + * the clock. To fix it we need to provide this local method and call
> + * it later in board_init.
> + */


I don't like this change. It messes with the initialization sequence,
which is not a good thing. Instead of shift more and more things
backwards, we should fix any incorrect clock calculations here.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A star captain's most solemn oath is that he will give his life, even
his entire crew, rather than violate the Prime Directive.
	-- Kirk, "The Omega Glory", stardate unknown


More information about the U-Boot mailing list