[PATCH v3] drm/omap: plane zpos/zorder management improvements

[Peter Ujfalusi <peter.ujfalusi@xxxxxx>]

Use the plane index as default zpos for all planes. Even if the
application is not setting zpos/zorder explicitly we will have unique zpos
for each plane.

Enforce that all planes must have unique zpos on the given crtc.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@xxxxxx>
---
Hi,

Changes since v2:
- The check for duplicate zpos is moved to omap_crtc

Changes since v1:
- Dropped the zpos normalization related patches
- New patch based on the discussion, see commit message.

Regards,
Peter

 drivers/gpu/drm/omapdrm/omap_crtc.c  | 24 +++++++++++++++++++++++-
 drivers/gpu/drm/omapdrm/omap_plane.c | 10 +++++-----
 2 files changed, 28 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/omapdrm/omap_crtc.c b/drivers/gpu/drm/omapdrm/omap_crtc.c
index 1b8154e58d18..ff0b17f8bb76 100644
--- a/drivers/gpu/drm/omapdrm/omap_crtc.c
+++ b/drivers/gpu/drm/omapdrm/omap_crtc.c
@@ -486,6 +486,28 @@ static void omap_crtc_mode_set_nofb(struct drm_crtc *crtc)
 	}
 }
 
+static int omap_crtc_validate_zpos(struct drm_crtc *crtc,
+				   struct drm_crtc_state *state)
+{
+	struct drm_plane *p1, *p2;
+	const struct drm_plane_state *pstate1, *pstate2;
+
+	drm_atomic_crtc_state_for_each_plane_state(p1, pstate1, state) {
+		drm_atomic_crtc_state_for_each_plane_state(p2, pstate2, state) {
+			if (drm_plane_index(p1) == drm_plane_index(p2))
+				continue;
+
+			if (pstate1->zpos == pstate2->zpos) {
+				DBG("crtc%u: zpos must be unique (zpos: %u)",
+				    crtc->index, pstate1->zpos);
+				return -EINVAL;
+			}
+		}
+	}
+
+	return 0;
+}
+
 static int omap_crtc_atomic_check(struct drm_crtc *crtc,
 				struct drm_crtc_state *state)
 {
@@ -509,7 +531,7 @@ static int omap_crtc_atomic_check(struct drm_crtc *crtc,
 		omap_crtc_state->rotation = pri_state->rotation;
 	}
 
-	return 0;
+	return omap_crtc_validate_zpos(crtc, state);
 }
 
 static void omap_crtc_atomic_begin(struct drm_crtc *crtc,
diff --git a/drivers/gpu/drm/omapdrm/omap_plane.c b/drivers/gpu/drm/omapdrm/omap_plane.c
index 7d789d1551a1..c1f93bfae7a5 100644
--- a/drivers/gpu/drm/omapdrm/omap_plane.c
+++ b/drivers/gpu/drm/omapdrm/omap_plane.c
@@ -31,6 +31,7 @@
 struct omap_plane {
 	struct drm_plane base;
 	enum omap_plane_id id;
+	int idx;
 	const char *name;
 };
 
@@ -97,8 +98,7 @@ static void omap_plane_atomic_disable(struct drm_plane *plane,
 	struct omap_plane *omap_plane = to_omap_plane(plane);
 
 	plane->state->rotation = DRM_MODE_ROTATE_0;
-	plane->state->zpos = plane->type == DRM_PLANE_TYPE_PRIMARY
-			   ? 0 : omap_plane->id;
+	plane->state->zpos = omap_plane->idx;
 
 	priv->dispc_ops->ovl_enable(omap_plane->id, false);
 }
@@ -194,8 +194,7 @@ static void omap_plane_reset(struct drm_plane *plane)
 	 * Set the zpos default depending on whether we are a primary or overlay
 	 * plane.
 	 */
-	plane->state->zpos = plane->type == DRM_PLANE_TYPE_PRIMARY
-			   ? 0 : omap_plane->id;
+	plane->state->zpos = omap_plane->idx;
 }
 
 static int omap_plane_atomic_set_property(struct drm_plane *plane,
@@ -282,6 +281,7 @@ struct drm_plane *omap_plane_init(struct drm_device *dev,
 	for (nformats = 0; formats[nformats]; ++nformats)
 		;
 	omap_plane->id = id;
+	omap_plane->idx = idx;
 	omap_plane->name = plane_id_to_name[id];
 
 	plane = &omap_plane->base;
@@ -295,7 +295,7 @@ struct drm_plane *omap_plane_init(struct drm_device *dev,
 	drm_plane_helper_add(plane, &omap_plane_helper_funcs);
 
 	omap_plane_install_properties(plane, &plane->base);
-	drm_plane_create_zpos_property(plane, 0, 0, num_planes - 1);
+	drm_plane_create_zpos_property(plane, idx, 0, num_planes - 1);
 
 	return plane;
 
-- 
Peter

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel