[U-Boot] [PATCH 2/2] support blackfin board initialization in generic board_f

[Tom Rini]

On Thu, Jul 24, 2014 at 10:47:02AM +0800, Sonic Zhang wrote:
> From: Sonic Zhang <sonic.zhang at analog.com>
> 
> - init hardware watchdog if applicable
> - use CONFIG_SYS_MONITOR_LEN as the gd monitor len for Blackfin
> - reserve u-boot memory at the top field of the RAM for Blackfin
> - avoid refer to CONFIG_SYS_MONITOR_LEN, which is not defined by Blackfin
> 
> Signed-off-by: Sonic Zhang <sonic.zhang at analog.com>
> ---
> 
>  common/board_f.c | 19 ++++++++++++++++++-
>  1 file changed, 18 insertions(+), 1 deletion(-)
> 
> diff --git a/common/board_f.c b/common/board_f.c
> index bdab38e..079e1fa 100644
> --- a/common/board_f.c
> +++ b/common/board_f.c
> @@ -107,6 +107,9 @@ __weak void blue_led_off(void) {}
>  #if defined(CONFIG_WATCHDOG)
>  static int init_func_watchdog_init(void)
>  {
> +#ifdef CONFIG_HW_WATCHDOG
> +	hw_watchdog_init();
> +#endif
>  	puts("       Watchdog enabled\n");
>  	WATCHDOG_RESET();
>  

So today we're inconsistent about hw_watchdog_init.  For ARM this is
done at the board level, for blackfin/m68k as you've noted it's done in
board_init_f, which makes sense I believe.  But we need to get folks to
test the conversion over, so for now make this CONFIG_HW_WATCHDOG &&
(BLACKFIN || M68K || MICROBLAZE || SH) and we'll get everyone converted
when we can.

> @@ -144,7 +147,11 @@ static int display_text_info(void)
>  	bss_end = (ulong)&__bss_end;
>  
>  	debug("U-Boot code: %08X -> %08lX  BSS: -> %08lX\n",
> +#ifdef CONFIG_SYS_TEXT_BASE
>  	      CONFIG_SYS_TEXT_BASE, bss_start, bss_end);
> +#else
> +	      CONFIG_SYS_MONITOR_BASE, bss_start, bss_end);
> +#endif
>  #endif

These aren't the same for you?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140729/36ec0a09/attachment.pgp>