diff options
author | Jigar Bhatt <jigar.bhatt@intel.com> | 2021-10-13 16:27:44 +0530 |
---|---|---|
committer | Swati Sharma <swati2.sharma@intel.com> | 2021-10-14 21:37:12 +0530 |
commit | 04b156109f1c5128d5ace67420ee2e35e8a24e1d (patch) | |
tree | aee0fcb6cc647640cf4ffe426b1f10118eeb7b5e /tests/i915/i915_pm_dc.c | |
parent | d9a0b0f884e910987af1ca14136bdab9a8f19cab (diff) |
tests/i915/i915_pm_dc: Fix condition in dc9-test
1.Inside dc9_wait_entry, we do not have DC9 counter,
so we rely on dc5/dc6 counter reset
to check if display engine was in DC9.
Test is passed if live counter is less
than the previous counter. Correcting condition
igt_wait(read_dc_counter(debugfs_fd, dc_target) <
prev_dc, seconds, 100);
2.Moving prev_dc = read_dc_counter(data->debugfs_fd,
dc_target); before setup_dc9_dpms avoid 0 reading.
3.Add sleep(1) after dpms_off waiting for reset counters.
Fixes: 2f4e6430a97f04284d1cafb1479e7c1b0b2d596a ("tests/i915/i915_pm_dc: Fix DC9 test")
Signed-off-by: Jigar Bhatt <jigar.bhatt@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Reviewed-by: Nischal Varide<nischal.varide@intel.com>
Diffstat (limited to 'tests/i915/i915_pm_dc.c')
-rw-r--r-- | tests/i915/i915_pm_dc.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index 3a7ce5bf..4ebeb3eb 100644 --- a/tests/i915/i915_pm_dc.c +++ b/tests/i915/i915_pm_dc.c @@ -410,7 +410,7 @@ static bool dc9_wait_entry(uint32_t debugfs_fd, int dc_target, int prev_dc, int * since we do not have DC9 counter, * so we rely on dc5/dc6 counter reset to check if display engine was in DC9. */ - return igt_wait(read_dc_counter(debugfs_fd, dc_target) > + return igt_wait(read_dc_counter(debugfs_fd, dc_target) < prev_dc, seconds, 100); } @@ -444,9 +444,10 @@ static void test_dc9_dpms(data_t *data) require_dc_counter(data->debugfs_fd, CHECK_DC5); dc_target = support_dc6(data->debugfs_fd) ? CHECK_DC6 : CHECK_DC5; - setup_dc9_dpms(data, dc_target); prev_dc = read_dc_counter(data->debugfs_fd, dc_target); + setup_dc9_dpms(data, dc_target); dpms_off(data); + sleep(1); /* wait for counters reset*/ check_dc9(data, dc_target, prev_dc); dpms_on(data); } |