Dear Heiko Schocher,
In message <1299591018-8944-2-git-send-email...@denx.de> you wrote:
> - This patch reworks all headerfiles for keymile boards (coge, supx4,
> eter1, suen3).
> Furthermore, a refactoring on the whole environment variables has been
> acomplished.
Two independent changes => Please submit as two separate patches.
...
> +#define CONFIG_HUSH_INIT_VAR 1
...
> +#define CONFIG_I2C_MULTI_BUS 1
...
> +#define CONFIG_I2C_MUX 1
Defines like these that select functions only should not be assigned
any numeric (or other value). Please omit thse. Please fix globally.
> +#define CONFIG_KM_DEF_ENV_CONSTANTS \
...
> + "default=" \
> + "setenv default \'run newenv; reset\' && " \
These backslashes are redundant at best. Why not drop them?
...
> + "u-boot=" xstr(CONFIG_HOSTNAME) "/u-boot.bin \0" \
Are you sure you really want that trailing blank? [Please check
globally.]
...
> +#define CONFIG_SYS_MONITOR_LEN (768 << 10) /* Reserve
> 768KB for Monitor */
Line too long. Please fix globally.
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: w...@denx.de
I object to intellect without discipline; I object to power without
constructive purpose.
-- Spock, "The Squire of Gothos", stardate 2124.5
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
