radeonsi: assert the CB format is valid (v2)
authorAlex Deucher <alexander.deucher@amd.com>
Thu, 15 Nov 2012 14:34:13 +0000 (09:34 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 15 Nov 2012 17:10:48 +0000 (12:10 -0500)
Assert the the CB format is valid and default to
the INVALID hw format rather than ~0U when the format
doesn't match for non-debug builds.

v2: use INVALID hw format rather than ~0U

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
src/gallium/drivers/radeonsi/si_state.c

index fbc636dece4fa648ec1cb07988b1b19dea1a3700..31a55a22006ab449e94169dc76455090a4d7cc3f 100644 (file)
@@ -831,7 +831,7 @@ static uint32_t si_translate_colorformat(enum pipe_format format)
        case PIPE_FORMAT_R4A4_UNORM:
        case PIPE_FORMAT_A4R4_UNORM:
        default:
-               return ~0U; /* Unsupported. */
+               return V_028C70_COLOR_INVALID; /* Unsupported. */
        }
 }
 
@@ -1432,7 +1432,7 @@ static bool si_is_vertex_format_supported(struct pipe_screen *screen, enum pipe_
 
 static bool si_is_colorbuffer_format_supported(enum pipe_format format)
 {
-       return si_translate_colorformat(format) != ~0U &&
+       return si_translate_colorformat(format) != V_028C70_COLOR_INVALID &&
                si_translate_colorswap(format) != ~0U;
 }
 
@@ -1606,6 +1606,7 @@ static void si_cb(struct r600_context *rctx, struct si_pm4_state *pm4,
        }
 
        format = si_translate_colorformat(surf->base.format);
+       assert(format != V_028C70_COLOR_INVALID);
        swap = si_translate_colorswap(surf->base.format);
        if (rtex->resource.b.b.usage == PIPE_USAGE_STAGING) {
                endian = V_028C70_ENDIAN_NONE;