<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [SKL] 5k tiled display issues (dual DP)" href="https://bugs.freedesktop.org/show_bug.cgi?id=95207#c19">Comment # 19</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [SKL] 5k tiled display issues (dual DP)" href="https://bugs.freedesktop.org/show_bug.cgi?id=95207">bug 95207</a> from <a class="email" href="mailto:bugs@bzatek.net" title="Tomas Bzatek <bugs@bzatek.net>"> Tomas Bzatek</a> <pre>(In reply to Dave Airlie from <a href="show_bug.cgi?id=95207#c18">comment #18</a>) > btw can I add Signed-off-by: to the two patches I took from here for you? Sure, no problem. Btw. I've noticed minor detail, since the detailed 1 timings live in a separate function now - the block id 0x03 is reported as unhandled: > [ 0.751759] [drm:drm_parse_display_id] block id 0x3, rev 0, len 60 > [ 0.751760] [drm:drm_parse_display_id] found DisplayID tag 0x3, unhandled > [ 0.751760] [drm:drm_parse_display_id] block id 0x7, rev 0, len 10 > [ 0.751761] [drm:drm_parse_display_id] found DisplayID tag 0x7, unhandled You may probably want to add empty DATA_BLOCK_TYPE_1_DETAILED_TIMING case to the drm_parse_display_id() switch. I originally raised the printk() to DRM_DEBUG_KMS() so that we know there's something more reported by the panel that we don't parse. This doesn't affect functionality though.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are on the CC list for the bug.</li> <li>You are the QA Contact for the bug.</li> <li>You are the assignee for the bug.</li> </ul> </body> </html>