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,v4 0/7] Add FIXED_MATRIX colorop tests
Date: Thu, 27 Aug 2026 00:09:25 +0530	[thread overview]
Message-ID: <20260826183932.351933-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:
- UAPI definition for DRM_COLOROP_FIXED_MATRIX
- FIXED_MATRIX_TYPE property support in igt_kms
- Colorop helper support for configuring FIXED_MATRIX blocks
- 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

Harry Wentland (1):
  tests/kms_colorop_helper: Add Fixed Matrix colorop infrastructure

Swati Sharma (6):
  include/drm-uapi: Add DRM_COLOROP_FIXED_MATRIX definition
  lib/igt_kms: Add FIXED_MATRIX_TYPE colorop property
  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

 include/drm-uapi/drm_mode.h                   |  11 +
 lib/igt_kms.c                                 |   1 +
 lib/igt_kms.h                                 |   1 +
 .../chamelium/kms_chamelium_color_pipeline.c  | 279 ++++++++++++++--
 tests/kms_color_pipeline.c                    | 305 ++++++++++++++++--
 tests/kms_colorop_helper.c                    |  64 ++++
 tests/kms_colorop_helper.h                    |  26 +-
 7 files changed, 637 insertions(+), 50 deletions(-)

-- 
2.25.1


             reply	other threads:[~2026-08-26 18:31 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-26 18:39 Swati Sharma [this message]
2026-08-26 18:39 ` [PATCH i-g-t, v4 1/7] include/drm-uapi: Add DRM_COLOROP_FIXED_MATRIX definition Swati Sharma
2026-08-26 18:39 ` [PATCH i-g-t, v4 2/7] lib/igt_kms: Add FIXED_MATRIX_TYPE colorop property Swati Sharma
2026-08-26 18:39 ` [PATCH i-g-t, v4 3/7] tests/kms_colorop_helper: Add Fixed Matrix colorop infrastructure Swati Sharma
2026-08-26 18:39 ` [PATCH i-g-t, v4 4/7] tests/kms_colorop_helper: Add Fixed Matrix enum names table Swati Sharma
2026-08-26 18:39 ` [PATCH i-g-t, v4 5/7] tests/kms_color_pipeline: Add FIXED_MATRIX colorop tests Swati Sharma
2026-08-26 18:39 ` [PATCH i-g-t, v4 6/7] tests/kms_color_pipeline: Remove unused color_depth and drm_format Swati Sharma
2026-08-26 18:39 ` [PATCH i-g-t, v4 7/7] tests/kms_chamelium_color_pipeline: Add FIXED_MATRIX colorop tests Swati Sharma
2026-08-31  7:19   ` Jani Nikula
2026-08-26 19:52 ` ✓ Xe.CI.BAT: success for Add FIXED_MATRIX colorop tests (rev4) Patchwork
2026-08-26 20:11 ` ✓ i915.CI.BAT: " Patchwork
2026-08-27  0:02 ` ✗ Xe.CI.FULL: failure " Patchwork
2026-08-27  3:09 ` ✗ i915.CI.Full: " 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=20260826183932.351933-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