diff options
| author | Mastan Katragadda <mastanx.katragadda@intel.com> | 2021-12-01 15:41:49 +0530 |
|---|---|---|
| committer | Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com> | 2021-12-02 15:11:07 +0530 |
| commit | 0933b7ccdb2bb054b6a8154171e35315d84299b7 (patch) | |
| tree | eb83ee4376a559df6867b3b1a39507f29b8726e2 /tests/i915/i915_pm_dc.c | |
| parent | f062f4ae60ecf47af4b037c8f9952a1360662579 (diff) | |
tests/i915/i915_pm_dc: Moved prev DC counter reading logic
Reading of prev_dc counter in dc9-dpms subtest as 0, after reset
comparison with 0 test gets failed. taken prev_dc counter after
enterd into shallow DC state.fixes dc9-dpms subtest failures.
changes since V1:
- Removed new lines and warnings from checkpatch.pl
- Updated Subject
Signed-off-by: Mastan Katragadda <mastanx.katragadda@intel.com>
Reviewed-by: Jigar Bhatt <jigar.bhatt@intel.com>
Diffstat (limited to 'tests/i915/i915_pm_dc.c')
| -rw-r--r-- | tests/i915/i915_pm_dc.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index 4ebeb3eb..2c2c5086 100644 --- a/tests/i915/i915_pm_dc.c +++ b/tests/i915/i915_pm_dc.c @@ -434,22 +434,22 @@ static void setup_dc9_dpms(data_t *data, int dc_target) dpms_off(data); igt_skip_on_f(!(igt_wait(read_dc_counter(data->debugfs_fd, dc_target) > prev_dc, 3000, 100)), "Unable to enters shallow DC states\n"); + prev_dc = read_dc_counter(data->debugfs_fd, dc_target); dpms_on(data); cleanup_dc_dpms(data); + dpms_off(data); + sleep(1); /* wait for counters reset*/ + check_dc9(data, dc_target, prev_dc); + dpms_on(data); } static void test_dc9_dpms(data_t *data) { - int prev_dc, dc_target; + int dc_target; require_dc_counter(data->debugfs_fd, CHECK_DC5); dc_target = support_dc6(data->debugfs_fd) ? CHECK_DC6 : CHECK_DC5; - 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); } static void kms_poll_state_restore(int sig) |