[PATCH i-g-t 3/3] tests/kms_hdr: Handle HDR panels requiring DSC
Borah, Chaitanya Kumar
chaitanya.kumar.borah at intel.com
Thu Aug 21 09:51:07 UTC 2025
On 8/14/2025 1:46 PM, Sharma, Swati2 wrote:
> Hi Chaitanya,
>
> On 11-08-2025 01:28 pm, Chaitanya Kumar Borah wrote:
>> Some panels needs DSC to support HDR (at least in higher modes).
>> In cases like these the output bpc and crc comparison fails if
>> the SDR/non-HDR mode does not require DSC. Therefore, force DSC
>> when in SDR mode so that output bpc and crc can be compared.
>>
>> Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah at intel.com>
>> ---
>> tests/kms_hdr.c | 52 +++++++++++++++++++++++++++++++++++++++++++++++--
>> 1 file changed, 50 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/kms_hdr.c b/tests/kms_hdr.c
>> index 76e103ac4..c1123531a 100644
>> --- a/tests/kms_hdr.c
>> +++ b/tests/kms_hdr.c
>> @@ -97,6 +97,7 @@ enum {
>> TEST_INVALID_METADATA_SIZES = 1 << 4,
>> TEST_INVALID_HDR = 1 << 5,
>> TEST_BRIGHTNESS = 1 << 6,
>> + TEST_NEEDS_DSC = 1 << 7,
>> };
>> /* BPC connector state. */
>> @@ -491,9 +492,20 @@ static void test_static_toggle(data_t *data, enum
>> pipe pipe,
>> igt_plane_set_size(data->primary, data->w, data->h);
>> set_hdr_output_metadata(data, NULL);
>> igt_output_set_prop_value(data->output, IGT_CONNECTOR_MAX_BPC, 8);
>> +
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_enable(data->fd, output->name);
> Can you add debug prints in lib func() ?
This will cause duplicate logs for force_dsc_enable_bpc().
We should try to converge both the function in a future clean up.
==
Chaitanya
>> + igt_assert(igt_is_force_dsc_enabled(data->fd, output->name));
>> + }
>> +
>> igt_display_commit_atomic(display,
>> DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
>> igt_assert_output_bpc_equal(data->fd, pipe, output->name, 8);
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_disable(data->fd, output->name);
>> + igt_assert(igt_is_force_dsc_disabled(data->fd, output->name));
>> + }
>> +
>> /* Apply HDR metadata and 10bpc. We expect a modeset for
>> entering. */
>> set_hdr_output_metadata(data, &hdr);
>> igt_output_set_prop_value(data->output, IGT_CONNECTOR_MAX_BPC, 10);
>> @@ -518,11 +530,22 @@ static void test_static_toggle(data_t *data,
>> enum pipe pipe,
>> /* Disable HDR metadata and drop back to 8bpc. We expect a
>> modeset for exiting. */
>> set_hdr_output_metadata(data, NULL);
>> igt_output_set_prop_value(data->output, IGT_CONNECTOR_MAX_BPC, 8);
>> +
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_enable(data->fd, output->name);
>> + igt_assert(igt_is_force_dsc_enabled(data->fd, output->name));
>> + }
>> +
>> igt_display_commit_atomic(display,
>> DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
>> igt_assert_output_bpc_equal(data->fd, pipe, output->name, 8);
>> igt_assert_crc_equal(&ref_crc, &new_crc);
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_disable(data->fd, output->name);
>> + igt_assert(igt_is_force_dsc_disabled(data->fd, output->name));
>> + }
>> +
>> cleanup:
>> test_fini(data);
>> igt_remove_fb(data->fd, &afb);
>> @@ -558,7 +581,7 @@ static void fill_hdr_output_metadata_sdr(struct
>> hdr_output_metadata *meta)
>> meta->hdmi_metadata_type1.max_cll = 0;
>> }
>> -static void test_static_swap(data_t *data, enum pipe pipe,
>> igt_output_t *output)
>> +static void test_static_swap(data_t *data, enum pipe pipe,
>> igt_output_t *output, uint32_t flags)
>> {
>> igt_display_t *display = &data->display;
>> igt_crc_t ref_crc, new_crc;
>> @@ -577,9 +600,20 @@ static void test_static_swap(data_t *data, enum
>> pipe pipe, igt_output_t *output)
>> igt_plane_set_fb(data->primary, &afb);
>> igt_plane_set_size(data->primary, data->w, data->h);
>> igt_output_set_prop_value(data->output, IGT_CONNECTOR_MAX_BPC, 8);
>> +
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_enable(data->fd, output->name);
>> + igt_assert(igt_is_force_dsc_enabled(data->fd, output->name));
>> + }
>> +
>> igt_display_commit_atomic(display,
>> DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
>> igt_assert_output_bpc_equal(data->fd, pipe, output->name, 8);
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_disable(data->fd, output->name);
>> + igt_assert(igt_is_force_dsc_disabled(data->fd, output->name));
>> + }
>> +
>> /* Enter HDR, a modeset is allowed here. */
>> fill_hdr_output_metadata_st2048(&hdr);
>> set_hdr_output_metadata(data, &hdr);
>> @@ -602,6 +636,10 @@ static void test_static_swap(data_t *data, enum
>> pipe pipe, igt_output_t *output)
>> else
>> igt_display_commit_atomic(display,
>> DRM_MODE_ATOMIC_ALLOW_MODESET, NULL);
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_enable(data->fd, output->name);
>> + igt_assert(igt_is_force_dsc_enabled(data->fd, output->name));
>> + }
>> /* Enter SDR via metadata, no modeset allowed for
>> * amd driver, whereas a modeset is required for
>> * intel driver. */
>> @@ -623,6 +661,11 @@ static void test_static_swap(data_t *data, enum
>> pipe pipe, igt_output_t *output)
>> /* Verify that the CRC didn't change while cycling metadata. */
>> igt_assert_crc_equal(&ref_crc, &new_crc);
>> + if (flags & TEST_NEEDS_DSC) {
>> + igt_force_dsc_disable(data->fd, output->name);
>> + igt_assert(igt_is_force_dsc_disabled(data->fd, output->name));
>> + }
>> +
>> test_fini(data);
>> igt_remove_fb(data->fd, &afb);
>> }
>> @@ -714,6 +757,11 @@ static void test_hdr(data_t *data, uint32_t flags)
>> break;
>> }
>> + if (igt_is_dsc_enabled(data->fd, output->name))
>> + flags |= TEST_NEEDS_DSC;
>> + else
>> + flags &= ~TEST_NEEDS_DSC;
>> +
>> set_hdr_output_metadata(data, NULL);
>> igt_display_commit2(display, display->is_atomic ?
>> COMMIT_ATOMIC : COMMIT_LEGACY);
>> @@ -728,7 +776,7 @@ static void test_hdr(data_t *data, uint32_t flags)
>> TEST_INVALID_HDR | TEST_BRIGHTNESS))
>> test_static_toggle(data, pipe, output, flags);
>> if (flags & TEST_SWAP)
>> - test_static_swap(data, pipe, output);
>> + test_static_swap(data, pipe, output, flags);
>> if (flags & TEST_INVALID_METADATA_SIZES)
>> test_invalid_metadata_sizes(data, output);
>> }
>
More information about the igt-dev
mailing list