On Thursday 17 December 2009 16:32:38 Wolfgang Denk wrote:
> Mike Frysinger wrote:
> > Signed-off-by: Mike Frysinger <vap...@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
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
