Re: missing break? linux/drivers/video/riva/fbdev.c

From: Petr Vandrovec (VANDROVE@vc.cvut.cz)
Date: Fri Sep 14 2001 - 11:13:22 EST


On 13 Sep 01 at 21:26, andrew may wrote:
> diff -u --recursive --new-file v2.4.9/linux/drivers/video/riva/fbdev.c linux/drivers/video/riva/fbdev.c
> --- v2.4.9/linux/drivers/video/riva/fbdev.c Wed Jul 25 17:10:24 2001
> +++ linux/drivers/video/riva/fbdev.c Fri Sep 7 09:28:38 2001
> @@ -1109,6 +1109,8 @@
> break;
> #endif
> #ifdef FBCON_HAS_CFB16
> + case 15:
> + rc = 15; /* fix for 15 bpp depths on Riva 128 based cards */
> case 16:
> rc = 16; /* directcolor... 16 entries SW palette */
> break; /* Mystique: truecolor, 16 entries SW palette, HW palette hardwired into 1:1 mapping */
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
I do not think that this comment should be in rivafb driver ;-) It
is completely unrelated to Riva hardware, it came from matroxfb.
                                                Best regards,
                                                    Petr Vandrovec
                                                    vandrove@vc.cvut.cz
                                                                             
-
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/



This archive was generated by hypermail 2b29 : Sat Sep 15 2001 - 21:00:47 EST