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

David Zhang dingchen.zhang at amd.com
Wed Apr 6 20:28:29 UTC 2022


[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



More information about the igt-dev mailing list