From 14311ef3f2031307fb660328db14eb535da9a4fa Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Sat, 9 Jun 2012 02:33:21 -0700 Subject: [PATCH] i965: Re-enable rendering to SNORM formats. Commit d73f6375f50b fixed the cause of the Piglit failure with ARB_color_buffer_float fragment clamp modes. Now that it's fixed, there's no reason to leave snorm format rendering disabled. Reviewed-by: Eric Anholt Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c index d09fe6dcb41..26e65afabd4 100644 --- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c +++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c @@ -525,14 +525,9 @@ brw_init_surface_formats(struct brw_context *brw) * integer, so we don't need hardware support for blending on it. Other * than that, GL in general requires alpha blending for render targets, * even though we don't support it for some formats. - * - * We don't currently support rendering to SNORM textures because some of - * the ARB_color_buffer_float clamping is broken for it - * (piglit arb_color_buffer_float-drawpixels GL_RGBA8_SNORM). */ if (gen >= rinfo->render_target && - (gen >= rinfo->alpha_blend || is_integer) && - _mesa_get_format_datatype(format) != GL_SIGNED_NORMALIZED) { + (gen >= rinfo->alpha_blend || is_integer)) { brw->render_target_format[format] = render; brw->format_supported_as_render_target[format] = true; } -- 2.30.2