[igt-dev] [V5 10/11] tests/kms_dither: Rename dynamic tests

Sharma, Swati2 swati2.sharma at intel.com
Wed Jun 8 15:37:28 UTC 2022


LGTM
Reviewed-by:  Swati Sharma <swati2.sharma at intel.com>


On 17-May-22 5:32 PM, Bhanuprakash Modem wrote:
> Instead of using dynamic subtests as <output name>-<pipe name>,
> update as <pipe name>-<output name>.
> 
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> ---
>   tests/kms_dither.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/kms_dither.c b/tests/kms_dither.c
> index 21c1618a..c72f83be 100644
> --- a/tests/kms_dither.c
> +++ b/tests/kms_dither.c
> @@ -210,7 +210,8 @@ run_dither_test(data_t *data, int fb_bpc, int fb_format, int output_bpc)
>   
>   		for_each_pipe(display, pipe) {
>   			if (igt_pipe_connector_valid(pipe, output)) {
> -				igt_dynamic_f("%s-pipe-%s", output->name, kmstest_pipe_name(pipe))
> +				igt_dynamic_f("pipe-%s-%s",
> +					      kmstest_pipe_name(pipe), output->name)
>   					test_dithering(data, pipe, output, fb_bpc,
>   							fb_format, output_bpc);
>   

-- 
~Swati Sharma


More information about the igt-dev mailing list