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.ujfal...@ti.com>
---
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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
