[PATCH 5/5] drm/todo: Add entry to rename drm_atomic_state
Maxime Ripard
mripard at kernel.org
Mon Dec 4 12:17:07 UTC 2023
The name of the structure drm_atomic_state is confusing. Let's add an
entry to our todo list to rename it.
Signed-off-by: Maxime Ripard <mripard at kernel.org>
---
Documentation/gpu/todo.rst | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
index b62c7fa0c2bc..fe95aea89d67 100644
--- a/Documentation/gpu/todo.rst
+++ b/Documentation/gpu/todo.rst
@@ -120,6 +120,24 @@ Contact: Daniel Vetter, respective driver maintainers
Level: Advanced
+Rename drm_atomic_state
+-----------------------
+
+The KMS framework uses two slightly different definitions for the ``state``
+concept. For a given object (plane, CRTC, encoder, etc., so
+``drm_$OBJECT_state``), the state is the entire state of that object. However,
+at the device level, ``drm_atomic_state`` refers to a state update for a
+limited number of objects.
+
+The state isn't the entire device state anymore, but only the full state of
+some objects in that device. This is confusing to newcomers, and
+``drm_atomic_state`` should be renamed to something clearer like
+``drm_atomic_update``.
+
+Contact: Maxime Ripard <mripard at kernel.org>
+
+Level: Advanced
+
Fallout from atomic KMS
-----------------------
--
2.43.0
More information about the dri-devel
mailing list