[drm-misc-next,v5,4/4] drm/todo: remove task to rename CMA helpers

Commit Message

Danilo Krummrich July 20, 2022, 3:35 p.m. UTC

Both, GEM and FB, CMA helpers were renamed to "GEM DMA" and "FB DMA",
hence the task can be removed.

Acked-by: Thomas Zimmermann <[e-mail]>
Reviewed-by: Laurent Pinchart <[e-mail]>
Signed-off-by: Danilo Krummrich <[e-mail]>
---
 Documentation/gpu/todo.rst | 13 -------------
 1 file changed, 13 deletions(-)

Comments

Sam Ravnborg July 20, 2022, 5:29 p.m. UTC | #1

On Wed, Jul 20, 2022 at 05:35:32PM +0200, Danilo Krummrich wrote:
> Both, GEM and FB, CMA helpers were renamed to "GEM DMA" and "FB DMA",
> hence the task can be removed.
> 
> Acked-by: Thomas Zimmermann <[e-mail]>
> Reviewed-by: Laurent Pinchart <[e-mail]>
> Signed-off-by: Danilo Krummrich <[e-mail]>

It is good to see that someone picks up a task fro the todo list!
Thanks for doing this tedious job.

Acked-by: Sam Ravnborg <[e-mail]>

Patch

diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
index 10bfb50908d1..fd5b3f2fb19e 100644
--- a/Documentation/gpu/todo.rst
+++ b/Documentation/gpu/todo.rst
@@ -343,19 +343,6 @@  converted, except for struct drm_driver.gem_prime_mmap.
 
 Level: Intermediate
 
-Rename CMA helpers to DMA helpers
----------------------------------
-
-CMA (standing for contiguous memory allocator) is really a bit an accident of
-what these were used for first, a much better name would be DMA helpers. In the
-text these should even be called coherent DMA memory helpers (so maybe CDM, but
-no one knows what that means) since underneath they just use dma_alloc_coherent.
-
-Contact: Laurent Pinchart, Daniel Vetter
-
-Level: Intermediate (mostly because it is a huge tasks without good partial
-milestones, not technically itself that challenging)
-
 connector register/unregister fixes
 -----------------------------------