diff options
author | José Roberto de Souza <jose.souza@intel.com> | 2019-11-27 16:49:02 -0800 |
---|---|---|
committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-12-11 16:30:06 +0200 |
commit | 9e0c82b6d70065db894ececf2be8de372e983cf0 (patch) | |
tree | 47c63b4482d97b26860ea5f05993c0a3d4923f6e | |
parent | 3b938d9254be900e7ff205dcd8cd88b7624840f3 (diff) |
tests/i915_pm_dc: Wait for PSR2 entry
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@intel.com>
Cc: Anshuman Gupta <anshuman.gupta@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
-rw-r--r-- | tests/i915/i915_pm_dc.c | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index 1b46bbbc..da5c1c64 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"); } |