drm/tilcdc: remove check for return value of debugfs functions.
authorWambui Karuga <wambui.karugax@gmail.com>
Tue, 10 Mar 2020 13:31:06 +0000 (16:31 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 18 Mar 2020 15:21:41 +0000 (16:21 +0100)
Since 987d65d01356 (drm: debugfs: make
drm_debugfs_create_files() never fail), drm_debugfs_create_files() never
fails. Therefore, remove the check and error handling of the return
value of drm_debugfs_create_files() as it is not needed in
tilcdc_debugfs_init().

Also remove local variables that are not used after the changes.

v2: remove conversion of tilcdc_debugfs_init() to void to avoid build
breakage and enable individual compilation.

References: https://lists.freedesktop.org/archives/dri-devel/2020-February/257183.html
Signed-off-by: Wambui Karuga <wambui.karugax@gmail.com>
Acked-by: Jyri Sarha <jsarha@ti.com>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20200310133121.27913-3-wambui.karugax@gmail.com
drivers/gpu/drm/tilcdc/tilcdc_drv.c

index 0791a02..3f7071e 100644 (file)
@@ -480,24 +480,17 @@ static struct drm_info_list tilcdc_debugfs_list[] = {
 
 static int tilcdc_debugfs_init(struct drm_minor *minor)
 {
-       struct drm_device *dev = minor->dev;
        struct tilcdc_module *mod;
-       int ret;
 
-       ret = drm_debugfs_create_files(tilcdc_debugfs_list,
-                       ARRAY_SIZE(tilcdc_debugfs_list),
-                       minor->debugfs_root, minor);
+       drm_debugfs_create_files(tilcdc_debugfs_list,
+                                ARRAY_SIZE(tilcdc_debugfs_list),
+                                minor->debugfs_root, minor);
 
        list_for_each_entry(mod, &module_list, list)
                if (mod->funcs->debugfs_init)
                        mod->funcs->debugfs_init(mod, minor);
 
-       if (ret) {
-               dev_err(dev->dev, "could not install tilcdc_debugfs_list\n");
-               return ret;
-       }
-
-       return ret;
+       return 0;
 }
 #endif