[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <DB3PR10MB683528B8252ED2A802A0E154E8D8A@DB3PR10MB6835.EURPRD10.PROD.OUTLOOK.COM>
Date: Mon, 23 Oct 2023 22:00:56 +0530
From: Yuran Pereira <yuran.pereira@...mail.com>
To: airlied@...il.com
Cc: Yuran Pereira <yuran.pereira@...mail.com>,
neil.armstrong@...aro.org, sam@...nborg.org, daniel@...ll.ch,
maarten.lankhorst@...ux.intel.com, mripard@...nel.org,
tzimmermann@...e.de, corbet@....net, sumit.semwal@...aro.org,
christian.koenig@....com, dri-devel@...ts.freedesktop.org,
linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-media@...r.kernel.org, linaro-mm-sig@...ts.linaro.org,
dianders@...omium.org,
linux-kernel-mentees@...ts.linuxfoundation.org
Subject: [PATCH] drm: docs: Remove item from TODO list
Since "Clean up checks for already prepared/enabled in panels" has
already been done and merged [1], I think there is no longer a need
for this item to be in the gpu TODO.
[1] https://patchwork.freedesktop.org/patch/551421/
Signed-off-by: Yuran Pereira <yuran.pereira@...mail.com>
---
Documentation/gpu/todo.rst | 25 -------------------------
1 file changed, 25 deletions(-)
diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
index 03fe5d1247be..280020b0ad4d 100644
--- a/Documentation/gpu/todo.rst
+++ b/Documentation/gpu/todo.rst
@@ -460,31 +460,6 @@ Contact: Thomas Zimmermann <tzimmermann@...e.de>
Level: Starter
-Clean up checks for already prepared/enabled in panels
-------------------------------------------------------
-
-In a whole pile of panel drivers, we have code to make the
-prepare/unprepare/enable/disable callbacks behave as no-ops if they've already
-been called. To get some idea of the duplicated code, try::
-
- git grep 'if.*>prepared' -- drivers/gpu/drm/panel
- git grep 'if.*>enabled' -- drivers/gpu/drm/panel
-
-In the patch ("drm/panel: Check for already prepared/enabled in drm_panel")
-we've moved this check to the core. Now we can most definitely remove the
-check from the individual panels and save a pile of code.
-
-In adition to removing the check from the individual panels, it is believed
-that even the core shouldn't need this check and that should be considered
-an error if other code ever relies on this check. The check in the core
-currently prints a warning whenever something is relying on this check with
-dev_warn(). After a little while, we likely want to promote this to a
-WARN(1) to help encourage folks not to rely on this behavior.
-
-Contact: Douglas Anderson <dianders@...omium.org>
-
-Level: Starter/Intermediate
-
Core refactorings
=================
--
2.25.1
Powered by blists - more mailing lists