[igt-dev] [PATCH i-g-t 2/2] tests/i915_pm_dc: Wait for PSR2 entry

José Roberto de Souza jose.souza at intel.com
Thu Nov 28 00:49:02 UTC 2019


After enable PSR2 we can not check right away if PSR2 is enabled, we
need let some time to PSR2 to be enabled.
For that we have psr_wait_entry(), that is why the previous patch was
reverted as psr_enabled() is not much useful.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=112411
Cc: Jeevan B <jeevan.b at intel.com>
Cc: Anshuman Gupta <anshuman.gupta at intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler at intel.com>
Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
---
 tests/i915/i915_pm_dc.c | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c
index 02b08189..cd2828b8 100644
--- a/tests/i915/i915_pm_dc.c
+++ b/tests/i915/i915_pm_dc.c
@@ -84,16 +84,6 @@ static void display_fini(data_t *data)
 	igt_display_fini(&data->display);
 }
 
-static bool edp_psr2_enabled(data_t *data)
-{
-	char buf[512];
-
-	igt_debugfs_simple_read(data->debugfs_fd, "i915_edp_psr_status",
-				buf, sizeof(buf));
-
-	return strstr(buf, "PSR mode: PSR2 enabled") != NULL;
-}
-
 static void cleanup_dc_psr(data_t *data)
 {
 	igt_plane_t *primary;
@@ -298,7 +288,7 @@ static void setup_dc3co(data_t *data)
 {
 	data->op_psr_mode = PSR_MODE_2;
 	psr_enable(data->debugfs_fd, data->op_psr_mode);
-	igt_require_f(edp_psr2_enabled(data),
+	igt_require_f(psr_wait_entry(data->debugfs_fd, data->op_psr_mode),
 		      "PSR2 is not enabled\n");
 }
 
-- 
2.24.0



More information about the igt-dev mailing list