<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p><br>
</p>
<div class="moz-cite-prefix">On 05-05-2020 18:50, Anshuman Gupta
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:20200505132032.827-2-anshuman.gupta@intel.com">
<pre class="moz-quote-pre" wrap="">panel-fitter subtest is only applicable to HASWELL/BROADWELL.
It just tests non-lpsp(lpsp disabled) when panel fitter is
enforced by std 1024x768 mode.
There is no ROI to test non-lpsp kind of thing,
If HSW_GLOBAL power well is not enabled we would see
other display issues in other display tests.
Gen9+ platform has display scaling feature in built per
pipe, as lpsp is all about low power single pipe (PIPE A)
usage, gen9+ platforms also don't require panel-fitter subtest.
This should save some CI cycles.
Signed-off-by: Anshuman Gupta <a class="moz-txt-link-rfc2396E" href="mailto:anshuman.gupta@intel.com"><anshuman.gupta@intel.com></a></pre>
</blockquote>
<pre>Changes looks good to me.
Reviewed-by: Animesh Manna <a class="moz-txt-link-rfc2396E" href="mailto:animesh.manna@intel.com"><animesh.manna@intel.com></a>
</pre>
<blockquote type="cite"
cite="mid:20200505132032.827-2-anshuman.gupta@intel.com">
<pre class="moz-quote-pre" wrap="">
---
tests/i915/i915_pm_lpsp.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/i915/i915_pm_lpsp.c b/tests/i915/i915_pm_lpsp.c
index 42938e10..08f82e7c 100644
--- a/tests/i915/i915_pm_lpsp.c
+++ b/tests/i915/i915_pm_lpsp.c
@@ -219,8 +219,6 @@ igt_main
screens_disabled_subtest(drm_fd, drm_res);
igt_subtest("edp-native")
edp_subtest(drm_fd, drm_res, drm_connectors, devid, false);
- igt_subtest("edp-panel-fitter")
- edp_subtest(drm_fd, drm_res, drm_connectors, devid, true);
igt_subtest("non-edp")
non_edp_subtest(drm_fd, drm_res, drm_connectors);
</pre>
</blockquote>
</body>
</html>