On Tue, 5 Apr 2011 10:24:14 +0100, Chris Wilson <[email protected]> wrote:
> crtc = intel_get_crtc_for_plane(dev, plane);
> - if (crtc->fb == NULL || !crtc->enabled)
> + if (crtc->fb == NULL || !crtc->enabled) {
> + *cursor_wm = *plane_wm = display->guard_size;
> return false;
> + }
Would it be clearer to have g4x_update_wm set these instead?
I'm also a bit concerned about the default value; it would be lovely to
have the docs say what the value should be for disabled pipes, but I
couldn't find any mention of them.
--
[email protected]
pgpR39ILV8W8b.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
