On Thu, 12 Dec 2013 22:30:41 +0000
Chris Wilson <ch...@chris-wilson.co.uk> wrote:

> On Thu, Dec 12, 2013 at 12:41:57PM -0800, Jesse Barnes wrote:
> > Otherwise subsequent fb activity will try to do blank/unblank on outputs
> > that were never fully enabled.
> 
> Hmm, actually this highlights a bug in drm_setup_crtcs() that we do not
> reconstruct enabled[] after a return false here.
> 
> Only the first enabled[i] = false is required where we continue rather
> than abort.

Yeah this one took me awhile to find.  Not surprised if there are other
bugs lurking here.  This one fixes the issue introduced by these
patches at least in the multihead config.

-- 
Jesse Barnes, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to