The ordering of the bytes in the 24 bpp RGB colour mode was being set to BGR. This now matches the values returned in the struct fb_var_screeninfo .red, .green, and .blue offsets as well as the output format of other blocs, such as the VEU Signed-off-by: Damian Hobson-Garcia <dhobsong@xxxxxxxxxx> --- drivers/video/sh_mobile_lcdcfb.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/video/sh_mobile_lcdcfb.c b/drivers/video/sh_mobile_lcdcfb.c index 04f2260..a442471 100644 --- a/drivers/video/sh_mobile_lcdcfb.c +++ b/drivers/video/sh_mobile_lcdcfb.c @@ -590,7 +590,7 @@ static int sh_mobile_lcdc_start(struct sh_mobile_lcdc_priv *priv) tmp |= 0x03; break; case 24: - tmp |= 0x0b; + tmp |= 0x02; break; case 32: break; @@ -1178,7 +1178,7 @@ static int sh_mobile_lcdc_set_bpp(struct fb_var_screeninfo *var, int bpp, var->transp.length = 0; break; - case 24: /* PKF[4:0] = 01011 - RGB 888 */ + case 24: /* PKF[4:0] = 00010 - RGB 888 */ var->red.offset = 16; var->red.length = 8; var->green.offset = 8; -- 1.7.1 -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
