[Mesa-dev] [PATCH] intel: Do not expose OES_compressed_ETC1_RGB8_texture or ARB_texture_rgb10_a2ui pre-GEN4

[Anuj Phogat]

On Fri, Feb 8, 2013 at 6:05 PM, Ian Romanick <idr at freedesktop.org> wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
>
> Older hardware cannot do ARB_texture_rgb10_a2ui, and the translation
> code for OES_compressed_ETC1_RGB8_texture was never implemented in the
> i915 driver.
>
> NOTE: This is a candidate for all stable branches.
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
> ---
>  src/mesa/drivers/dri/intel/intel_extensions.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/intel/intel_extensions.c b/src/mesa/drivers/dri/intel/intel_extensions.c
> index ee4f0ae..bf5e2b5 100755
> --- a/src/mesa/drivers/dri/intel/intel_extensions.c
> +++ b/src/mesa/drivers/dri/intel/intel_extensions.c
> @@ -85,8 +85,6 @@ intelInitExtensions(struct gl_context *ctx)
>     ctx->Extensions.TDFX_texture_compression_FXT1 = true;
>     ctx->Extensions.OES_EGL_image = true;
>     ctx->Extensions.OES_draw_texture = true;
> -   ctx->Extensions.OES_compressed_ETC1_RGB8_texture = true;
> -   ctx->Extensions.ARB_texture_rgb10_a2ui = true;
>
>     if (intel->gen >= 6)
>        ctx->Const.GLSLVersion = 140;
> @@ -144,6 +142,7 @@ intelInitExtensions(struct gl_context *ctx)
>        ctx->Extensions.EXT_packed_float = true;
>        ctx->Extensions.ARB_texture_compression_rgtc = true;
>        ctx->Extensions.ARB_texture_rg = true;
> +      ctx->Extensions.ARB_texture_rgb10_a2ui = true;
>        ctx->Extensions.ARB_vertex_type_2_10_10_10_rev = true;
>        ctx->Extensions.EXT_draw_buffers2 = true;
>        ctx->Extensions.EXT_framebuffer_sRGB = true;
> @@ -157,6 +156,7 @@ intelInitExtensions(struct gl_context *ctx)
>        ctx->Extensions.ATI_envmap_bumpmap = true;
>        ctx->Extensions.MESA_texture_array = true;
>        ctx->Extensions.NV_conditional_render = true;
> +      ctx->Extensions.OES_compressed_ETC1_RGB8_texture = true;
>        ctx->Extensions.OES_standard_derivatives = true;
>     }
>
> --
> 1.7.11.7
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

LGTM.
Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>