mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2025-12-30 16:30:10 +01:00
vulkan/wsi: remove support for VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT
It's not really clear whether or not it should use gamma 2.2 or the piece-wise transfer function, or how clients would use it for wider gamut in general. Currently no compositors I know of support ext_srgb, so this shouldn't affect applications in practice. Signed-off-by: Xaver Hugl <xaver.hugl@kde.org> Fixes:4b663d56("vulkan/wsi: implement support for VK_EXT_hdr_metadata on Wayland") (cherry picked from commit14fcf145e3) Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/38167>
This commit is contained in:
parent
2a0a2cc5b0
commit
6a7effe059
2 changed files with 4 additions and 7 deletions
|
|
@ -594,7 +594,7 @@
|
|||
"description": "vulkan/wsi: remove support for VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT",
|
||||
"nominated": true,
|
||||
"nomination_type": 2,
|
||||
"resolution": 0,
|
||||
"resolution": 1,
|
||||
"main_sha": null,
|
||||
"because_sha": "4b663d561b877c24fe1f500e14cf7efc62fd5373",
|
||||
"notes": null
|
||||
|
|
|
|||
|
|
@ -1002,12 +1002,9 @@ struct Colorspace colorspace_mapping[] = {
|
|||
/* VK_COLOR_SPACE_ADOBERGB_NONLINEAR_EXT is left out because there's no
|
||||
* exactly matching transfer function in the Wayland protocol */
|
||||
/* VK_COLOR_SPACE_PASS_THROUGH_EXT is handled elsewhere */
|
||||
{
|
||||
.colorspace = VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT,
|
||||
.primaries = WP_COLOR_MANAGER_V1_PRIMARIES_SRGB,
|
||||
.tf = WP_COLOR_MANAGER_V1_TRANSFER_FUNCTION_EXT_SRGB,
|
||||
.should_use_hdr_metadata = true,
|
||||
},
|
||||
/* VK_COLOR_SPACE_EXTENDED_SRGB_NONLINEAR_EXT is intentionally not added
|
||||
* as it's a bit unclear how exactly it should be used
|
||||
* and whether or not the transfer function should be gamma 2.2 or piece-wise */
|
||||
/* VK_COLOR_SPACE_DISPLAY_NATIVE_AMD isn't supported */
|
||||
/* VK_COLORSPACE_SRGB_NONLINEAR_KHR is just an alias */
|
||||
/* VK_COLOR_SPACE_DCI_P3_LINEAR_EXT is just an alias */
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue