From mboxrd@z Thu Jan  1 00:00:00 1970
From: Anatolij Gustschin <agust@denx.de>
Date: Wed, 04 Feb 2009 23:46:56 +0100
Subject: [U-Boot] [PATCH 6/9] Add 16bpp BMP support
In-Reply-To: <Pine.LNX.4.64.0902041743530.6279@axis700.grange>
References: <Pine.LNX.4.64.0902041532100.6279@axis700.grange>
	<Pine.LNX.4.64.0902041743530.6279@axis700.grange>
Message-ID: <498A1AE0.1070801@denx.de>
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

Hello Guennadi,

>  #if defined(CONFIG_PXA250) || defined(CONFIG_ATMEL_LCD)
> -			*(fb++) = *(bmap++);
> +				*(fb++) = *(bmap++);
>  #elif defined(CONFIG_MPC823) || defined(CONFIG_MCC200)
> -			*(fb++)=255-*(bmap++);
> +				*(fb++)=255-*(bmap++);

please add space around '=' and '-' here.

> +#if defined(CONFIG_ATMEL_LCD_BGR555)
> +				*(fb++) = ((bmap[0] & 0x1f) << 2) | (bmap[1] & 0x03);
> +				*(fb++) = (bmap[0] & 0xe0) | ((bmap[1] & 0x7c) >> 2);

both lines above to long (> 80), please fix.

Thanks,
Anatolij