From: Harry Wentland <harry.wentland@amd.com>
To: <dri-devel@lists.freedesktop.org>, <amd-gfx@lists.freedesktop.org>
Cc: Harry Wentland <harry.wentland@amd.com>,
Alex Hung <alex.hung@amd.com>,
Daniel Stone <daniels@collabora.com>,
Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>,
Uma Shankar <uma.shankar@intel.com>,
"Louis Chauvet" <louis.chauvet@bootlin.com>,
Melissa Wen <mwen@igalia.com>, Simon Ser <contact@emersion.fr>,
Robert Mader <robert.mader@collabora.com>
Subject: [PATCH v5 00/10] YUV conversion colorop with amdgpu and VKMS
Date: Fri, 31 Jul 2026 14:15:11 -0400 [thread overview]
Message-ID: <20260731181521.681990-1-harry.wentland@amd.com> (raw)
When we merged the drm_plane color pipeline API the major gap
that existed was the lack of a YUV to RGB conversion colorop.
We deprecated any legacy drm_plane color properties, which
means that the COLOR_RANGE and COLOR_ENCODING properties can't
be used with the COLOR_PIPELINE property on a drm_plane. In
practice this means that we can't use a COLOR_PIPELINE on
YCbCr encoded framebuffers.
This patchset expands on the Fixed Matrix colorop proposed by Chaitanya
and adds limited range variants of the YCbCr to RGB conversions.
His full patchset can be found at
https://patchwork.freedesktop.org/patch/709860
This code has been tested with IGT and an experimental KWin branch.
All patches are now reviewed and tested. We have a Weston and
KWin implementation. IGT patches are missing one review. I
deem these patches ready to merge once the last IGT patch review
comes in.
IGT branch:
https://gitlab.freedesktop.org/hwentland/igt-gpu-tools/-/tree/yuv-fm-colorop
KWin branch used for testing:
https://invent.kde.org/hwentlan/kwin/-/tree/yuv-fm-colorop
The kernel branch containing these changes, based on drm-misc-next
can be found at:
https://gitlab.freedesktop.org/hwentland/linux/-/tree/yuv-fm-colorop
Further background on this work can be found at:
https://hwentland.github.io/2026/03/10/plane-color-pipeline-csc-3d-lut-kwin.html
v5:
- Drop new VKMS kunit tests for conversion matrices
- Added script to show how VKMS kunit test values are computed (Pekka)
- Removed fixed-matrix enums for "YCbCr limtied to full" and
"RGB709 to RGB2020" as they're currently unused by userspace (Robert)
v4:
- Specify matrix entries in docs (Pekka)
- Squash limited-range enums into "Add FM" patch (Robert)
- Don't reject RGB planes with fixed matrix in VKMS as
we don't want or need to make a colorop dependent on
the framebuffer's pixel format. (Robert)
- Fix conversion matrices in VKMS and implement kunit
tests (discovered while documenting the matrices)
v3:
- base on Chaitanya's updated patch and rename code accordingly
to Fixed_Matrix instead of CSC Fixed-Function
v2:
- use Chaitanya's CSC_FF block for named matrices
Cc: Alex Hung <alex.hung@amd.com>
Cc: Daniel Stone <daniels@collabora.com>
Cc: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Cc: Uma Shankar <uma.shankar@intel.com>
Cc: Louis Chauvet <louis.chauvet@bootlin.com>
Cc: Melissa Wen <mwen@igalia.com>
Cc: Simon Ser <contact@emersion.fr>
Cc: Robert Mader <robert.mader@collabora.com>
Chaitanya Kumar Borah (1):
drm/colorop: Add DRM_COLOROP_FIXED_MATRIX
Harry Wentland (9):
drm/vkms: Fix limited-range YCbCr to RGB conversion scaling
drm/vkms: Add fixed matrix colorop to color pipeline
drm/vkms: Add atomic check and matrix handling for fixed matrix
colorop
drm/amd/display: Add fixed matrix colorop to color pipeline
drm/amd/display: Implement fixed matrix colorop color space mapping
drm/amd/display: Use GAMCOR for first TF if YUV conversion is needed
drm/amd/display: Check actual state during commit_tail
drm/amd/display: Set color_space to plane_infos
drm/amd/display: Force GAMCOR for subsampled surfaces with
PQ/Gamma22/HLG
.../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 14 +-
.../amd/display/amdgpu_dm/amdgpu_dm_color.c | 85 +++++++++++-
.../amd/display/amdgpu_dm/amdgpu_dm_colorop.c | 27 +++-
.../amd/display/amdgpu_dm/amdgpu_dm_colorop.h | 1 +
.../amd/display/modules/color/color_gamma.c | 3 +-
drivers/gpu/drm/drm_atomic.c | 4 +
drivers/gpu/drm/drm_atomic_uapi.c | 7 +
drivers/gpu/drm/drm_colorop.c | 107 +++++++++++++++
.../gpu/drm/vkms/tests/gen_yuv_conversion.py | 87 ++++++++++++
drivers/gpu/drm/vkms/tests/vkms_format_test.c | 40 +++---
drivers/gpu/drm/vkms/vkms_colorop.c | 66 ++++++---
drivers/gpu/drm/vkms/vkms_composer.c | 6 +
drivers/gpu/drm/vkms/vkms_formats.c | 64 ++++++---
drivers/gpu/drm/vkms/vkms_formats.h | 2 +-
drivers/gpu/drm/vkms/vkms_plane.c | 55 +++++++-
include/drm/drm_colorop.h | 127 ++++++++++++++++++
include/uapi/drm/drm_mode.h | 12 ++
17 files changed, 639 insertions(+), 68 deletions(-)
create mode 100755 drivers/gpu/drm/vkms/tests/gen_yuv_conversion.py
--
2.55.0
next reply other threads:[~2026-07-31 18:15 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-31 18:15 Harry Wentland [this message]
2026-07-31 18:15 ` [PATCH v5 01/10] drm/colorop: Add DRM_COLOROP_FIXED_MATRIX Harry Wentland
2026-07-31 18:15 ` [PATCH v5 02/10] drm/vkms: Fix limited-range YCbCr to RGB conversion scaling Harry Wentland
2026-07-31 18:15 ` [PATCH v5 03/10] drm/vkms: Add fixed matrix colorop to color pipeline Harry Wentland
2026-07-31 18:15 ` [PATCH v5 04/10] drm/vkms: Add atomic check and matrix handling for fixed matrix colorop Harry Wentland
2026-07-31 18:15 ` [PATCH v5 05/10] drm/amd/display: Add fixed matrix colorop to color pipeline Harry Wentland
2026-07-31 18:15 ` [PATCH v5 06/10] drm/amd/display: Implement fixed matrix colorop color space mapping Harry Wentland
2026-07-31 18:15 ` [PATCH v5 07/10] drm/amd/display: Use GAMCOR for first TF if YUV conversion is needed Harry Wentland
2026-07-31 18:15 ` [PATCH v5 08/10] drm/amd/display: Check actual state during commit_tail Harry Wentland
2026-07-31 18:15 ` [PATCH v5 09/10] drm/amd/display: Set color_space to plane_infos Harry Wentland
2026-07-31 18:15 ` [PATCH v5 10/10] drm/amd/display: Force GAMCOR for subsampled surfaces with PQ/Gamma22/HLG Harry Wentland
2026-08-01 9:42 ` [PATCH v5 00/10] YUV conversion colorop with amdgpu and VKMS Robert Mader
2026-08-14 19:33 ` Harry Wentland
2026-08-14 20:03 ` Robert Mader
2026-08-14 20:31 ` Harry Wentland
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260731181521.681990-1-harry.wentland@amd.com \
--to=harry.wentland@amd.com \
--cc=alex.hung@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=chaitanya.kumar.borah@intel.com \
--cc=contact@emersion.fr \
--cc=daniels@collabora.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=louis.chauvet@bootlin.com \
--cc=mwen@igalia.com \
--cc=robert.mader@collabora.com \
--cc=uma.shankar@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.