[U-Boot] PATCH add rgb555 for at91

[Detlev Zundel <dzu@denx.de>]

Hi Giulio,

> Signed-off-by: giulio.benetti at micronovasrl.com

Can you please provide a somewhat more descriptive description?  It may
be obvious for you what this does (or what new features can now be
used), but it isn't for me.

>
> diff -urpN b/common/lcd.c a/common/lcd.c
> --- b/common/lcd.c      2009-06-14 21:30:39.000000000 +0200
> +++ a/common/lcd.c      2009-06-18 12:45:15.000000000 +0200
> @@ -1,4 +1,4 @@
> -/*
> +/*OC
>   * Common LCD routines for supported CPUs
>   *
>   * (C) Copyright 2001-2002
> @@ -575,6 +575,10 @@ void bitmap_plot (int x, int y)
>                         lut_entry = ((colreg & 0x000F) << 11) |
>                                     ((colreg & 0x00F0) <<  2) |
>                                     ((colreg & 0x0F00) >>  7);
> +#elif defined (CONFIG_ATMEL_LCD_RGB555)
> +                       lut_entry = ((colreg & 0x000F) << 1) |
> +                                   ((colreg & 0x00F0) <<  2) |
> +                                   ((colreg & 0x0F00) <<  3);
>  #else /* CONFIG_ATMEL_LCD_RGB565 */
>                         lut_entry = ((colreg & 0x000F) << 1) |
>                                     ((colreg & 0x00F0) << 3) |
> diff -urpN b/drivers/video/atmel_lcdfb.c a/drivers/video/atmel_lcdfb.c
> --- b/drivers/video/atmel_lcdfb.c       2009-06-14 21:30:39.000000000 +0200
> +++ a/drivers/video/atmel_lcdfb.c       2009-06-18 12:45:15.000000000 +0200
> @@ -57,6 +57,9 @@ void lcd_setcolreg(ushort regno, ushort
>  #if defined(CONFIG_ATMEL_LCD_BGR555)
                                ^^^
I know this line is not touched by you, but is BGR correct?

>         lcdc_writel(panel_info.mmio, ATMEL_LCDC_LUT(regno),
>                     (red >> 3) | ((green & 0xf8) << 2) | ((blue & 0xf8) << 7));
> +#elif defined(CONFIG_ATMEL_LCD_RGB555)
> +       lcdc_writel(panel_info.mmio, ATMEL_LCDC_LUT(regno),
> +                   (blue >> 3) | ((green & 0xf8) << 2) | ((red & 0xf8) << 7));
>  #else
>         lcdc_writel(panel_info.mmio, ATMEL_LCDC_LUT(regno),
>                     (blue >> 3) | ((green & 0xfc) << 3) | ((red & 0xf8) << 8));

Cheers
  Detlev

-- 
To you I'm an atheist; to God, I'm the Loyal Opposition.
                                        -- Woody Allen
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu@denx.de