
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