[Mesa-dev] [PATCH 0/3] common bits from HDR POC

[Tapani Pälli]

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.palli at intel.com 
> <mailto:tapani.palli at 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 at lists.freedesktop.org <mailto:mesa-dev at lists.freedesktop.org>
>     https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>     <https://lists.freedesktop.org/mailman/listinfo/mesa-dev>
> 
>