[PATCH i-g-t 2/2] tests/intel/kms_frontbuffer_tracking: pass non-NULL output pointer

[Hogander, Jouni]

On Tue, 2025-07-15 at 13:38 +0200, Michał Grzelak wrote:
> Currently, there is NULL pointer passed to the test as output. Pass
> non-NULL output pointer to the test.
> 
> Signed-off-by: Michał Grzelak <michal.grzelak at intel.com>
> ---
>  tests/intel/kms_frontbuffer_tracking.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/kms_frontbuffer_tracking.c
> b/tests/intel/kms_frontbuffer_tracking.c
> index 66dcdc281..c55b296ec 100644
> --- a/tests/intel/kms_frontbuffer_tracking.c
> +++ b/tests/intel/kms_frontbuffer_tracking.c
> @@ -2329,7 +2329,7 @@ static void do_status_assertions(int flags)
>  	}
>  
>  	if (flags & ASSERT_PSR_ENABLED) {
> -		igt_assert_f(psr_wait_entry(drm.debugfs, PSR_MODE_1,
> NULL),
> +		igt_assert_f(psr_wait_entry(drm.debugfs, PSR_MODE_1,
> prim_mode_params.output),
>  			     "PSR still disabled\n");

Do you have specific reason to pass the output? Originally output was
added as a parameter to handle DP (Panel Replay) cases. PSR1 is not
supported by DP so passing the output is not needed.

BR,

Jouni Högander

>  		psr_sink_error_check(drm.debugfs, PSR_MODE_1,
> prim_mode_params.output);
>  	} else if (flags & ASSERT_PSR_DISABLED)