cell: be more conservative in cell_is_format_supported()
authorBrian Paul <brianp@vmware.com>
Wed, 17 Mar 2010 22:33:29 +0000 (16:33 -0600)
committerBrian Paul <brianp@vmware.com>
Wed, 17 Mar 2010 22:35:29 +0000 (16:35 -0600)
This fixes a regression from commit a84575cdc0c8193b2c7858734e2ec6b1ec4511b2
which changed the depth/stencil format we were trying to use.

src/gallium/drivers/cell/ppu/cell_screen.c

index a43f8638dcdc71daa5e7dcb7f2ef0bd6a37ae325..eada62181af95b96fb03c56b78234beb5935f1e8 100644 (file)
@@ -134,12 +134,16 @@ cell_is_format_supported( struct pipe_screen *screen,
                           unsigned tex_usage, 
                           unsigned geom_flags )
 {
-   /* cell supports most formats, XXX for now anyway */
-   if (format == PIPE_FORMAT_DXT5_RGBA ||
-       format == PIPE_FORMAT_A8B8G8R8_SRGB)
-      return FALSE;
-   else
+   /* only a few formats are known to work at this time */
+   switch (format) {
+   case PIPE_FORMAT_Z24S8_UNORM:
+   case PIPE_FORMAT_Z24X8_UNORM:
+   case PIPE_FORMAT_B8G8R8A8_UNORM:
+   case PIPE_FORMAT_I8_UNORM:
       return TRUE;
+   default:
+      return FALSE;
+   }
 }