On Saturday 14 January 2012 20:57:25 Simon Glass wrote:
> On Sat, Jan 14, 2012 at 5:42 PM, Mike Frysinger wrote:
> > On Saturday 14 January 2012 19:47:24 Simon Glass wrote:
> >> --- a/common/cmd_echo.c
> >> +++ b/common/cmd_echo.c
> >> @@ -44,8 +44,9 @@ int do_echo(cmd_tbl_t *cmdtp, int flag, int argc, char
> >> * const argv[]) }
> >>       }
> >> 
> >> +     /* Use puts() so that the LCD sees it as a new line */
> >>       if (putnl)
> >> -             putc('\n');
> >> +             puts("\n");
> >> 
> >>       return 0;
> >>  }
> > 
> > this sounds like a hack for a broken LCD core and so the fix should be
> > somewhere in there
> 
> Well it's actually that I don't want to flush after every character
> (it slows things down a lot)

right, but ...

> so puts is a good place to look

most puts() that i've seen are simple loops that call putc()

> I suppose we could flush after every newline we see in lcd_putc().

that would be the right way to go i think
-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