Re: [PATCH i-g-t 2/2] tests/intel/kms_psr2_sf: Add checks for early transport

["Hogander, Jouni" <jouni.hogander@intel.com>]

On Mon, 2024-05-20 at 15:16 +0530, Jeevan B wrote:
> Add checks weather early transport if supported then
> we check if early trasport is working.
> 
> Signed-off-by: Jeevan B <jeevan.b@intel.com>
> ---
>  tests/intel/kms_psr2_sf.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/tests/intel/kms_psr2_sf.c b/tests/intel/kms_psr2_sf.c
> index 4c3e34aa0..4908f2444 100644
> --- a/tests/intel/kms_psr2_sf.c
> +++ b/tests/intel/kms_psr2_sf.c
> @@ -916,6 +916,9 @@ static void run(data_t *data)
>                 igt_assert_f(intel_fbc_wait_until_enabled(data-
> >drm_fd,
>                                                           data-
> >pipe),
>                                                           "FBC still
> disabled\n");
> +       if (psr_sink_support(data->drm_fd, data->debugfs_fd, ET_MODE,
> NULL))
> +               igt_assert_f(psr2_early_transport_check(data-
> >debugfs_fd),
> +                            "PSR2 early transport not enabled\n");

You are adding new mode (ET_MODE) in previous patch. You should carry
it in data->psr_mode and modify set_sel_fetch_mode_for_output
accordingly. This way you could just rely on current checks. You need
to implement psr_active_check for this new mode in patch 01/02.

BR,

Jouni Högander

>  
>         data->screen_changes = 0;
>