[U-Boot] [PATCH] VGA text console support
Wolfgang Denk
wd at denx.de
Sun Apr 7 19:04:17 CEST 2013
Dear Vladimir 'φ-coder/phcoder' Serbinenko,
I cannot comment on the actual technical content of your patches, but
here are a few formal comments; I highly recommend to study
http://www.denx.de/wiki/U-Boot/Patches
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: quoted-printable
Please send plain text messages. Do not send multipart/mixed
messages.
> Most coreboot users use just VGA text console, not graphics. Support it.
> Remaining problem is that software and hardware cursor blinking superimpose.
> I disabled software blinking locally but it's not part of this patch since
> it's not done properly.
>
> diff --git a/drivers/video/cfb_console.c b/drivers/video/cfb_console.c
> index 61e1058..0c3c26d 100644
Signed-off-by: line missing.
> +static int VIDEO_VISIBLE_COLS;
> +static int VIDEO_VISIBLE_ROWS;
> +static int VIDEO_PIXEL_SIZE;
Illegal variable names - these must always be lower case.
Also, you are meddling here with a global file. Are you sure your
changes will have no bad effects to other use cases?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Don't tell me how hard you work. Tell me how much you get done.
-- James J. Ling
More information about the U-Boot
mailing list