Re: [Intel-gfx] [PATCH] drm/i915: fixup the plane-pipe fixup code

2012-09-12 Thread Daniel Vetter
On Tue, Sep 11, 2012 at 08:17:33AM +0200, Daniel Vetter wrote: We need to check whether the _other plane is on our pipe, not whether our plane is on the other pipe. Otherwise if not both pipes/planes are active, we won't properly clean up the mess and set up our desired plane-pipe mapping.

[Intel-gfx] [PATCH] drm/i915: fixup the plane-pipe fixup code

2012-09-11 Thread Daniel Vetter
We need to check whether the _other plane is on our pipe, not whether our plane is on the other pipe. Otherwise if not both pipes/planes are active, we won't properly clean up the mess and set up our desired plane-pipe mapping. v2: Fixup the logic, I've totally fumbled it. Noticed by Chris

[Intel-gfx] [PATCH] drm/i915: fixup the plane-pipe fixup code

2012-09-08 Thread Daniel Vetter
We need to check whether the _other plane is on our pipe, not whether our plane is on the other pipe. Otherwise if not both pipes/planes are active, we won't properly clean up the mess and set up our desired plane-pipe mapping. v2: Fixup the logic, I've totally fumbled it. Noticed by Chris

Re: [Intel-gfx] [PATCH] drm/i915: fixup the plane-pipe fixup code

2012-09-08 Thread Chris Wilson
On Sat, 8 Sep 2012 18:14:52 +0200, Daniel Vetter daniel.vet...@ffwll.ch wrote: We need to check whether the _other plane is on our pipe, not whether our plane is on the other pipe. Otherwise if not both pipes/planes are active, we won't properly clean up the mess and set up our desired

Re: [Intel-gfx] [PATCH] drm/i915: fixup the plane-pipe fixup code

2012-09-07 Thread Chris Wilson
On Thu, 6 Sep 2012 22:09:35 +0200, Daniel Vetter daniel.vet...@ffwll.ch wrote: We need to check whether the _other plane is on our pipe, not whether our plane is on the other pipe. Otherwise if not both pipes/planes are active, we won't properly clean up the mess and set up our desired

[Intel-gfx] [PATCH] drm/i915: fixup the plane-pipe fixup code

2012-09-06 Thread Daniel Vetter
We need to check whether the _other plane is on our pipe, not whether our plane is on the other pipe. Otherwise if not both pipes/planes are active, we won't properly clean up the mess and set up our desired plane-pipe mapping. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=51265