We never did anything with the return value, and it does not matter if
the call succeeds or not (it's just debugging code), so don't even check
it.

Cc: Larry Finger <larry.fin...@lwfinger.net>
Cc: Ping-Ke Shih <pks...@realtek.com>
Cc: Colin Ian King <colin.k...@canonical.com>
Cc: "Frank A. Cancio Bello" <fr...@generalsoftwareinc.com>
Cc: Nathan Chancellor <natechancel...@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/staging/rtlwifi/debug.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtlwifi/debug.c b/drivers/staging/rtlwifi/debug.c
index c041bc315616..9f04647ec3b0 100644
--- a/drivers/staging/rtlwifi/debug.c
+++ b/drivers/staging/rtlwifi/debug.c
@@ -519,12 +519,9 @@ static const struct file_operations file_ops_common_rw = {
 #define RTL_DEBUGFS_ADD_CORE(name, mode, fopname)                         \
        do {                                                               \
                rtl_debug_priv_ ##name.rtlpriv = rtlpriv;                  \
-               if (!debugfs_create_file(#name, mode,                      \
-                                        parent, &rtl_debug_priv_ ##name,  \
-                                        &file_ops_ ##fopname))            \
-                       pr_err("Unable to initialize debugfs:%s/%s\n",     \
-                              rtlpriv->dbg.debugfs_name,                  \
-                              #name);                                     \
+               debugfs_create_file(#name, mode, parent,                   \
+                                   &rtl_debug_priv_ ##name,               \
+                                   &file_ops_ ##fopname);                 \
        } while (0)
 
 #define RTL_DEBUGFS_ADD(name)                                             \
-- 
2.17.0

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to