On Fri, 24 Jun 2011 10:32:16 +0100, Chris Wilson <ch...@chris-wilson.co.uk> 
wrote:
> So here goes -depth 30...

So other than a couple of rendering paths where we write 8bpc data into
the depth-30 fb (go flash, go!), X looks good. Textured XVideo should just
work, GLX doesn't advertise a 10-bpc visual or fbconfig, so will only work
if we can use a render copy for copy-swaps.

However, I did need to:

diff --git a/drivers/gpu/drm/drm_crtc_helper.c 
b/drivers/gpu/drm/drm_crtc_helper.c
index 9236965..ef4f29c 100644
--- a/drivers/gpu/drm/drm_crtc_helper.c
+++ b/drivers/gpu/drm/drm_crtc_helper.c
@@ -560,7 +560,9 @@ int drm_crtc_helper_set_config(struct drm_mode_set *set)
                        mode_changed = true;
                } else if (set->fb == NULL) {
                        mode_changed = true;
-               } else
+               } else if (set->fb->depth != set->crtc->fb->depth)
+                       mode_changed = true;
+               else
                        fb_changed = true;
        }
 
for the kernel to spot the mode switch when starting X. Lack of logging
ftl. ;-)

For confirmation:

PIPEACONF: 0xc0000020 (enabled, active, 10bpc)
TRANSACONF: 0xc0000020 (enable, active)
FDI_RXA_CTL: 0xb0192050 (enable, train pattern not train, port width X4, 
10bpc,link_reverse_strap_overwrite no, dmi_link_reverse no, FDI PLL enable,FS 
ecc disable, FE ecc disable, FS err report disable, FE err report 
disable,scrambing enable, enhanced framing enable, PCDClk)


Tested-by: Chris Wilson <ch...@chris-wilson.co.uk>
But for my review by, I want to see some logging and validation ;-)
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to