[Mesa-dev] [PATCH] i965: Stop advertising GL_MESA_ycbcr_texture.

Ian Romanick idr at freedesktop.org
Tue Apr 8 16:17:23 PDT 2014


On 04/08/2014 01:56 PM, Kenneth Graunke wrote:
> The "new" fragment shader backend has never supported the necessary
> color conversion code for this to work.  We began using the new backend
> in Mesa 7.10 for GLSL (commit a81d423d93f22a948f3aa4bf73, October 2010),
> and for ARB_fragment_program in Mesa 9.1 (commit 97615b2d8c7c3cea6fd3a4,
> August 2012).
> 
> I haven't heard any complaints, so I don't think anyone will miss this
> feature.  I believe mplayer used it at one point, but these days
> defaults to other paths anyway.
> 
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> Cc: Topi Pohjolainen <topi.pohjolainen at intel.com>
> Cc: Ian Romanick <idr at freedesktop.org>
> Cc: Eric Anholt <eric at anholt.net>

Add a Cc: mesa-stable, and you've got

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

> ---
>  src/mesa/drivers/dri/i965/intel_extensions.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c b/src/mesa/drivers/dri/i965/intel_extensions.c
> index 15fcd30..760b605 100644
> --- a/src/mesa/drivers/dri/i965/intel_extensions.c
> +++ b/src/mesa/drivers/dri/i965/intel_extensions.c
> @@ -231,7 +231,6 @@ intelInitExtensions(struct gl_context *ctx)
>     ctx->Extensions.ATI_separate_stencil = true;
>     ctx->Extensions.ATI_texture_env_combine3 = true;
>     ctx->Extensions.MESA_pack_invert = true;
> -   ctx->Extensions.MESA_ycbcr_texture = true;
>     ctx->Extensions.NV_conditional_render = true;
>     ctx->Extensions.NV_primitive_restart = true;
>     ctx->Extensions.NV_texture_env_combine4 = true;
> 



More information about the mesa-dev mailing list