<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body><span class="vcard"><a class="email" href="mailto:harish.chegondi@intel.com" title="harish.chegondi@intel.com">harish.chegondi@intel.com</a>
</span> changed
<a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [CI][DRMTIP] igt@pm_backlight@* - incomplete - system hang?"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107820">bug 107820</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Resolution</td>
<td>---
</td>
<td>FIXED
</td>
</tr>
<tr>
<td style="text-align:right;">Status</td>
<td>ASSIGNED
</td>
<td>RESOLVED
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [CI][DRMTIP] igt@pm_backlight@* - incomplete - system hang?"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107820#c10">Comment # 10</a>
on <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [CI][DRMTIP] igt@pm_backlight@* - incomplete - system hang?"
href="https://bugs.freedesktop.org/show_bug.cgi?id=107820">bug 107820</a>
from <span class="vcard"><a class="email" href="mailto:harish.chegondi@intel.com" title="harish.chegondi@intel.com">harish.chegondi@intel.com</a>
</span></b>
<pre>The following commit has been pushed into the master branch of igt-gpu-tools
commit 8770e24fe4008404da769c16b7edac6142225ad7 (HEAD -> master)
Author: Harish Chegondi <<a href="mailto:harish.chegondi@intel.com">harish.chegondi@intel.com</a>>
Date: Wed Apr 3 18:26:30 2019 -0700
i915/pm_backlight: Do not turn off DPMS before system suspend
backlight fade with suspend test turns off dpms which turns off the edp
backlight. Then it does a system suspend and resume. After resume,
the edp backlight would still be off, but the test sets the brightness
value and reads it back. Since the edp backlight is off, the brightness
values written and read are different causing the test to fail.
Do not turn off the DPMS before suspend so that after system resume,
the edp backlight would be on and setting the brightness value would
be successful.
v2: Remove "DPMS off" before system suspend instead of adding
"DPMS on" after system resume.
Cc: Jyoti Yadav <<a href="mailto:jyoti.r.yadav@intel.com">jyoti.r.yadav@intel.com</a>>
Cc: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
Cc: Daniel Vetter <<a href="mailto:daniel.vetter@intel.com">daniel.vetter@intel.com</a>>
Cc: Clinton Taylor <<a href="mailto:clinton.a.taylor@intel.com">clinton.a.taylor@intel.com</a>>
Signed-off-by: Harish Chegondi <<a href="mailto:harish.chegondi@intel.com">harish.chegondi@intel.com</a>>
References: <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - [CI][DRMTIP] igt@pm_backlight@* - incomplete - system hang?"
href="show_bug.cgi?id=107820">https://bugs.freedesktop.org/show_bug.cgi?id=107820</a>
Fixes: 377752242995 ("Brightness test with DPMS and System suspend.")
Reviewed-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>>
Reviewed-by: José Roberto de Souza <<a href="mailto:jose.souza@intel.com">jose.souza@intel.com</a>></pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the QA Contact for the bug.</li>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>