diff options
| author | Paulo Zanoni | 2015-02-09 17:46:29 +0100 |
|---|---|---|
| committer | Daniel Vetter | 2015-02-13 23:28:15 +0100 |
| commit | e35fef211bab1a69da1b392c3ece906e287ed6e4 (patch) | |
| tree | 1e609b2514add70cbc91073f6b7e0159a1194ada /drivers/gpu/drm/i915/intel_sprite.c | |
| parent | drm/i915: don't keep reassigning FBC_UNSUPPORTED (diff) | |
| download | kernel-qcow2-linux-e35fef211bab1a69da1b392c3ece906e287ed6e4.tar.gz kernel-qcow2-linux-e35fef211bab1a69da1b392c3ece906e287ed6e4.tar.xz kernel-qcow2-linux-e35fef211bab1a69da1b392c3ece906e287ed6e4.zip | |
drm/i915: change dev_priv->fbc.plane to dev_priv->fbc.crtc
Since the mapping from CRTCs to planes is fixed, looking at the CRTC
is essentially the same as looking at the plane. Also, the next
patches wil start using the frontbuffer_bits macros, and they take the
pipe as the parameter instead of the plane, and this could differ on
gens 2 and 3.
Another nice thing is that we don't risk accidentally initializing
things to PLANE_A if we don't set the value before it is used for the
first time. But this shouldn't be a problem with the current code.
V2: Rebase.
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com> (v1)
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_sprite.c')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_sprite.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c index 0a52c44ad03d..5afc89e00a2a 100644 --- a/drivers/gpu/drm/i915/intel_sprite.c +++ b/drivers/gpu/drm/i915/intel_sprite.c @@ -993,7 +993,7 @@ intel_pre_disable_primary(struct drm_crtc *crtc) struct intel_crtc *intel_crtc = to_intel_crtc(crtc); mutex_lock(&dev->struct_mutex); - if (dev_priv->fbc.plane == intel_crtc->plane) + if (dev_priv->fbc.crtc == intel_crtc) intel_fbc_disable(dev); mutex_unlock(&dev->struct_mutex); |