<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [drm:si_dpm_set_power_state] *ERROR* si_disable_ulv failed" href="https://bugs.freedesktop.org/show_bug.cgi?id=82889#c19">Comment # 19</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [drm:si_dpm_set_power_state] *ERROR* si_disable_ulv failed" href="https://bugs.freedesktop.org/show_bug.cgi?id=82889">bug 82889</a> from <a class="email" href="mailto:agd5f@yahoo.com" title="Alex Deucher <agd5f@yahoo.com>"> Alex Deucher</a> <pre>(In reply to Alexandre Demers from <a href="show_bug.cgi?id=82889#c18">comment #18</a>) > > But is ulv support truly supposed to be available on Tahiti? In fact, prior > to your patch, why is there already a comment "/* XXX disable for A0 tahiti > */" in drivers/gpu/drm/radeon/si_dpm.c but ulv.supported is set to true > anyway just on the next line (the one you propose to change in your patch)? > To me, it's like saying a thing and doing exactly the opposite at the same > time, isn't it? Or is it because there is a special case (Tahiti) that we > should be addressing identified by the comment that we are not? A0 is first silicon (basically the initial silicon samples we get back from the fab during bring up). The issue was fixed in later silicon revisions. There usually aren't any A0 boards in the wild.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>