Enabled has_pc8 global for ICL and Gen9. Added PC8+ residency test for display enabled case as well. Signed-off-by: Anshuman Gupta <anshuman.gupta@xxxxxxxxx> --- tests/pm_rpm.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c index be296f5..c84f199 100644 --- a/tests/pm_rpm.c +++ b/tests/pm_rpm.c @@ -686,7 +686,8 @@ static void setup_pc8(void) has_pc8 = false; /* Only Haswell supports the PC8 feature. */ - if (!IS_HASWELL(ms_data.devid) && !IS_BROADWELL(ms_data.devid)) + if (!IS_HASWELL(ms_data.devid) && !IS_BROADWELL(ms_data.devid) + && !IS_GEN9(ms_data.devid) && !IS_ICELAKE(ms_data.devid)) return; /* Make sure our Kernel supports MSR and the module is loaded. */ @@ -806,11 +807,15 @@ static void pc8_residency_subtest(void) igt_assert_f(pc8_plus_residency_changed(30), "Machine is not reaching PC8+ states, please check its " "configuration.\n"); - - /* Make sure PC8+ residencies stop! */ enable_one_screen(&ms_data); - igt_assert_f(!pc8_plus_residency_changed(10), + if (IS_HASWELL(ms_data.devid) && IS_BROADWELL(ms_data.devid)) { + /* Make sure PC8+ residencies stop! */ + igt_assert_f(!pc8_plus_residency_changed(10), + "PC8+ residency didn't stop with screen enabled.\n"); + } else { + igt_assert_f(pc8_plus_residency_changed(10), "PC8+ residency didn't stop with screen enabled.\n"); + } } static void modeset_subtest(enum screen_type type, int rounds, int wait_flags) -- 2.7.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx