Commit 4dc55525 authored by Laurent Pinchart's avatar Laurent Pinchart

drm: plane: Verify that no or all planes have a zpos property

The zpos property is used by userspace to sort the order of planes.
While the property is not mandatory for drivers to implement, mixing
planes with and without zpos confuses userspace, and shall not be
allowed. Clarify this in the documentation and warn at runtime if the
drivers mixes planes with and without zpos properties.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Acked-by: default avatarSam Ravnborg <sam@ravnborg.org>
Reviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 07de3d31
...@@ -135,7 +135,9 @@ ...@@ -135,7 +135,9 @@
* are underneath planes with higher Z position values. Two planes with the * are underneath planes with higher Z position values. Two planes with the
* same Z position value have undefined ordering. Note that the Z position * same Z position value have undefined ordering. Note that the Z position
* value can also be immutable, to inform userspace about the hard-coded * value can also be immutable, to inform userspace about the hard-coded
* stacking of planes, see drm_plane_create_zpos_immutable_property(). * stacking of planes, see drm_plane_create_zpos_immutable_property(). If
* any plane has a zpos property (either mutable or immutable), then all
* planes shall have a zpos property.
* *
* pixel blend mode: * pixel blend mode:
* Pixel blend mode is set up with drm_plane_create_blend_mode_property(). * Pixel blend mode is set up with drm_plane_create_blend_mode_property().
...@@ -344,10 +346,10 @@ EXPORT_SYMBOL(drm_rotation_simplify); ...@@ -344,10 +346,10 @@ EXPORT_SYMBOL(drm_rotation_simplify);
* should be set to 0 and max to maximal number of planes for given crtc - 1. * should be set to 0 and max to maximal number of planes for given crtc - 1.
* *
* If zpos of some planes cannot be changed (like fixed background or * If zpos of some planes cannot be changed (like fixed background or
* cursor/topmost planes), driver should adjust min/max values and assign those * cursor/topmost planes), drivers shall adjust the min/max values and assign
* planes immutable zpos property with lower or higher values (for more * those planes immutable zpos properties with lower or higher values (for more
* information, see drm_plane_create_zpos_immutable_property() function). In such * information, see drm_plane_create_zpos_immutable_property() function). In such
* case driver should also assign proper initial zpos values for all planes in * case drivers shall also assign proper initial zpos values for all planes in
* its plane_reset() callback, so the planes will be always sorted properly. * its plane_reset() callback, so the planes will be always sorted properly.
* *
* See also drm_atomic_normalize_zpos(). * See also drm_atomic_normalize_zpos().
......
...@@ -289,6 +289,8 @@ EXPORT_SYMBOL(drm_universal_plane_init); ...@@ -289,6 +289,8 @@ EXPORT_SYMBOL(drm_universal_plane_init);
int drm_plane_register_all(struct drm_device *dev) int drm_plane_register_all(struct drm_device *dev)
{ {
unsigned int num_planes = 0;
unsigned int num_zpos = 0;
struct drm_plane *plane; struct drm_plane *plane;
int ret = 0; int ret = 0;
...@@ -297,8 +299,15 @@ int drm_plane_register_all(struct drm_device *dev) ...@@ -297,8 +299,15 @@ int drm_plane_register_all(struct drm_device *dev)
ret = plane->funcs->late_register(plane); ret = plane->funcs->late_register(plane);
if (ret) if (ret)
return ret; return ret;
if (plane->zpos_property)
num_zpos++;
num_planes++;
} }
drm_WARN(dev, num_zpos && num_planes != num_zpos,
"Mixing planes with and without zpos property is invalid\n");
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment