On Thursday 17 December 2009 16:32:38 Wolfgang Denk wrote:
Mike Frysinger wrote:
Signed-off-by: Mike Frysinger vapier@gentoo.org
cpu/blackfin/initcode.c | 3 +++ lib_blackfin/board.c | 2 ++ 2 files changed, 5 insertions(+), 0 deletions(-)
...
static int init_baudrate(void) { +#ifdef CONFIG_BAUDRATE char baudrate[15]; int i = getenv_r("baudrate", baudrate, sizeof(baudrate)); gd->bd->bi_baudrate = gd->baudrate = (i > 0) ? simple_strtoul(baudrate, NULL, 10)
: CONFIG_BAUDRATE;+#endif return 0; }
Out of curiosity: what do these boards do when CONFIG_BAUDRATE is not defined?
the boards lack serial and/or dont use it. they're controlled via JTAG.
It seems yo me you add a new, additional meaning to the CONFIG_BAUDRATE option here. This should be documented (at least in the README).
i could just fall back to a value of 0 ... there'd be a little more useless code added, but not much to be done here seeing as how the rest of u-boot assumes baudrate is usable. -mike