Input: applespi - no need to check return value of debugfs_create functions
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 12 Aug 2019 07:06:38 +0000 (00:06 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 12 Aug 2019 07:10:59 +0000 (00:10 -0700)
When calling debugfs functions, there is no need to ever check the
return value.  The function can work or not, but the code logic should
never do something different based on this.

Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/keyboard/applespi.c

index 584289b67fb3cbccc20f9a26200c624fea468ef9..d38398526965dbfdeedcffdd58c016c123fa53b1 100644 (file)
@@ -1797,30 +1797,12 @@ static int applespi_probe(struct spi_device *spi)
 
        /* set up debugfs entries for touchpad dimensions logging */
        applespi->debugfs_root = debugfs_create_dir("applespi", NULL);
-       if (IS_ERR(applespi->debugfs_root)) {
-               if (PTR_ERR(applespi->debugfs_root) != -ENODEV)
-                       dev_warn(&applespi->spi->dev,
-                                "Error creating debugfs root entry (%ld)\n",
-                                PTR_ERR(applespi->debugfs_root));
-       } else {
-               struct dentry *ret;
-
-               ret = debugfs_create_bool("enable_tp_dim", 0600,
-                                         applespi->debugfs_root,
-                                         &applespi->debug_tp_dim);
-               if (IS_ERR(ret))
-                       dev_dbg(&applespi->spi->dev,
-                               "Error creating debugfs entry enable_tp_dim (%ld)\n",
-                               PTR_ERR(ret));
-
-               ret = debugfs_create_file("tp_dim", 0400,
-                                         applespi->debugfs_root, applespi,
-                                         &applespi_tp_dim_fops);
-               if (IS_ERR(ret))
-                       dev_dbg(&applespi->spi->dev,
-                               "Error creating debugfs entry tp_dim (%ld)\n",
-                               PTR_ERR(ret));
-       }
+
+       debugfs_create_bool("enable_tp_dim", 0600, applespi->debugfs_root,
+                           &applespi->debug_tp_dim);
+
+       debugfs_create_file("tp_dim", 0400, applespi->debugfs_root, applespi,
+                           &applespi_tp_dim_fops);
 
        return 0;
 }