From mboxrd@z Thu Jan  1 00:00:00 1970
From: Pierre AUBERT <p.aubert@staubli.com>
Date: Mon, 15 Mar 2004 10:02:01 +0100
Subject: [U-Boot-Users] PATCH - add CMD_BMP and SPLASH_SCREEN support
 in cfb_console
In-Reply-To: <20040314165811.3C1A6C0655@atlas.denx.de>
References: <20040314165811.3C1A6C0655@atlas.denx.de>
Message-ID: <40557109.1070509@staubli.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Wolfgang

Wolfgang Denk wrote:

>In message <40509BDC.6060506@staubli.com> you wrote:
>  
>
>>The attached patch add bitmap (8bpp and 24bpp) support on cfb_console.
>>    
>>
>
>It also does some other things which you  don't  bother  to  mention
>here.
>
>Please ALWAYS describe ALL extensions to the code.
>
>  
>
>>Be carefull: this patch moves the environment location in flash for the 
>>Lite5200 board.
>>U-boot room became too small ...
>>    
>>
>
>Thanks for the warning, but actually I would like to ask you to SPLIT
>such modifications into separate, independend patches.
>
>
>Also, it is not exactly nice to enable  non-standard  features  (like
>CONFIG_VIDEO   or  CONFIG_CFB_CONSOLE,  CFG_CMD_BMP)  while  deleting
>  
>
Sorry, I've enabled this config for testing purpose and I've forgotten
to remove it.

>others   (CONFIG_DOS_PARTITION)   in   public   configuration   files
>(IceCube.h).
>  
>
I've deleted CONFIG_DOS_PARTITION because it's defined twice in 
Icecube.h !!!
(and it's always defined twice in the CVS ...)

>
>  
>
>Best regards,
>
>Wolfgang Denk
>
>  
>
Best regards