[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