drm/tilcdc: Remove tilcdc_debugfs_cleanup()
authorNoralf Trønnes <noralf@tronnes.org>
Thu, 26 Jan 2017 22:56:17 +0000 (23:56 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 27 Jan 2017 14:27:45 +0000 (15:27 +0100)
drm_debugfs_cleanup() now removes all minor->debugfs_list entries
automatically, so the drm_driver.debugfs_cleanup callback is not
needed. Also remove the unused tilcdc_module_ops.debugfs_cleanup()
callback. drm_debugfs_cleanup() removes all debugfs files using
debugfs_remove_recursive(), so there should be no need for such a
callback in the future.

Cc: jsarha@ti.com
Cc: tomi.valkeinen@ti.com
Signed-off-by: Noralf Trønnes <noralf@tronnes.org>
Reviewed-by: Jyri Sarha <jsarha@ti.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20170126225621.12314-16-noralf@tronnes.org
drivers/gpu/drm/tilcdc/tilcdc_drv.c
drivers/gpu/drm/tilcdc/tilcdc_drv.h

index ec15585c7a2783090e3518b0d97f07be4a1d4391..919294a735febf0731dda1004032b09ef92b5d17 100644 (file)
@@ -539,17 +539,6 @@ static int tilcdc_debugfs_init(struct drm_minor *minor)
 
        return ret;
 }
-
-static void tilcdc_debugfs_cleanup(struct drm_minor *minor)
-{
-       struct tilcdc_module *mod;
-       drm_debugfs_remove_files(tilcdc_debugfs_list,
-                       ARRAY_SIZE(tilcdc_debugfs_list), minor);
-
-       list_for_each_entry(mod, &module_list, list)
-               if (mod->funcs->debugfs_cleanup)
-                       mod->funcs->debugfs_cleanup(mod, minor);
-}
 #endif
 
 static const struct file_operations fops = {
@@ -589,7 +578,6 @@ static struct drm_driver tilcdc_driver = {
        .gem_prime_mmap         = drm_gem_cma_prime_mmap,
 #ifdef CONFIG_DEBUG_FS
        .debugfs_init       = tilcdc_debugfs_init,
-       .debugfs_cleanup    = tilcdc_debugfs_cleanup,
 #endif
        .fops               = &fops,
        .name               = "tilcdc",
index 0e71daf5b5cb30ec2dfa756ece83156122e4f2b9..8caa11bc7aec3c308a699a1a5c6256090f44087b 100644 (file)
@@ -111,8 +111,6 @@ struct tilcdc_module_ops {
 #ifdef CONFIG_DEBUG_FS
        /* create debugfs nodes (can be NULL): */
        int (*debugfs_init)(struct tilcdc_module *mod, struct drm_minor *minor);
-       /* cleanup debugfs nodes (can be NULL): */
-       void (*debugfs_cleanup)(struct tilcdc_module *mod, struct drm_minor *minor);
 #endif
 };