[RFC 2/2] drm/panel: Use generic object registry

Thierry Reding thierry.reding at gmail.com
Tue Nov 4 08:29:28 PST 2014


From: Thierry Reding <treding at nvidia.com>

Convert to the new generic object registry and introduce proper object
and module reference counting. This should make panel registration and
removal a lot more robust.

Signed-off-by: Thierry Reding <treding at nvidia.com>
---
 drivers/gpu/drm/drm_panel.c | 100 ++++++++++++++++++++++++++++++++++----------
 include/drm/drm_panel.h     |  13 ++++--
 2 files changed, 86 insertions(+), 27 deletions(-)

diff --git a/drivers/gpu/drm/drm_panel.c b/drivers/gpu/drm/drm_panel.c
index 3dfe3c886502..c1b58bd421a3 100644
--- a/drivers/gpu/drm/drm_panel.c
+++ b/drivers/gpu/drm/drm_panel.c
@@ -23,13 +23,11 @@
 
 #include <linux/err.h>
 #include <linux/module.h>
+#include <linux/registry.h>
 
 #include <drm/drm_crtc.h>
 #include <drm/drm_panel.h>
 
-static DEFINE_MUTEX(panel_lock);
-static LIST_HEAD(panel_list);
-
 /**
  * DOC: drm panel
  *
@@ -38,6 +36,33 @@ static LIST_HEAD(panel_list);
  * drivers.
  */
 
+/*
+ * DRM panel registry
+ */
+static struct registry panels = {
+	.lock = __MUTEX_INITIALIZER(panels.lock),
+	.list = LIST_HEAD_INIT(panels.list),
+	.owner = THIS_MODULE,
+};
+
+static inline struct drm_panel *to_drm_panel(struct registry_record *record)
+{
+	return container_of(record, struct drm_panel, record);
+}
+
+static void drm_panel_release(struct registry_record *record)
+{
+	struct drm_panel *panel = to_drm_panel(record);
+
+	/*
+	 * The .release() callback is optional because drivers may not need
+	 * to manually release any resources (e.g. if they've used devm_*()
+	 * helper functions).
+	 */
+	if (panel->funcs && panel->funcs->release)
+		panel->funcs->release(panel);
+}
+
 /**
  * drm_panel_init - initialize a panel
  * @panel: DRM panel
@@ -47,11 +72,48 @@ static LIST_HEAD(panel_list);
  */
 void drm_panel_init(struct drm_panel *panel)
 {
-	INIT_LIST_HEAD(&panel->list);
+	registry_record_init(&panel->record);
+	panel->record.release = drm_panel_release;
 }
 EXPORT_SYMBOL(drm_panel_init);
 
 /**
+ * drm_panel_ref - acquire a reference to a panel
+ * @panel: DRM panel
+ *
+ * Increases the reference on a panel and returns a pointer to it.
+ *
+ * Return: A reference to the panel on success or NULL on failure.
+ */
+struct drm_panel *drm_panel_ref(struct drm_panel *panel)
+{
+	if (panel) {
+		struct registry_record *record;
+
+		record = registry_record_ref(&panel->record);
+		if (!record)
+			panel = NULL;
+	}
+
+	return panel;
+}
+EXPORT_SYMBOL(drm_panel_ref);
+
+/**
+ * drm_panel_unref - release a reference to a panel
+ * @panel: DRM panel
+ *
+ * Decreases the reference count on a panel. If the reference count reaches 0
+ * the panel is destroyed.
+ */
+void drm_panel_unref(struct drm_panel *panel)
+{
+	if (panel)
+		registry_record_unref(&panel->record);
+}
+EXPORT_SYMBOL(drm_panel_unref);
+
+/**
  * drm_panel_add - add a panel to the global registry
  * @panel: panel to add
  *
@@ -62,11 +124,10 @@ EXPORT_SYMBOL(drm_panel_init);
  */
 int drm_panel_add(struct drm_panel *panel)
 {
-	mutex_lock(&panel_lock);
-	list_add_tail(&panel->list, &panel_list);
-	mutex_unlock(&panel_lock);
+	panel->record.owner = panel->dev->driver->owner;
+	panel->record.dev = panel->dev;
 
-	return 0;
+	return registry_add(&panels, &panel->record);
 }
 EXPORT_SYMBOL(drm_panel_add);
 
