dri_util: Remove support for RGB332 framebuffers
authorIan Romanick <ian.d.romanick@intel.com>
Thu, 12 Jul 2012 18:42:54 +0000 (11:42 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Mon, 29 Oct 2012 16:55:32 +0000 (09:55 -0700)
None of the remaining DRI drivers in Mesa use this.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/common/utils.c

index 6d8cb4e29a641782f699a8aefd0f5c2d86d6a02f..1b337824f503e8ed3497cdbd35550d4ed12ea79a 100644 (file)
@@ -194,15 +194,12 @@ driCreateConfigs(GLenum fb_format, GLenum fb_type,
 {
    static const uint8_t bits_table[4][4] = {
      /* R  G  B  A */
-      { 3, 3, 2, 0 }, /* Any GL_UNSIGNED_BYTE_3_3_2 */
       { 5, 6, 5, 0 }, /* Any GL_UNSIGNED_SHORT_5_6_5 */
       { 8, 8, 8, 0 }, /* Any RGB with any GL_UNSIGNED_INT_8_8_8_8 */
       { 8, 8, 8, 8 }  /* Any RGBA with any GL_UNSIGNED_INT_8_8_8_8 */
    };
 
    static const uint32_t masks_table_rgb[6][4] = {
-      { 0x000000E0, 0x0000001C, 0x00000003, 0x00000000 }, /* 3_3_2       */
-      { 0x00000007, 0x00000038, 0x000000C0, 0x00000000 }, /* 2_3_3_REV   */
       { 0x0000F800, 0x000007E0, 0x0000001F, 0x00000000 }, /* 5_6_5       */
       { 0x0000001F, 0x000007E0, 0x0000F800, 0x00000000 }, /* 5_6_5_REV   */
       { 0xFF000000, 0x00FF0000, 0x0000FF00, 0x00000000 }, /* 8_8_8_8     */
@@ -210,8 +207,6 @@ driCreateConfigs(GLenum fb_format, GLenum fb_type,
    };
 
    static const uint32_t masks_table_rgba[6][4] = {
-      { 0x000000E0, 0x0000001C, 0x00000003, 0x00000000 }, /* 3_3_2       */
-      { 0x00000007, 0x00000038, 0x000000C0, 0x00000000 }, /* 2_3_3_REV   */
       { 0x0000F800, 0x000007E0, 0x0000001F, 0x00000000 }, /* 5_6_5       */
       { 0x0000001F, 0x000007E0, 0x0000F800, 0x00000000 }, /* 5_6_5_REV   */
       { 0xFF000000, 0x00FF0000, 0x0000FF00, 0x000000FF }, /* 8_8_8_8     */
@@ -219,8 +214,6 @@ driCreateConfigs(GLenum fb_format, GLenum fb_type,
    };
 
    static const uint32_t masks_table_bgr[6][4] = {
-      { 0x00000007, 0x00000038, 0x000000C0, 0x00000000 }, /* 3_3_2       */
-      { 0x000000E0, 0x0000001C, 0x00000003, 0x00000000 }, /* 2_3_3_REV   */
       { 0x0000001F, 0x000007E0, 0x0000F800, 0x00000000 }, /* 5_6_5       */
       { 0x0000F800, 0x000007E0, 0x0000001F, 0x00000000 }, /* 5_6_5_REV   */
       { 0x0000FF00, 0x00FF0000, 0xFF000000, 0x00000000 }, /* 8_8_8_8     */
@@ -228,8 +221,6 @@ driCreateConfigs(GLenum fb_format, GLenum fb_type,
    };
 
    static const uint32_t masks_table_bgra[6][4] = {
-      { 0x00000007, 0x00000038, 0x000000C0, 0x00000000 }, /* 3_3_2       */
-      { 0x000000E0, 0x0000001C, 0x00000003, 0x00000000 }, /* 2_3_3_REV   */
       { 0x0000001F, 0x000007E0, 0x0000F800, 0x00000000 }, /* 5_6_5       */
       { 0x0000F800, 0x000007E0, 0x0000001F, 0x00000000 }, /* 5_6_5_REV   */
       { 0x0000FF00, 0x00FF0000, 0xFF000000, 0x000000FF }, /* 8_8_8_8     */
@@ -237,8 +228,6 @@ driCreateConfigs(GLenum fb_format, GLenum fb_type,
    };
 
    static const uint8_t bytes_per_pixel[6] = {
-      1, /* 3_3_2       */
-      1, /* 2_3_3_REV   */
       2, /* 5_6_5       */
       2, /* 5_6_5_REV   */
       4, /* 8_8_8_8     */
@@ -255,23 +244,17 @@ driCreateConfigs(GLenum fb_format, GLenum fb_type,
    unsigned num_accum_bits = (enable_accum) ? 2 : 1;
 
    switch ( fb_type ) {
-      case GL_UNSIGNED_BYTE_3_3_2:
-        index = 0;
-        break;
-      case GL_UNSIGNED_BYTE_2_3_3_REV:
-        index = 1;
-        break;
       case GL_UNSIGNED_SHORT_5_6_5:
-        index = 2;
+        index = 0;
         break;
       case GL_UNSIGNED_SHORT_5_6_5_REV:
-        index = 3;
+        index = 1;
         break;
       case GL_UNSIGNED_INT_8_8_8_8:
-        index = 4;
+        index = 2;
         break;
       case GL_UNSIGNED_INT_8_8_8_8_REV:
-        index = 5;
+        index = 3;
         break;
       default:
         fprintf( stderr, "[%s:%u] Unknown framebuffer type 0x%04x.\n",
@@ -310,16 +293,13 @@ driCreateConfigs(GLenum fb_format, GLenum fb_type,
    }
 
    switch ( bytes_per_pixel[ index ] ) {
-      case 1:
-        bits = bits_table[0];
-        break;
       case 2:
-        bits = bits_table[1];
+        bits = bits_table[0];
         break;
       default:
         bits = ((fb_format == GL_RGB) || (fb_format == GL_BGR))
-           ? bits_table[2]
-           : bits_table[3];
+           ? bits_table[1]
+           : bits_table[2];
         break;
    }