[igt-dev] [v2 5/9] tests/i915/kms_dsc: add subtest

[Nautiyal, Ankit K]

Lets add a little more detail in subject line phrase.

Same for the other patch with same subject line.

Otherwise LGTM.

Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal at intel.com>


On 7/4/2023 11:01 PM, Swati Sharma wrote:
> New subtest is added validating output format with different
> input bpc.
>
> Signed-off-by: Swati Sharma <swati2.sharma at intel.com>
> ---
>   tests/i915/kms_dsc.c | 13 +++++++++++++
>   1 file changed, 13 insertions(+)
>
> diff --git a/tests/i915/kms_dsc.c b/tests/i915/kms_dsc.c
> index df9b27225..d1935dcd3 100644
> --- a/tests/i915/kms_dsc.c
> +++ b/tests/i915/kms_dsc.c
> @@ -357,6 +357,19 @@ igt_main_args("l", NULL, help_str, opt_handler, &data)
>   				 output_format_list[k]);
>   	}
>   
> +	igt_describe("Tests basic display stream compression functionality if supported "
> +		     "by a connector by forcing DSC and output format on all connectors "
> +		     "that support it with certain input BPC for the connector");
> +	igt_subtest_with_dynamic("dsc-with-output-formats-with-bpc") {
> +		for (int k = 0; k < ARRAY_SIZE(output_format_list); k++) {
> +			for (int j = 0; j < ARRAY_SIZE(bpc_list); j++) {
> +				test_dsc(&data, TEST_DSC_OUTPUT_FORMAT | TEST_DSC_BPC,
> +					 bpc_list[j], DRM_FORMAT_XRGB8888,
> +					 output_format_list[k]);
> +			}
> +		}
> +	}
> +
>   	igt_fixture {
>   		igt_display_fini(&data.display);
>   		drm_close_driver(data.drm_fd);