[U-Boot-Users] [PATCH] Total5100 and Total5200 platform support

[Pantelis Antoniou]

Wolfgang Denk wrote:

>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
>
>
Excuse me Wolfgang but what is the problem?

In my board I don't use an extra console variable. I
just wait for a key sequence to enable the lcd console
which does use the standard u-boot mechanism.

Have I missed something here?

Regards

Pantelis