drm/i915: clarify gen2 pageflip cmd
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 20 Jan 2012 09:43:44 +0000 (10:43 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Sat, 21 Jan 2012 16:09:05 +0000 (17:09 +0100)
I've reviewed gen2 pageflip code to hunt down multiple prepare pageflip
issues. The only thing I've found is a slight but functionally
meaningless confusion about the length of the mi cmd.

Fix it up and add a comment about what this dword should be (according
to docs at least).

Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c

index ebe71eda9546f75cc4159f4fb743afca4bd4f6fc..4247a7b1823b4296f2ba7b690040b4da9131de8a 100644 (file)
@@ -7288,7 +7288,7 @@ static int intel_gen2_queue_flip(struct drm_device *dev,
                 MI_DISPLAY_FLIP_PLANE(intel_crtc->plane));
        OUT_RING(fb->pitches[0]);
        OUT_RING(obj->gtt_offset + offset);
-       OUT_RING(MI_NOOP);
+       OUT_RING(0); /* aux display base address, unused */
        ADVANCE_LP_RING();
 out:
        return ret;