drm/exynos: add property for plane zpos

The exynos drm driver used a specific ioctl - DRM_EXYNOS_PLANE_SET_ZPOS
to set zpos of plane. It can be substitute to property of plane. This
patch adds a property for plane zpos and removes
DRM_EXYNOS_PLANE_SET_ZPOS ioctl.

Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Inki Dae <inki.dae@samsung.com>

1 files changed, 47 insertions, 45 deletions

diff --git a/drivers/gpu/drm/exynos/exynos_drm_plane.c b/drivers/gpu/drm/exynos/exynos_drm_plane.c
index 232e323d93c8..f018c9d32639 100644
--- a/drivers/gpu/drm/exynos/exynos_drm_plane.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_plane.c
@@ -150,7 +150,6 @@ static int exynos_disable_plane(struct drm_plane *plane)
                         exynos_drm_encoder_plane_disable);
 
         exynos_plane->enabled = false;
-        exynos_plane->overlay.zpos = DEFAULT_ZPOS;
 
         return 0;
 }
@@ -166,26 +165,66 @@ static void exynos_plane_destroy(struct drm_plane *plane)
         kfree(exynos_plane);
 }
 
+static int exynos_plane_set_property(struct drm_plane *plane,
+                                     struct drm_property *property,
+                                     uint64_t val)
+{
+        struct drm_device *dev = plane->dev;
+        struct exynos_plane *exynos_plane = to_exynos_plane(plane);
+        struct exynos_drm_private *dev_priv = dev->dev_private;
+
+        DRM_DEBUG_KMS("[%d] %s\n", __LINE__, __func__);
+
+        if (property == dev_priv->plane_zpos_property) {
+                exynos_plane->overlay.zpos = val;
+                return 0;
+        }
+
+        return -EINVAL;
+}
+
 static struct drm_plane_funcs exynos_plane_funcs = {
         .update_plane   = exynos_update_plane,
         .disable_plane  = exynos_disable_plane,
         .destroy        = exynos_plane_destroy,
+        .set_property   = exynos_plane_set_property,
 };
 
+static void exynos_plane_attach_zpos_property(struct drm_plane *plane)
+{
+        struct drm_device *dev = plane->dev;
+        struct exynos_drm_private *dev_priv = dev->dev_private;
+        struct drm_property *prop;
+
+        DRM_DEBUG_KMS("[%d] %s\n", __LINE__, __func__);
+
+        prop = dev_priv->plane_zpos_property;
+        if (!prop) {
+                prop = drm_property_create_range(dev, 0, "zpos", 0,
+                                                 MAX_PLANE - 1);
+                if (!prop)
+                        return;
+
+                dev_priv->plane_zpos_property = prop;
+        }
+
+        drm_object_attach_property(&plane->base, prop, 0);
+}
+
 struct drm_plane *exynos_plane_init(struct drm_device *dev,
                                     unsigned int possible_crtcs, bool priv)
 {
         struct exynos_plane *exynos_plane;
         int err;
 
+        DRM_DEBUG_KMS("[%d] %s\n", __LINE__, __func__);
+
         exynos_plane = kzalloc(sizeof(struct exynos_plane), GFP_KERNEL);
         if (!exynos_plane) {
                 DRM_ERROR("failed to allocate plane\n");
                 return NULL;
         }
 
-        exynos_plane->overlay.zpos = DEFAULT_ZPOS;
-
         err = drm_plane_init(dev, &exynos_plane->base, possible_crtcs,
                               &exynos_plane_funcs, formats, ARRAY_SIZE(formats),
                               priv);
@@ -195,47 +234,10 @@ struct drm_plane *exynos_plane_init(struct drm_device *dev,
                 return NULL;
         }
 
-        return &exynos_plane->base;
-}
-
-int exynos_plane_set_zpos_ioctl(struct drm_device *dev, void *data,
-                                struct drm_file *file_priv)
-{
-        struct drm_exynos_plane_set_zpos *zpos_req = data;
-        struct drm_mode_object *obj;
-        struct drm_plane *plane;
-        struct exynos_plane *exynos_plane;
-        int ret = 0;
-
-        DRM_DEBUG_KMS("[%d] %s\n", __LINE__, __func__);
-
-        if (!drm_core_check_feature(dev, DRIVER_MODESET))
-                return -EINVAL;
+        if (priv)
+                exynos_plane->overlay.zpos = DEFAULT_ZPOS;
+        else
+                exynos_plane_attach_zpos_property(&exynos_plane->base);
 
-        if (zpos_req->zpos < 0 || zpos_req->zpos >= MAX_PLANE) {
-                if (zpos_req->zpos != DEFAULT_ZPOS) {
-                        DRM_ERROR("zpos not within limits\n");
-                        return -EINVAL;
-                }
-        }
-
-        mutex_lock(&dev->mode_config.mutex);
-
-        obj = drm_mode_object_find(dev, zpos_req->plane_id,
-                        DRM_MODE_OBJECT_PLANE);
-        if (!obj) {
-                DRM_DEBUG_KMS("Unknown plane ID %d\n",
-                              zpos_req->plane_id);
-                ret = -EINVAL;
-                goto out;
-        }
-
-        plane = obj_to_plane(obj);
-        exynos_plane = to_exynos_plane(plane);
-
-        exynos_plane->overlay.zpos = zpos_req->zpos;
-
-out:
-        mutex_unlock(&dev->mode_config.mutex);
-        return ret;
+        return &exynos_plane->base;
 }