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

["Nautiyal, Ankit K" <ankit.k.nautiyal@intel.com>]

LGTM.

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

On 7/4/2023 11:01 PM, Swati Sharma wrote:
> New subtest is added to validate fractional bpp with different
> input bpc.
>
> Signed-off-by: Swati Sharma <swati2.sharma@intel.com>
> ---
>   tests/i915/kms_dsc.c | 10 ++++++++++
>   1 file changed, 10 insertions(+)
>
> diff --git a/tests/i915/kms_dsc.c b/tests/i915/kms_dsc.c
> index 26d6e2ffb..d46a5e74d 100644
> --- a/tests/i915/kms_dsc.c
> +++ b/tests/i915/kms_dsc.c
> @@ -392,6 +392,16 @@ igt_main_args("l", NULL, help_str, opt_handler, &data)
>   				 DEFAULT_BPC, DRM_FORMAT_XRGB8888,
>   				 DSC_FORMAT_RGB);
>   
> +	igt_describe("Tests fractional compressed bpp functionality if supported "
> +		     "by a connector by forcing fractional_bpp on all connectors that support it "
> +		     "with certain input BPC for the connector.");
> +	igt_subtest_with_dynamic("dsc-fractional-bpp-with-bpc") {
> +		for (int j = 0; j < ARRAY_SIZE(bpc_list); j++)
> +			test_dsc(&data, TEST_DSC_FRACTIONAL_BPP | TEST_DSC_BPC,
> +				 bpc_list[j], DRM_FORMAT_XRGB8888,
> +				 DSC_FORMAT_RGB);
> +	}
> +
>   	igt_fixture {
>   		igt_display_fini(&data.display);
>   		drm_close_driver(data.drm_fd);