Enabled has_pc8 global for ICL and Gen9.
Added PC8+ residency test for display enabled case as well.

Signed-off-by: Anshuman Gupta <[email protected]>
---
 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
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to