[igt-dev] [PATCH i-g-t] tests/kms_available_modes_crc : Added test description

Martin Peres martin.peres at mupuf.org
Wed Apr 7 20:29:33 UTC 2021


On 06/04/2021 05:19, Nidhi Gupta wrote:
> Added test description.
> 
> Signed-off-by: Nidhi Gupta <nidhi1.gupta at intel.com>
> ---
>   tests/kms_available_modes_crc.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/kms_available_modes_crc.c b/tests/kms_available_modes_crc.c
> index 17e0f237..a1dd1289 100644
> --- a/tests/kms_available_modes_crc.c
> +++ b/tests/kms_available_modes_crc.c
> @@ -448,6 +448,7 @@ igt_main
>   
>   	data.commit = data.display.is_atomic ? COMMIT_ATOMIC : COMMIT_LEGACY;
>   
> +	igt_describe("CRC test for all different plane modes which kernel advertises.");

This description looks more like a how rather than a what. How about 
"Check that all the plane modes advertised by the kernel produce a valid 
output"?

Martin

>   	igt_subtest("available_mode_test_crc") {
>   		gem_require_mappable_ggtt(data.gfx_fd);
>   		test_available_modes(&data);
> 


More information about the igt-dev mailing list