intel: Remove duplicate test for texture attachment completeness.
authorEric Anholt <eric@anholt.net>
Tue, 15 Nov 2011 20:49:25 +0000 (12:49 -0800)
committerEric Anholt <eric@anholt.net>
Tue, 22 Nov 2011 21:58:38 +0000 (13:58 -0800)
We are already testing this if appropriate in
intel_validate_framebuffer (FBO completeness), so no need to avoid
attaching the texture to the renderbuffer here.

This causes MESA_FORMAT_R11_G11_B10_FLOAT to now be renderable as a texture
attachment on i965.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/intel/intel_fbo.c

index 348b927..9005402 100644 (file)
@@ -981,14 +981,6 @@ intel_renderbuffer_update_wrapper(struct intel_context *intel,
    struct gl_renderbuffer *rb = &irb->Base;
 
    rb->Format = format;
-   if (!intel_span_supports_format(rb->Format)) {
-      DBG("Render to texture BAD FORMAT %s\n",
-         _mesa_get_format_name(rb->Format));
-      return false;
-   } else {
-      DBG("Render to texture %s\n", _mesa_get_format_name(rb->Format));
-   }
-
    rb->InternalFormat = internal_format;
    rb->DataType = intel_mesa_format_to_rb_datatype(rb->Format);
    rb->_BaseFormat = _mesa_get_format_base_format(rb->Format);