drm/i915: don't set the FBC plane select bits on HSW+
authorPaulo Zanoni <paulo.r.zanoni@intel.com>
Fri, 12 Jun 2015 17:36:21 +0000 (14:36 -0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 15 Jun 2015 16:36:42 +0000 (18:36 +0200)
This commit is just to make the intentions explicit: on HSW+ these
bits are MBZ, but since we only support plane A and the macro
evaluates to zero when plane A is the parameter, we're not fixing any
bug.

v2:
 - Remove useless extra blank like (Chris).
 - Init dpfc_ctl in another place (Chris).

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_fbc.c

index 1ff288c..50ed333 100644 (file)
@@ -262,7 +262,10 @@ static void gen7_fbc_enable(struct drm_crtc *crtc)
 
        dev_priv->fbc.enabled = true;
 
-       dpfc_ctl = IVB_DPFC_CTL_PLANE(intel_crtc->plane);
+       dpfc_ctl = 0;
+       if (IS_IVYBRIDGE(dev))
+               dpfc_ctl |= IVB_DPFC_CTL_PLANE(intel_crtc->plane);
+
        if (drm_format_plane_cpp(fb->pixel_format, 0) == 2)
                dev_priv->fbc.threshold++;