<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - No package C-states on Dell XPS 15 9570 after resume from deep sleep" href="https://bugs.freedesktop.org/show_bug.cgi?id=111623#c23">Comment # 23</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - No package C-states on Dell XPS 15 9570 after resume from deep sleep" href="https://bugs.freedesktop.org/show_bug.cgi?id=111623">bug 111623</a> from <a class="email" href="mailto:don.hiatt@intel.com" title="Don Hiatt <don.hiatt@intel.com>"> Don Hiatt</a> <pre>(In reply to Tomas Janousek from <a href="show_bug.cgi?id=111623#c22">comment #22</a>) > Created <a href="attachment.cgi?id=145808" name="attach_145808" title="v5.3.7-resume-without-guc-submission.patch">attachment 145808</a> <a href="attachment.cgi?id=145808&action=edit" title="v5.3.7-resume-without-guc-submission.patch">[details]</a> <a href='page.cgi?id=splinter.html&bug=111623&attachment=145808'>[review]</a> [review] > v5.3.7-resume-without-guc-submission.patch > > (In reply to Don Hiatt from <a href="show_bug.cgi?id=111623#c21">comment #21</a>) > > Pretty much what I concluded, I just posted this for review, can you give it > > a try? Thanks. > > > > <a href="https://patchwork.freedesktop.org/patch/337552/">https://patchwork.freedesktop.org/patch/337552/</a> > > I backported this to 5.3.7 (attached) and yeah, it does fix the issue. I > disagree with the patch description, though: the function is definitely > being called with enable_guc=-1, otherwise I wouldn't need this fix. > > (And it does seem a bit weird to skip resume but not skip suspend, but I > have no idea what this non-free blobby whatever does, so I guess I'll leave > that up to you guys.) Thanks for test this, Tomas. Yeah, I'll fix up the commit message.</pre> </div> <hr> You are receiving this mail because: <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>