Create a helper to encapsulate the code that creates a new debugfs files
from a linked list related to an object. Moreover, the helper also
provides more flexibily on the type of the object.

Signed-off-by: Maíra Canal <mca...@igalia.com>
---
 drivers/gpu/drm/drm_debugfs.c | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/drm_debugfs.c b/drivers/gpu/drm/drm_debugfs.c
index 0e3f3ffa9f88..b4d2e7dd87f5 100644
--- a/drivers/gpu/drm/drm_debugfs.c
+++ b/drivers/gpu/drm/drm_debugfs.c
@@ -243,11 +243,21 @@ static void drm_debugfs_files_add(struct 
drm_debugfs_files *debugfs_files, struc
        mutex_unlock(&debugfs_files->mutex);
 }
 
+static void drm_debugfs_register(struct drm_minor *minor, struct 
drm_debugfs_files *debugfs_files)
+{
+       struct drm_debugfs_entry *entry, *tmp;
+
+       list_for_each_entry_safe(entry, tmp, &debugfs_files->list, list) {
+               debugfs_create_file(entry->file.name, 0444,
+                                   minor->debugfs_root, entry, 
&drm_debugfs_entry_fops);
+               list_del(&entry->list);
+       }
+}
+
 int drm_debugfs_init(struct drm_minor *minor, int minor_id,
                     struct dentry *root)
 {
        struct drm_device *dev = minor->dev;
-       struct drm_debugfs_entry *entry, *tmp;
        char name[64];
 
        INIT_LIST_HEAD(&minor->debugfs_list);
@@ -270,11 +280,7 @@ int drm_debugfs_init(struct drm_minor *minor, int minor_id,
        if (dev->driver->debugfs_init)
                dev->driver->debugfs_init(minor);
 
-       list_for_each_entry_safe(entry, tmp, &dev->debugfs_files->list, list) {
-               debugfs_create_file(entry->file.name, 0444,
-                                   minor->debugfs_root, entry, 
&drm_debugfs_entry_fops);
-               list_del(&entry->list);
-       }
+       drm_debugfs_register(minor, dev->debugfs_files);
 
        return 0;
 }
@@ -282,16 +288,11 @@ int drm_debugfs_init(struct drm_minor *minor, int 
minor_id,
 void drm_debugfs_late_register(struct drm_device *dev)
 {
        struct drm_minor *minor = dev->primary;
-       struct drm_debugfs_entry *entry, *tmp;
 
        if (!minor)
                return;
 
-       list_for_each_entry_safe(entry, tmp, &dev->debugfs_files->list, list) {
-               debugfs_create_file(entry->file.name, 0444,
-                                   minor->debugfs_root, entry, 
&drm_debugfs_entry_fops);
-               list_del(&entry->list);
-       }
+       drm_debugfs_register(minor, dev->debugfs_files);
 }
 
 int drm_debugfs_remove_files(const struct drm_info_list *files, int count,
-- 
2.39.1

Reply via email to