@@ -74,13 +135,12 @@ EXPORT_SYMBOL(drm_panel_add);
  * drm_panel_remove - remove a panel from the global registry
  * @panel: DRM panel
  *
- * Removes a panel from the global registry.
+ * Removes a panel from the global registry. References to the object can
+ * still exist, but drivers won't be able to look the panel up again.
  */
 void drm_panel_remove(struct drm_panel *panel)
 {
-	mutex_lock(&panel_lock);
-	list_del_init(&panel->list);
-	mutex_unlock(&panel_lock);
+	registry_remove(&panels, &panel->record);
 }
 EXPORT_SYMBOL(drm_panel_remove);
 
@@ -132,25 +192,19 @@ EXPORT_SYMBOL(drm_panel_detach);
  * @np: device tree node of the panel
  *
  * Searches the set of registered panels for one that matches the given device
- * tree node. If a matching panel is found, return a pointer to it.
+ * tree node. If a matching panel is found, return a reference to it.
  *
  * Return: A pointer to the panel registered for the specified device tree
  * node or NULL if no panel matching the device tree node can be found.
  */
 struct drm_panel *of_drm_find_panel(struct device_node *np)
 {
-	struct drm_panel *panel;
-
-	mutex_lock(&panel_lock);
+	struct registry_record *record;
 
-	list_for_each_entry(panel, &panel_list, list) {
-		if (panel->dev->of_node == np) {
-			mutex_unlock(&panel_lock);
-			return panel;
-		}
-	}
+	record = registry_find_by_of_node(&panels, np);
+	if (record)
+		return container_of(record, struct drm_panel, record);
 
-	mutex_unlock(&panel_lock);
 	return NULL;
 }
 EXPORT_SYMBOL(of_drm_find_panel);
diff --git a/include/drm/drm_panel.h b/include/drm/drm_panel.h
index b88b4dcd698b..117e221dbc08 100644
--- a/include/drm/drm_panel.h
+++ b/include/drm/drm_panel.h
@@ -24,7 +24,8 @@
 #ifndef __DRM_PANEL_H__
 #define __DRM_PANEL_H__
 
-#include <linux/list.h>
+#include <linux/module.h>
+#include <linux/registry.h>
 
 struct drm_connector;
 struct drm_device;
@@ -32,6 +33,7 @@ struct drm_panel;
 
 /**
  * struct drm_panel_funcs - perform operations on a given panel
+ * @release: called when the final reference is dropped
  * @disable: disable panel (turn off back light, etc.)
  * @unprepare: turn off panel
  * @prepare: turn on panel and perform set up
@@ -63,6 +65,7 @@ struct drm_panel;
  * the panel. This is the job of the .unprepare() function.
  */
 struct drm_panel_funcs {
+	void (*release)(struct drm_panel *panel);
 	int (*disable)(struct drm_panel *panel);
 	int (*unprepare)(struct drm_panel *panel);
 	int (*prepare)(struct drm_panel *panel);
@@ -72,20 +75,20 @@ struct drm_panel_funcs {
 
 /**
  * struct drm_panel - DRM panel object
+ * @record: registry record
  * @drm: DRM device owning the panel
  * @connector: DRM connector that the panel is attached to
  * @dev: parent device of the panel
  * @funcs: operations that can be performed on the panel
- * @list: panel entry in registry
  */
 struct drm_panel {
+	struct registry_record record;
+
 	struct drm_device *drm;
 	struct drm_connector *connector;
 	struct device *dev;
 
 	const struct drm_panel_funcs *funcs;
-
-	struct list_head list;
 };
 
 /**
@@ -180,6 +183,8 @@ static inline int drm_panel_get_modes(struct drm_panel *panel)
 }
 
 void drm_panel_init(struct drm_panel *panel);
+struct drm_panel *drm_panel_ref(struct drm_panel *panel);
+void drm_panel_unref(struct drm_panel *panel);
 
 int drm_panel_add(struct drm_panel *panel);
 void drm_panel_remove(struct drm_panel *panel);
-- 
2.1.3



More information about the dri-devel mailing list