[U-Boot] [PATCH 12/17] lcd: Add support for flushing LCD fb from dcache after update

[Simon Glass]

Hi Mike,

On Sat, Jan 14, 2012 at 6:16 PM, Mike Frysinger <vapier at gentoo.org> wrote:

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

Yes indeed, will fix.


> -mike
>

Regards,
Simon