diff options
| author |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-05-29 16:29:36 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-06-01 10:47:43 +0200 |
| commit | e206a0d47efa04b90eb5de5b9adee3c81bda9b03 (patch) | |
| tree | b53892b8d3e5957a0a87baffa588fac3bd2281dd /drivers/staging/rtlwifi | |
| parent | staging: greybus: camera: no need to check debugfs return values (diff) | |
| download | linux-dev-e206a0d47efa04b90eb5de5b9adee3c81bda9b03.tar.xz linux-dev-e206a0d47efa04b90eb5de5b9adee3c81bda9b03.zip | |
staging: rtlwifi: don't check the return value of debugfs_create_file
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.Finger@lwfinger.net>
Cc: Ping-Ke Shih <pkshih@realtek.com>
Cc: Colin Ian King <colin.king@canonical.com>
Cc: "Frank A. Cancio Bello" <frank@generalsoftwareinc.com>
Cc: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/rtlwifi')
| -rw-r--r-- | drivers/staging/rtlwifi/debug.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/staging/rtlwifi/debug.c b/drivers/staging/rtlwifi/debug.c index bf360f855c12..8999feda29b4 100644 --- a/drivers/staging/rtlwifi/debug.c +++ b/drivers/staging/rtlwifi/debug.c @@ -521,12 +521,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) \ |