<html> <head> <base href="https://bugs.freedesktop.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - [HSW HDMI] Audio with 44.1 khz not working when 1920x1080p screen refreshrate > 30hz" href="https://bugs.freedesktop.org/show_bug.cgi?id=97442#c14">Comment # 14</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - [HSW HDMI] Audio with 44.1 khz not working when 1920x1080p screen refreshrate > 30hz" href="https://bugs.freedesktop.org/show_bug.cgi?id=97442">bug 97442</a> from <a class="email" href="mailto:Mark_Wilczynski@hotmail.com" title="Mark Wilczynski <Mark_Wilczynski@hotmail.com>"> Mark Wilczynski</a> <pre>(In reply to Chris Wilson from <a href="show_bug.cgi?id=97442#c11">comment #11</a>) > Created <a href="attachment.cgi?id=126057" name="attach_126057" title="Limit the depth of the display pipeline to the framebuffer">attachment 126057</a> <a href="attachment.cgi?id=126057&action=edit" title="Limit the depth of the display pipeline to the framebuffer">[details]</a> <a href='page.cgi?id=splinter.html&bug=97442&attachment=126057'>[review]</a> [review] > Limit the depth of the display pipeline to the framebuffer > > Compile tested only! If I understand your intent correctly, I think there's a logic bug in this patch. + if (plane_state->fb->depth > bpp) + bpp = 8*3; Shouldn't that say: + if (plane_state->fb->depth < bpp) + bpp = 8*3; That is if the FB bit depth is less than the supported bpp of the display (36-bit in my case), then clamp the bpp to that of the frame buffer (24-bit).</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>