[U-Boot] [PATCH 4/4] MX51: Add video support to vision2 board

[stefano babic <sbabic@denx.de>]

Am 12.10.2010 13:37, schrieb Wolfgang Denk:
> Dear Stefano Babic,
> 
> In message <1286875504-9058-5-git-send-email-sbabic@denx.de> you wrote:
>> The patch adds support for LCD to the vision board.
> ...
> 
>> +#define CONFIG_PREBOOT
> 
> Normally one defines a default command here ?

Probably I misunderstand its usage, but my intention is to enable the
PREBOOT feature, but let the user to decide with the "preboot" variable
if it is required or not. In my case for this board, I use it to load an
image from a NAND device, as "splashimage" can be used only with an address.

In common/main.c:

#ifdef CONFIG_PREBOOT
        if ((p = getenv ("preboot")) != NULL) {

So I need to set this config even without a value if I would like to use
it later setting the "preboot" variable.

> 
>> +#define CONFIG_BMP_16BPP
>> +#define CMD_DISPLAY
> 
> CONFIG_CMD_DISPLAY ???

This is wrong ! Thanks, I forget to remove it.

Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================