[Mesa-dev] [PATCH v4 0/3] Add and enable extension EXT_sRGB_write_control

Gert Wollny gw.fossdev at gmail.com
Tue Oct 23 17:29:21 UTC 2018


From: Gert Wollny <gert.wollny at collabora.com>

another update to the series, 

thanks for reviewing, 
Gert 

v4: Use PIPE_FORMAT_R8G8B8A8_SRGB as key format to check support 
    for this extension.

v3: correct Gallium to only create sRGB renderbuffers based on 
    Visual.srgbCapable when on desktop GL. 

v2: 
 - core: Correct the extension table to list the extension as GLES only
 - core: drop EXTRA_API_ES3, the requirements for get operations listed
      here get OR-ed. (both Ilia Mirkin)
 - gallium: always enable the extension when sRGB is supported (Ilia Mirkin).
 - gallium: Correct handling by moving extension initialization to the
     place where gallium/st actually takes care of this. This also
     fixes properly disabling the extension via MESA_EXTENSION_OVERRIDE
 - gallium: reinstate check for desktop GL and add check for the extension
     when creating the framebuffer
 - intel: Also text whether the extension is properly disabled by
     MESA_EXTENSION_OVERRIDE 

best, 
Gert

Gert Wollny (3):
  mesa/core: Add support for EXT_sRGB_write_control
  mesa/st: enable EXT_sRGB_write_control for Gallium drivers that
    support it
  intel/i965: Enable extension EXT_sRGB_write_control

 src/gallium/docs/source/screen.rst           |  3 ++
 src/mesa/drivers/dri/i965/intel_extensions.c |  1 +
 src/mesa/main/enable.c                       |  4 +--
 src/mesa/main/extensions_table.h             |  1 +
 src/mesa/main/get.c                          |  5 +++
 src/mesa/main/get_hash_params.py             |  1 +
 src/mesa/main/mtypes.h                       |  1 +
 src/mesa/state_tracker/st_extensions.c       |  8 ++++-
 src/mesa/state_tracker/st_manager.c          | 37 ++++++++++++--------
 9 files changed, 44 insertions(+), 17 deletions(-)

-- 
2.18.1



More information about the mesa-dev mailing list