[U-Boot] [PATCH v3 1/2] video: atmel: implement lcd_setcolreg funtion

[Marek Vasut]

Dear Bo Shen,

> Hi Marek,
> 
> On 11/7/2012 6:54, Marek Vasut wrote:
> > Dear Andreas Bießmann,
> > 
> >> From: Bo Shen <voice.shen at atmel.com>
> > 
> > Missing commit message
> > 
> >> Signed-off-by: Bo Shen <voice.shen at atmel.com>
> >> Signed-off-by: Andreas Bießmann <andreas.devel at googlemail.com>
> >> ---
> >> 
> >> since v2:
> >>   * add this single patch
> >>   
> >>   drivers/video/atmel_hlcdfb.c |    6 ++++++
> >>   1 file changed, 6 insertions(+)
> >> 
> >> diff --git a/drivers/video/atmel_hlcdfb.c b/drivers/video/atmel_hlcdfb.c
> >> index beb7fa3..4110d4d 100644
> >> --- a/drivers/video/atmel_hlcdfb.c
> >> +++ b/drivers/video/atmel_hlcdfb.c
> >> @@ -51,6 +51,12 @@ short console_row;
> >> 
> >>   #define lcdc_readl(reg)		__raw_readl((reg))
> >>   #define lcdc_writel(reg, val)	__raw_writel((val), (reg))
> >> 
> >> +void lcd_setcolreg(ushort regno, ushort red, ushort green, ushort blue)
> >> +{
> >> +	lcdc_writel((red << 16) | (green << 8) | blue,
> >> +			panel_info.mmio + ATMEL_LCDC_LUT(regno));
> > 
> > So this is RGB666? Or what are those magic numbers ?
> 
> This is a little different with the driver of atmel_lcdfb.c.
> The register for LUT is layout as following:
> RCLUT (24 ~ 16), GCLUT (15 ~ 8) and BCLUT (7 ~ 0).
> So, use those magic numbers.

Good, can you define those magic offsets then please?

> More information, you can get from [1] on page 1163.
> 
> 1. http://www.atmel.com/Images/doc11053.pdf

[...]

Best regards,
Marek Vasut