On 01.05.2018 01:22, Jason Ekstrand wrote:
Here's a really dumb question:  What good does it do someone to have the extension if no actual window-system HDR is supported?  I'm just wondering if we want to only expose the extension if hardware can actually do it.

I think it's a good question. My understanding is that main source of 'is hdr supported?' comes from EDID and is then exposed to clients via different mechanisms/frameworks depending on system.

With EGL we could default to off and then different EGL drivers could ask if HDR is supported and toggle extension on? Alternatively we just have it toggled on and the metadata would get utilized only when hw support is there, HDR capable display is connected and window system in question supports transmitting the data.

I'm fine both ways, I left it on by default as it felt harmless enough.


On Sun, Apr 29, 2018 at 11:42 PM, Tapani Pälli <tapani.pa...@intel.com <mailto:tapani.pa...@intel.com>> wrote:

    Hi;

    I took Ville's HDR POC Mesa patches, fixed some issues and removed all
    Wayland specific parts. Here are the bits that could be already
    integrated,
    rest will be window system specific parts.

    // Tapani


    Tapani Pälli (1):
       egl: implement EXT_surface_SMPTE2086_metadata and
         EXT_surface_CTA861_3_metadata

    Ville Syrjälä (2):
       vulkan/wsi: VK_EXT_hdr_metadata plumbing
       anv: Plug in VK_EXT_hdr_metadata

      src/egl/drivers/dri2/egl_dri2.c     |   3 +
      src/egl/main/eglapi.c               |   2 +
      src/egl/main/egldisplay.h           |   2 +
      src/egl/main/eglsurface.c           | 169
    ++++++++++++++++++++++++++++++++++++
      src/egl/main/eglsurface.h           |  19 ++++
      src/intel/vulkan/anv_extensions.py  |   1 +
      src/intel/vulkan/anv_wsi.c          |   9 ++
      src/vulkan/wsi/wsi_common.c         |  12 +++
      src/vulkan/wsi/wsi_common.h         |   5 ++
      src/vulkan/wsi/wsi_common_private.h |   2 +
      10 files changed, 224 insertions(+)

-- 2.13.6

    _______________________________________________
    mesa-dev mailing list
    mesa-dev@lists.freedesktop.org <mailto:mesa-dev@lists.freedesktop.org>
    https://lists.freedesktop.org/mailman/listinfo/mesa-dev
    <https://lists.freedesktop.org/mailman/listinfo/mesa-dev>


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to