Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Swati Sharma <swati2.sharma@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: chaitanya.kumar.borah@intel.com, Swati Sharma <swati2.sharma@intel.com>
Subject: [PATCH i-g-t,v5 0/7] Add FIXED_MATRIX colorop tests
Date: Tue,  1 Sep 2026 00:59:28 +0530	[thread overview]
Message-ID: <20260831192935.463134-1-swati2.sharma@intel.com> (raw)

Add support for testing the DRM FIXED_MATRIX colorop type, which
selects from predefined hardware color space conversion matrices
rather than programmable coefficients.

The series adds:
- Pipe-CRC based tests in kms_color_pipeline
- Chamelium frame-capture based tests in kms_chamelium_color_pipeline

With this, the SDR plane color pipeline looks like:

[YUV Full/Limited -> RGB] -> [1D LUT]

A single FIXED_MATRIX colorop handles both full and limited range
YCbCr to RGB conversion.

Supported FIXED_MATRIX modes:
  - YCbCr 601 Full to RGB
  - YCbCr 601 Limited to RGB
  - YCbCr 709 Full to RGB
  - YCbCr 709 Limited to RGB
  - YCbCr 2020 NC Full to RGB
  - YCbCr 2020 NC Limited to RGB

v2: -Naming changes (CSC_FF -> FIXED_MATRIX)
v3: -Drop YCbCr limited to full and RGB709 to RGB2020 matrices
     until we have non-IGT userspace that shows their use
    -Rename subtests to yuv-rgb-btXXX with limited range variants
    -Add limited range test coverage
v4: -Reuse _test_plane_colorops() for FIXED_MATRIX instead of
     separate _test_plane_fixed_matrix_colorops()
    -Store encoding/range in colorop fixed_matrix_info
v5: -Rebase

Swati Sharma (7):
  lib/igt_kms: Rename IGT_COLOROP_FIXED_MATRIX to
    IGT_COLOROP_FIXED_MATRIX_TYPE
  lib/igt_kms: Order IGT_COLOROP_FIXED_MATRIX_TYPE before
    IGT_COLOROP_NEXT
  tests/kms_colorop_helper: Add Fixed Matrix enum names table
  tests/kms_color_pipeline: Add FIXED_MATRIX colorop tests
  tests/kms_color_pipeline: Remove unused color_depth and drm_format
  tests/kms_chamelium_color_pipeline: Add FIXED_MATRIX colorop tests
  tests/chamelium/kms_chamelium_color_pipeline: Fix CRTC naming

 lib/igt_kms.c                                 |   2 +-
 lib/igt_kms.h                                 |   2 +-
 .../chamelium/kms_chamelium_color_pipeline.c  | 281 ++++++++++++++--
 tests/kms_color_pipeline.c                    | 305 ++++++++++++++++--
 tests/kms_colorop_helper.c                    |  26 +-
 tests/kms_colorop_helper.h                    |  14 +-
 6 files changed, 570 insertions(+), 60 deletions(-)

-- 
2.25.1


             reply	other threads:[~2026-08-31 19:21 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-31 19:29 Swati Sharma [this message]
2026-08-31 19:29 ` [PATCH i-g-t, v5 1/7] lib/igt_kms: Rename IGT_COLOROP_FIXED_MATRIX to IGT_COLOROP_FIXED_MATRIX_TYPE Swati Sharma
2026-09-09 10:27   ` Borah, Chaitanya Kumar
2026-08-31 19:29 ` [PATCH i-g-t, v5 2/7] lib/igt_kms: Order IGT_COLOROP_FIXED_MATRIX_TYPE before IGT_COLOROP_NEXT Swati Sharma
2026-09-09 10:27   ` [PATCH i-g-t,v5 " Borah, Chaitanya Kumar
2026-08-31 19:29 ` [PATCH i-g-t, v5 3/7] tests/kms_colorop_helper: Add Fixed Matrix enum names table Swati Sharma
2026-09-09 10:28   ` [PATCH i-g-t,v5 " Borah, Chaitanya Kumar
2026-08-31 19:29 ` [PATCH i-g-t, v5 4/7] tests/kms_color_pipeline: Add FIXED_MATRIX colorop tests Swati Sharma
2026-09-09 10:29   ` [PATCH i-g-t,v5 " Borah, Chaitanya Kumar
2026-08-31 19:29 ` [PATCH i-g-t, v5 5/7] tests/kms_color_pipeline: Remove unused color_depth and drm_format Swati Sharma
2026-08-31 19:29 ` [PATCH i-g-t, v5 6/7] tests/kms_chamelium_color_pipeline: Add FIXED_MATRIX colorop tests Swati Sharma
2026-09-09 10:29   ` [PATCH i-g-t,v5 " Borah, Chaitanya Kumar
2026-08-31 19:29 ` [PATCH i-g-t, v5 7/7] tests/chamelium/kms_chamelium_color_pipeline: Fix CRTC naming Swati Sharma
2026-09-09 10:30   ` [PATCH i-g-t,v5 " Borah, Chaitanya Kumar
2026-09-01 19:21 ` ✓ Xe.CI.BAT: success for Add FIXED_MATRIX colorop tests (rev5) Patchwork
2026-09-01 19:51 ` ✓ i915.CI.BAT: " Patchwork
2026-09-02  5:27 ` ✗ i915.CI.Full: failure " Patchwork

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=20260831192935.463134-1-swati2.sharma@intel.com \
    --to=swati2.sharma@intel.com \
    --cc=chaitanya.kumar.borah@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox