aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorVille Syrjala <syrjala@sci.fi>2008-03-04 17:28:47 -0500
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2008-03-04 19:35:13 -0500
commitfedbb3625b3c16441de1378ca00a74e9c1733289 (patch)
tree3d3eaaf4bf181d4115b0c46daefa744517c262f9 /drivers/video
parent5619d823bc0e49d05618cdf955b04055b51a7de5 (diff)
sm501fb: RGB offsets are reversed in 16bpp modes
The RGB offsets were reversed in 16bpp modes. Simply trying to reverse the offsets when endianness differs is clearly the wrong thing to do but that is an issue for another patch. Signed-off-by: Ville Syrjala <syrjala@sci.fi> Acked-by: Ben Dooks <ben-linux@fluff.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video')
-rw-r--r--drivers/video/sm501fb.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/video/sm501fb.c b/drivers/video/sm501fb.c
index 6e7810d914cc..3a98d5a86af1 100644
--- a/drivers/video/sm501fb.c
+++ b/drivers/video/sm501fb.c
@@ -286,13 +286,13 @@ static int sm501fb_check_var(struct fb_var_screeninfo *var,
286 286
287 case 16: 287 case 16:
288 if (sm->pdata->flags & SM501_FBPD_SWAP_FB_ENDIAN) { 288 if (sm->pdata->flags & SM501_FBPD_SWAP_FB_ENDIAN) {
289 var->red.offset = 11;
290 var->green.offset = 5;
291 var->blue.offset = 0;
292 } else {
293 var->blue.offset = 11; 289 var->blue.offset = 11;
294 var->green.offset = 5; 290 var->green.offset = 5;
295 var->red.offset = 0; 291 var->red.offset = 0;
292 } else {
293 var->red.offset = 11;
294 var->green.offset = 5;
295 var->blue.offset = 0;
296 } 296 }
297 297
298 var->red.length = 5; 298 var->red.length = 5;