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

[Petr Vandrovec]

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
                                                    [e-mail]