Re: [igt-dev] [PATCH i-g-t] tests/i915/kms_psr2_sf: Fix cleanup

["B, Jeevan" <jeevan.b@intel.com>]

Can we set display_reset before setting pipe ?? 

> -----Original Message-----
> From: Hogander, Jouni <jouni.hogander@intel.com>
> Sent: Thursday, March 23, 2023 1:08 PM
> To: igt-dev@lists.freedesktop.org
> Cc: Hogander, Jouni <jouni.hogander@intel.com>; B, Jeevan
> <jeevan.b@intel.com>
> Subject: [PATCH i-g-t] tests/i915/kms_psr2_sf: Fix cleanup
> 
> Cleanup is leaving pipe reserved. This is causing failures like:
> 
> igt_kms-CRITICAL: DP-1 and eDP-1 are both trying to use pipe A
> 
> Fix by setting pipe of output as PIPE_NONE.
> 
> Cc: Jeevan B <jeevan.b@intel.com>
> 
> Signed-off-by: Jouni Högander <jouni.hogander@intel.com>
> ---
>  tests/i915/kms_psr2_sf.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/i915/kms_psr2_sf.c b/tests/i915/kms_psr2_sf.c index
> 2d761d31..4a8608c1 100644
> --- a/tests/i915/kms_psr2_sf.c
> +++ b/tests/i915/kms_psr2_sf.c
> @@ -846,10 +846,10 @@ static void cleanup(data_t *data)
>  		igt_plane_set_fb(sprite, NULL);
>  	}
> 
> -	if (data->coexist_feature & FEATURE_DSC) {
> +	if (data->coexist_feature & FEATURE_DSC)
>  		restore_force_dsc_en();
> -		igt_output_set_pipe(output, PIPE_NONE);
> -	}
> +
> +	igt_output_set_pipe(output, PIPE_NONE);
> 
>  	igt_display_commit2(&data->display, COMMIT_ATOMIC);
> 
> --
> 2.34.1