[PATCH v2 1/1] tests/kms_plane: Restricting number of pixel formats for simulation

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Feb 14 13:07:18 UTC 2024


Hi Samala,,
On 2024-02-14 at 14:23:28 +0530, Samala, Pranay wrote:
> The pixel-format subtest runs on all pixel formats, reducing it to single
> format to reduce execution time on simulation.
> 
> Cc: B S, Karthik <karthik.b.s at intel.com>
----- ^^^^
Should be:
Cc: Karthik B S <karthik.b.s at intel.com>

> Cc: B, Jeevan <jeevan.b at intel.com>
----- ^^^^
Same here:
Cc: Jeevan B <jeevan.b at intel.com>

> 
> Signed-off-by: Samala, Pranay <pranay.samala at intel.com>

Please correct this in your git config, this should be:

Signed-off-by: Pranay Samala <pranay.samala at intel.com>

This can be corrected at merge,

Regards,
Kamil

> ---
>  tests/kms_plane.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/tests/kms_plane.c b/tests/kms_plane.c
> index 406aecc04..fbffc3cb0 100644
> --- a/tests/kms_plane.c
> +++ b/tests/kms_plane.c
> @@ -1081,6 +1081,9 @@ static bool test_format_plane(data_t *data, enum pipe pipe,
>  		    f.modifier == ref.modifier)
>  			continue;
>  
> +		if (f.format != DRM_FORMAT_XRGB8888 && igt_run_in_simulation())
> +			continue;
> +
>  		/* test each format "class" only once in non-extended tests */
>  		if (!data->extended && f.modifier != DRM_FORMAT_MOD_LINEAR) {
>  			struct format_mod rf = {
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list