<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [Topaz] Regression introduces in 4.8-rc3"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98505#c8">Comment # 8</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [Topaz] Regression introduces in 4.8-rc3"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98505">bug 98505</a>
              from <span class="vcard"><a class="email" href="mailto:nayan26deshmukh@gmail.com" title="Nayan Deshmukh <nayan26deshmukh@gmail.com>"> <span class="fn">Nayan Deshmukh</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=127678" name="attach_127678" title="amdgpu patch that checks whether the new interface can be used for PM">attachment 127678</a> <a href="attachment.cgi?id=127678&action=edit" title="amdgpu patch that checks whether the new interface can be used for PM">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=98505&attachment=127678'>[review]</a>
amdgpu patch that checks whether the new interface can be used for PM

Review of <span class=""><a href="attachment.cgi?id=127678" name="attach_127678" title="amdgpu patch that checks whether the new interface can be used for PM">attachment 127678</a> <a href="attachment.cgi?id=127678&action=edit" title="amdgpu patch that checks whether the new interface can be used for PM">[details]</a></span> <a href='page.cgi?id=splinter.html&bug=98505&attachment=127678'>[review]</a>:
-----------------------------------------------------------------

::: drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
@@ +206,5 @@
<span class="quote">>        atpx->is_hybrid = false;
>    if (valid_bits & ATPX_MS_HYBRID_GFX_SUPPORTED) {
>            printk("ATPX Hybrid Graphics\n");
> +          /* Disable legacy PM methods only when pcie port PM is usable. */
> +          atpx->functions.power_cntl = !amdgpu_atpx_priv->bridge_pm_usable;</span >

With this replaced by 
atpx->functions.power_cntl = !amdgpu_atpx_priv.bridge_pm_usable;

The patch fixes the issue. :)</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>