On 05/15/2012 03:49 PM, Kenneth Graunke wrote: > We should just set the bits of functionality that we support; the > GL/ES1/ES2 flags in extensions.c will take care of advertising the > appropriate extensions for the current API. > > This enables the GL_EXT_texture_compression_dxt1 extension on ES1/ES2 > when libtxc_dxtn is installed or the force_s3tc driconf option is set. > The main extension code set this up properly, but the ES-specific code > failed to do so. > > Otherwise, the extension strings reported by es1_info, es2_info, and > glxinfo all remain the same. > > This patch manually disables the ARB_framebuffer_object bit on ES > to preserve the behavior of 1c0f5d8324c4db2720247989ddc4a45315b55a85. > > Cc: Daniel Charles <daniel.char...@intel.com> > Signed-off-by: Kenneth Graunke <kenn...@whitecape.org> > --- > src/mesa/drivers/dri/i915/Makefile.sources | 1 - > src/mesa/drivers/dri/i915/intel_extensions_es.c | 1 - > src/mesa/drivers/dri/i965/Makefile.sources | 1 - > src/mesa/drivers/dri/i965/intel_extensions_es.c | 1 - > src/mesa/drivers/dri/intel/intel_context.c | 12 +- > src/mesa/drivers/dri/intel/intel_extensions.c | 7 +- > src/mesa/drivers/dri/intel/intel_extensions_es.c | 132 > ---------------------- > 7 files changed, 7 insertions(+), 148 deletions(-) > delete mode 120000 src/mesa/drivers/dri/i915/intel_extensions_es.c > delete mode 120000 src/mesa/drivers/dri/i965/intel_extensions_es.c > delete mode 100644 src/mesa/drivers/dri/intel/intel_extensions_es.c
The code looks good to me. Reviewed-by: Chad Versace <chad.vers...@linux.intel.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
