diff options
author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-08-10 13:17:16 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-08-11 01:25:47 +0300 |
commit | a62052ba2aecb9269a32efeb3e22f96b83a13304 (patch) | |
tree | 47b4a528a298070c8bf0a880cf01402dbd99a667 /net/wimax/wimax-internal.h | |
parent | 38b9e0f6d981daf46a935c8c63a67f3949a15b6a (diff) | |
download | linux-a62052ba2aecb9269a32efeb3e22f96b83a13304.tar.xz |
wimax: no need to check return value of debugfs_create functions
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.
This cleans up a lot of unneeded code and logic around the debugfs wimax
files, making all of this much simpler and easier to understand.
Cc: Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>
Cc: linux-wimax@intel.com
Cc: netdev@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/wimax/wimax-internal.h')
-rw-r--r-- | net/wimax/wimax-internal.h | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/net/wimax/wimax-internal.h b/net/wimax/wimax-internal.h index e819a09337ee..40751207296c 100644 --- a/net/wimax/wimax-internal.h +++ b/net/wimax/wimax-internal.h @@ -57,13 +57,10 @@ void __wimax_state_set(struct wimax_dev *wimax_dev, enum wimax_st state) void __wimax_state_change(struct wimax_dev *, enum wimax_st); #ifdef CONFIG_DEBUG_FS -int wimax_debugfs_add(struct wimax_dev *); +void wimax_debugfs_add(struct wimax_dev *); void wimax_debugfs_rm(struct wimax_dev *); #else -static inline int wimax_debugfs_add(struct wimax_dev *wimax_dev) -{ - return 0; -} +static inline void wimax_debugfs_add(struct wimax_dev *wimax_dev) {} static inline void wimax_debugfs_rm(struct wimax_dev *wimax_dev) {} #endif |