From mboxrd@z Thu Jan  1 00:00:00 1970
From: Anatolij Gustschin <agust@denx.de>
Date: Sat, 1 May 2010 20:51:33 +0200
Subject: [U-Boot] [PATCH 2/3] video: add support for display controller
 in MB86R0x SoCs
In-Reply-To: <4BD0493B.2090706@arcor.de>
References: <1271932257-14618-1-git-send-email-weisserm@arcor.de>
	<1271932257-14618-2-git-send-email-weisserm@arcor.de>
	<1271932257-14618-3-git-send-email-weisserm@arcor.de>
	<20100422124157.A9EE9C36DD0@gemini.denx.de>
	<4BD0493B.2090706@arcor.de>
Message-ID: <20100501205133.6776f306@wker>
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

Dear Matthias,

On Thu, 22 Apr 2010 15:03:55 +0200
Matthias Wei?er <weisserm@arcor.de> wrote:
...
> >> +/*
> >> + * Set a RGB color in the LUT
> >> + */
> >> +void video_set_lut(unsigned int index, unsigned char r,
> >> +					unsigned char g, unsigned char b)
> >> +{
> >> +
> >> +}
> >
> > Code seems to be missing?
> 
> The driver doesn't support palletized color format at the moment but 
> removing this function leads to a linker error.
> 
> Maybe we should add a config option to disable palletized color format 
> or add a weak function somewhere. Maybe Anatolij can comment on this 
> issue also.

I will send a patch adding weak default video_set_lut() to cfb_console
driver. You can remove empty function then.

Best regards,
Anatolij