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

[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.

More information, you can get from [1] on page 1163.

1. http://www.atmel.com/Images/doc11053.pdf

BRs
Bo Shen

>> +}
>> +
>>   void lcd_ctrl_init(void *lcdbase)
>>   {
>>   	unsigned long value;
>
> Best regards,
> Marek Vasut
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>