On Saturday, September 17, 2011 12:48:56 Simon Glass wrote:
> --- /dev/null
> +++ b/include/configs/sandbox.h

missing #ifdef multiple include protection

> +#define CONFIG_NR_DRAM_BANKS 1
> +#define CONFIG_SYS_SDRAM_BASE        0
> +#define CONFIG_DRAM_SIZE     (128 << 20)

do you need any of this ?

> +#define      CONFIG_SYS_PROMPT               "=>"    /* Command Prompt */

this file has a bunch of "#define<tab>" where it should be "#define<space>"

> +#define CONFIG_LMB

i dont think you need this either ?
-mike

Attachment: 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

Reply via email to