On Saturday, September 17, 2011 12:48:44 Simon Glass wrote: > These files are taken from the ARM board implementation and then reduced > to remove unneeded cr!uft.
"cr!uft" -> "cruft"
> --- /dev/null
> +++ b/arch/sandbox/lib/Makefile
>
> +LIBGCC = $(obj)libgcc.o
>
> +LGOBJS := $(addprefix $(obj),$(GLSOBJS)) \
> + $(addprefix $(obj),$(GLCOBJS))
>
> +# Build private libgcc only when asked for
> +ifdef USE_PRIVATE_LIBGCC
> +TARGETS += $(LIBGCC)
> +endif
>
> +$(LIBGCC): $(obj).depend $(LGOBJS)
> + $(call cmd_link_o_target, $(LGOBJS))
i think this is arm cruft that should get scrubbed here
> --- /dev/null
> +++ b/arch/sandbox/lib/board.c
>
> +#ifndef CONFIG_IDENT_STRING
> +#define CONFIG_IDENT_STRING ""
> +#endif
> +
> +const char version_string[] =
> + U_BOOT_VERSION" (" U_BOOT_DATE " - " U_BOOT_TIME ")"CONFIG_IDENT_STRING;
common code takes care of this for you already, so just delete it
> +static int display_banner(void)
> +{
> + printf("\n\n%s\n\n", version_string);
call display_options() instead of writing version_string yourself
> +static int display_dram_config(void)
is any of this dram logic needed ? looks like an ARM wart that should get
removed from all the sandbox code.
> +#if !defined(CONFIG_SYS_NO_FLASH)
> +static char *failed = "*** failed ***\n";
> +#endif
i dont see this getting used anywhere, so punt it
-mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
