[igt-dev] [PATCH 2/2] tests/amdgpu/amd_psr: drop the PSR state check for PSR-SU MPO test

[Pillai, Aurabindo]

[AMD Official Use Only]

Reviewed-by: Aurabindo Pillai <aurabindo.pillai at amd.com>

--

Regards,
Jay
________________________________
From: Zhang, Dingchen (David) <Dingchen.Zhang at amd.com>
Sent: Wednesday, April 6, 2022 4:28 PM
To: igt-dev at lists.freedesktop.org <igt-dev at lists.freedesktop.org>
Cc: Siqueira, Rodrigo <Rodrigo.Siqueira at amd.com>; Wentland, Harry <Harry.Wentland at amd.com>; Li, Sun peng (Leo) <Sunpeng.Li at amd.com>; Pillai, Aurabindo <Aurabindo.Pillai at amd.com>; Lin, Wayne <Wayne.Lin at amd.com>
Subject: [PATCH 2/2] tests/amdgpu/amd_psr: drop the PSR state check for PSR-SU MPO test

[why & how]
Currently the enablement of PSR-SU for Linux kernel is still WIP
and the PSR state to check for PSR-SU MPO is not upstreamed yet.
In addition, all the PSR-SU test case currently by default is
validated with visual confirm.

Drop the PSR state check in MPO PSR-SU sub-test case in the loop.

Cc: Rodrigo Siqueira <rodrigo.siqueira at amd.com>
Cc: Harry Wentland <harry.wentland at amd.com>
Cc: Leo Li <sunpeng.li at amd.com>
Cc: Jay Pillai <aurabindo.pillai at amd.com>
Cc: Wayne Lin <wayne.lin at amd.com>

Signed-off-by: David Zhang <dingchen.zhang at amd.com>
---
 tests/amdgpu/amd_psr.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/tests/amdgpu/amd_psr.c b/tests/amdgpu/amd_psr.c
index 83e44c4e..8ea73080 100644
--- a/tests/amdgpu/amd_psr.c
+++ b/tests/amdgpu/amd_psr.c
@@ -292,18 +292,6 @@ static void run_check_psr_su_mpo(data_t *data)
                                       flip_fb->fb_id, DRM_MODE_PAGE_FLIP_EVENT, NULL);
                 igt_require(ret == 0);
                 kmstest_wait_for_pageflip(data->fd);
-
-               /* check PSR state */
-               if (i > PSR_SETTLE_DELAY * frame_rate) {
-                       psr_state = igt_amd_read_psr_state(data->fd, data->output->name);
-                       igt_fail_on_f(psr_state < 0, "Open PSR state debugfs failed\n");
-                       igt_fail_on_f(psr_state == PSR_STATE0,
-                               "PSR was not enabled for connector %s\n", data->output->name);
-                       igt_fail_on_f(psr_state == PSR_STATE_INVALID,
-                               "PSR is invalid for connector %s\n", data->output->name);
-                       igt_fail_on_f(psr_state != PSR_STATE3,
-                               "PSR state is expected to be STATE_3 for connector %s\n", data->output->name);
-               }
         }

         /* fini */
--
2.25.1

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20220406/2d3b6897/attachment.htm>