diff options
author | Jyoti Yadav <jyoti.r.yadav@intel.com> | 2019-09-19 23:00:31 +0530 |
---|---|---|
committer | Petri Latvala <petri.latvala@intel.com> | 2019-09-20 13:51:55 +0300 |
commit | 0f1e4a9635f35be751cf183b9dd42017da5697e4 (patch) | |
tree | 8435a6a7a5e76e915dcc6201689874d4738a6424 /tests/i915/i915_pm_dc.c | |
parent | b2da6d45bd17d4a56952ac5fc7d5212231d09140 (diff) |
tests/i915/i915_pm_dc: Added test for DC5 during DPMS
Added new subtest for DC5 entry during DPMS on/off cycle.
During DPMS on/off cycle DC5 counter is incremented.
v2: Rename the subtest with meaningful name.
v3: Rebased.
v4: Addressed review comments by removing leftover code
cleanup().
v5: Addressed the review comment by removing redundant
read_dc_counter() suggested by Imre.
Listing actual change in patch set changelog to make review easier.
v6: Three way patch applied, no functional change.
v7: Disabling runtime suspend for the platform which support, DC9.
rebased due to test name pm_dc changed to i915_pm_dc, aligning to
other PM tests.
v8: Introduced setup_dc_dpms() in order to disable runtime pm, restoring
POWER_DIR values to its original and enabling runtime pm for other
followed sub-tests.
v9: Check DC5 counter value after DPMS off, broke the dpms_on_off
function to dpms_on and dpms_off. [Imre]
v10:Added AT_LEAST_Gen11 condition instead of IS_ICELAKE in order to
disable runtime suspend. [Imre]
v11:Added a cleanup_dc_dpms() function. [Imre]
Signed-off-by: Jyoti Yadav <jyoti.r.yadav@intel.com>
Signed-off-by: Anshuman Gupta <anshuman.gupta@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Diffstat (limited to 'tests/i915/i915_pm_dc.c')
-rw-r--r-- | tests/i915/i915_pm_dc.c | 68 |
1 files changed, 68 insertions, 0 deletions
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index 944618a1..553f8407 100644 --- a/tests/i915/i915_pm_dc.c +++ b/tests/i915/i915_pm_dc.c @@ -44,6 +44,7 @@ typedef struct { enum psr_mode op_psr_mode; drmModeModeInfo *mode; igt_output_t *output; + bool runtime_suspend_disabled; } data_t; static bool dc_state_wait_entry(int drm_fd, int dc_flag, int prev_dc_count); @@ -171,6 +172,67 @@ static void test_dc_state_psr(data_t *data, int dc_flag) cleanup_dc_psr(data); } +static void cleanup_dc_dpms(data_t *data) +{ + /* + * if runtime PM is disabled for i915 restore it, + * so any other sub-test can use runtime-PM. + */ + if (data->runtime_suspend_disabled) { + igt_restore_runtime_pm(); + igt_setup_runtime_pm(); + } +} + +static void setup_dc_dpms(data_t *data) +{ + if (IS_BROXTON(data->devid) || IS_GEMINILAKE(data->devid) || + AT_LEAST_GEN(data->devid, 11)) { + igt_disable_runtime_pm(); + data->runtime_suspend_disabled = true; + } else { + data->runtime_suspend_disabled = false; + } +} + +static void dpms_off(data_t *data) +{ + for (int i = 0; i < data->display.n_outputs; i++) { + kmstest_set_connector_dpms(data->drm_fd, + data->display.outputs[i].config.connector, + DRM_MODE_DPMS_OFF); + } + + if (!data->runtime_suspend_disabled) + igt_assert(igt_wait_for_pm_status + (IGT_RUNTIME_PM_STATUS_SUSPENDED)); +} + +static void dpms_on(data_t *data) +{ + for (int i = 0; i < data->display.n_outputs; i++) { + kmstest_set_connector_dpms(data->drm_fd, + data->display.outputs[i].config.connector, + DRM_MODE_DPMS_ON); + } + + if (!data->runtime_suspend_disabled) + igt_assert(igt_wait_for_pm_status + (IGT_RUNTIME_PM_STATUS_ACTIVE)); +} + +static void test_dc_state_dpms(data_t *data, int dc_flag) +{ + uint32_t dc_counter; + + setup_dc_dpms(data); + dc_counter = read_dc_counter(data->drm_fd, dc_flag); + dpms_off(data); + check_dc_counter(data->drm_fd, dc_flag, dc_counter); + dpms_on(data); + cleanup_dc_dpms(data); +} + IGT_TEST_DESCRIPTION("These tests validate Display Power DC states"); int main(int argc, char *argv[]) { @@ -213,6 +275,12 @@ int main(int argc, char *argv[]) test_dc_state_psr(&data, CHECK_DC6); } + igt_describe("This test validates display engine entry to DC5 state " + "while all connectors's DPMS property set to OFF"); + igt_subtest("dc5-dpms") { + test_dc_state_dpms(&data, CHECK_DC5); + } + igt_fixture { close(data.debugfs_fd); display_fini(&data); |