vga_switcheroo: Remove unnecessary checks
authorThierry Reding <treding@nvidia.com>
Wed, 12 Aug 2015 14:32:12 +0000 (16:32 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 12 Aug 2015 15:13:19 +0000 (17:13 +0200)
debugfs_remove() gracefully ignores NULL parameters, so the explicit
checks can be removed.

Signed-off-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/vga/vga_switcheroo.c

index c7771466595f18e5deb993ff47b379871e3892cb..21060668fd25ba59b514c6a5a915a542ef8da159 100644 (file)
@@ -525,14 +525,11 @@ static const struct file_operations vga_switcheroo_debugfs_fops = {
 
 static void vga_switcheroo_debugfs_fini(struct vgasr_priv *priv)
 {
-       if (priv->switch_file) {
-               debugfs_remove(priv->switch_file);
-               priv->switch_file = NULL;
-       }
-       if (priv->debugfs_root) {
-               debugfs_remove(priv->debugfs_root);
-               priv->debugfs_root = NULL;
-       }
+       debugfs_remove(priv->switch_file);
+       priv->switch_file = NULL;
+
+       debugfs_remove(priv->debugfs_root);
+       priv->debugfs_root = NULL;
 }
 
 static int vga_switcheroo_debugfs_init(struct vgasr_priv *priv)