diff options
author | Ville Syrjala <syrjala@sci.fi> | 2008-03-04 14:28:47 -0800 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-03-04 16:35:13 -0800 |
commit | fedbb3625b3c16441de1378ca00a74e9c1733289 (patch) | |
tree | 3d3eaaf4bf181d4115b0c46daefa744517c262f9 /drivers/video | |
parent | 5619d823bc0e49d05618cdf955b04055b51a7de5 (diff) | |
download | op-kernel-dev-fedbb3625b3c16441de1378ca00a74e9c1733289.zip op-kernel-dev-fedbb3625b3c16441de1378ca00a74e9c1733289.tar.gz |
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.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/video/sm501fb.c b/drivers/video/sm501fb.c index 6e7810d..3a98d5a 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, case 16: if (sm->pdata->flags & SM501_FBPD_SWAP_FB_ENDIAN) { - var->red.offset = 11; - var->green.offset = 5; - var->blue.offset = 0; - } else { var->blue.offset = 11; var->green.offset = 5; var->red.offset = 0; + } else { + var->red.offset = 11; + var->green.offset = 5; + var->blue.offset = 0; } var->red.length = 5; |