[U-Boot-Users] [PATCH] Total5100 and Total5200 platform support 
    Wolfgang Denk 
    wd at denx.de
       
    Mon Jul  5 15:22:18 CEST 2004
    
    
  
In message <40E95290.3040607 at freescale.com> you wrote:
>
> the attached patch adds support for the Total5100's and Total5200's LCD 
> screen. By default only the logo is displayed because the environment 
> variable "console" is set to "serial". By clearing this variable stdout 
> and stderr will be displayed on the LCD. The patch should be applied 
> after the initial (20040701) and the second patch (20040702).
Can you please fix this patch to  use  the  standard  console  device
mechanism of U-Boot? Instead of introducing a new "console" variable.
the  standard  mechanism  of assigning to the stdin / stdout / stderr
streams should be used.
I just learned that there are already other bad examples using such a
"console" setting:
File:				Board:		Maintainer:
board/eltec/elppc/misc.c	ELPPC		Frank Gottschling
board/netphone/phone_console.c	NETPHONE	Pantelis Antoniou
drivers/cfb_console.c		?		Frank Gottschling
Frank & Pantelis, can you please submit patches  to  clean  this  up,
too? Let's get rid of this "console" stuff before anybody else copies
it. [Sorry that this escaped me so far.]
Best regards,
Wolfgang Denk
-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
"What people have been reduced to are  mere  3-D  representations  of
their own data."                                      - Arthur Miller
    
    
More information about the U-Boot
mailing list