[igt-dev] [PATCH i-g-t 1/2] tests/kms_atomic_transition: Add explicit curly brackets

Petri Latvala petri.latvala at intel.com
Thu Jul 23 11:32:46 UTC 2020


On Thu, Jul 23, 2020 at 02:16:14PM +0300, Arkadiusz Hiler wrote:
> To clean up a bit and get rid of compilation warnings.
> 
> Cc: Mohammed Khajapasha <mohammed.khajapasha at intel.com>
> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler at intel.com>

Reviewed-by: Petri Latvala <petri.latvala at intel.com>



> ---
>  tests/kms_atomic_transition.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c
> index f54591e6..a71bbe10 100644
> --- a/tests/kms_atomic_transition.c
> +++ b/tests/kms_atomic_transition.c
> @@ -788,8 +788,9 @@ static void run_modeset_tests(igt_display_t *display, int howmany, bool nonblock
>  
>  			if (fencing)
>  				igt_pipe_request_out_fence(&display->pipes[i]);
> -		} else
> +		} else {
>  			igt_plane_set_fb(plane, NULL);
> +		}
>  	}
>  
>  	iter_max = 1 << j;
> @@ -859,9 +860,10 @@ static void run_modeset_tests(igt_display_t *display, int howmany, bool nonblock
>  	set_combinations(display, 0, NULL);
>  	igt_display_commit2(display, COMMIT_ATOMIC);
>  
> -	if (is_i915_device(display->drm_fd))
> +	if (is_i915_device(display->drm_fd)) {
>  		for_each_pipe(display, i)
>  			igt_pipe_crc_free(pipe_crcs[i]);
> +	}
>  
>  	igt_remove_fb(display->drm_fd, &fbs[1]);
>  	igt_remove_fb(display->drm_fd, &fbs[0]);
> -- 
> 2.25.4
> 
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev


More information about the igt-dev mailing list