<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEEDINFO " title="NEEDINFO - [BAT eDP PSR] *ERROR* TIMEOUT: Sink CRC counter is not zeroed" href="https://bugs.freedesktop.org/show_bug.cgi?id=93694#c5">Comment # 5</a> on <a class="bz_bug_link bz_status_NEEDINFO " title="NEEDINFO - [BAT eDP PSR] *ERROR* TIMEOUT: Sink CRC counter is not zeroed" href="https://bugs.freedesktop.org/show_bug.cgi?id=93694">bug 93694</a> from <a class="email" href="mailto:daniel@ffwll.ch" title="Daniel Vetter <daniel@ffwll.ch>"> Daniel Vetter</a> <pre>(In reply to Rodrigo Vivi from <a href="show_bug.cgi?id=93694#c4">comment #4</a>) > Oh, this is odd since the first check on this path is for the dpcd bit that > tells sink crc is supported. > But anyway I'm going to tune down it to DRM_DEBUG_KMS and probably remove > the "_basic" from sink crc since it isn't a basic feature or functionality, > but just used for testing another features like psr and fbc. what do you > think? It's basic validation feature, if it doesn't work we can't validate the feature (PSR) itself. That's why imo it must be in BAT. Same reason we have a basic testcase for pipe CRC sanity in BAT, just to make sure that feature tests can give correct results. wrt tuning down another option is that the kernel should maybe give the testcase more information (like return -ENOTTY to signal sink crc isn't working/present, so that the testcase can skip correctly.</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>