drm/i915: Use skl_plane_ctl() for the SKL "sprite" planes

On SKL the planes are uniform so the "sprites" can use the
primary plane code perfectly fine. The only difference we
have is the color key handling, but since we never enable that
for the primary plane the same code works just fine.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170317211808.14693-3-ville.syrjala@linux.intel.com
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>

1 files changed, 1 insertions, 17 deletions

diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c
index b931d0bd7a64..455a9ed44204 100644
--- a/drivers/gpu/drm/i915/intel_sprite.c
+++ b/drivers/gpu/drm/i915/intel_sprite.c
@@ -232,23 +232,7 @@ skl_update_plane(struct drm_plane *drm_plane,
         uint32_t src_h = drm_rect_height(&plane_state->base.src) >> 16;
         unsigned long irqflags;
 
-        plane_ctl = PLANE_CTL_ENABLE;
-
-        if (!IS_GEMINILAKE(dev_priv)) {
-                plane_ctl |=
-                        PLANE_CTL_PIPE_GAMMA_ENABLE |
-                        PLANE_CTL_PIPE_CSC_ENABLE |
-                        PLANE_CTL_PLANE_GAMMA_DISABLE;
-        }
-
-        plane_ctl |= skl_plane_ctl_format(fb->format->format);
-        plane_ctl |= skl_plane_ctl_tiling(fb->modifier);
-        plane_ctl |= skl_plane_ctl_rotation(rotation);
-
-        if (key->flags & I915_SET_COLORKEY_DESTINATION)
-                plane_ctl |= PLANE_CTL_KEY_ENABLE_DESTINATION;
-        else if (key->flags & I915_SET_COLORKEY_SOURCE)
-                plane_ctl |= PLANE_CTL_KEY_ENABLE_SOURCE;
+        plane_ctl = skl_plane_ctl(crtc_state, plane_state);
 
         /* Sizes are 0 based */
         src_w--;