<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <style type="text/css" style="display:none;"></style> </head> <body dir="ltr"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr"> <p style="margin-top:0;margin-bottom:0">Upon closer inspection, this does not match any FDO bug. The FDO bug I thought this was related to was fixed by Michel Danzer, and this fixes an entirely separate issue.</p> </div> <hr style="display:inline-block;width:98%" tabindex="-1"> <div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Michel Dänzer <michel@daenzer.net><br> <b>Sent:</b> July 12, 2018 10:18:34 AM<br> <b>To:</b> Wentland, Harry; Francis, David; amd-gfx@lists.freedesktop.org<br> <b>Subject:</b> Re: [PATCH] amd/dc/dce100: On dce100, set clocks to 0 on suspend</font> <div> </div> </div> <div class="BodyFragment"><font size="2"><span style="font-size:11pt;"> <div class="PlainText">On 2018-07-12 04:13 PM, Harry Wentland wrote:<br> > On 2018-07-12 10:07 AM, David Francis wrote:<br> >> [Why]<br> >> When a dce100 asic was suspended, the clocks were not set to 0.<br> >> Upon resume, the new clock was compared to the existing clock,<br> >> they were found to be the same, and so the clock was not set.<br> >> This resulted in a pernicious blackscreen.<br> >><br> >> [How]<br> >> In atomic commit, check to see if there are any active pipes.<br> >> If no, set clocks to 0<br> >><br> > <br> > Please add Fixes: tag with link to freedesktop bugzilla bug that this fixes.<br> <br> That would be<br> <br> Bugzilla: <a href="https://">https://</a>...<br> <br> (the Fixes tag is for referencing another commit which is fixed by this one)<br> <br> <br> -- <br> Earthling Michel Dänzer | <a href="http://www.amd.com"> http://www.amd.com</a><br> Libre software enthusiast | Mesa and X developer<br> </div> </span></font></div> </body> </html>