Messages in this thread Patch in this message | | | Date | Wed, 27 Feb 2002 00:26:02 +0500 | From | Denis Zaitsev <> | Subject | [PATCH] matroxfb_base.c - a little fix |
| |
It was two identical if-else branches in matroxfb_decode_var - one for var->bits_per_pixel == 4 and the other for var->bits_per_pixel <= 8 . So, I've removed 4's one. It should be ok, if this branch was ok - i.e. was intentionally made the same as the 8bpp's. So, Petr, please, apply this fix, if all ok.
--- drivers/video/matrox/matroxfb_base.c.orig Mon Feb 25 21:38:33 2002 +++ drivers/video/matrox/matroxfb_base.c Tue Feb 26 16:38:33 2002 @@ -494,16 +494,6 @@ static int matroxfb_decode_var(CPMINFO s var->transp.offset = 0; var->transp.length = 0; *visual = MX_VISUAL_PSEUDOCOLOR; - } else if (var->bits_per_pixel == 4) { - var->red.offset = 0; - var->red.length = 8; - var->green.offset = 0; - var->green.length = 8; - var->blue.offset = 0; - var->blue.length = 8; - var->transp.offset = 0; - var->transp.length = 0; - *visual = MX_VISUAL_PSEUDOCOLOR; } else if (var->bits_per_pixel <= 8) { var->red.offset = 0; var->red.length = 8; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |