diff options
| author | Yingsha Xu <ysxu@hust.edu.cn> | 2023-04-19 18:45:47 +0800 |
|---|---|---|
| committer | Johannes Berg <johannes.berg@intel.com> | 2023-04-20 11:46:07 +0200 |
| commit | eb74bfcfa99415f1a65ce021b8ac2256b7ed50bc (patch) | |
| tree | 78df50bea3c8738b19f2575821a233091be6926f /net/mac80211/debugfs.c | |
| parent | 22b68fc6d693e7a2b1c0eb852463f4a72522fa08 (diff) | |
| download | linux-stable-eb74bfcfa99415f1a65ce021b8ac2256b7ed50bc.tar.gz linux-stable-eb74bfcfa99415f1a65ce021b8ac2256b7ed50bc.tar.bz2 linux-stable-eb74bfcfa99415f1a65ce021b8ac2256b7ed50bc.zip | |
wifi: mac80211: remove return value check of debugfs_create_dir()
Smatch complains that:
debugfs_hw_add() warn: 'statsd' is an error pointer or valid
Debugfs checks are generally not supposed to be checked for errors
and it is not necessary here.
Just delete the dead code.
Signed-off-by: Yingsha Xu <ysxu@hust.edu.cn>
Reviewed-by: Dongliang Mu <dzm91@hust.edu.cn>
Link: https://lore.kernel.org/r/20230419104548.30124-1-ysxu@hust.edu.cn
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/debugfs.c')
| -rw-r--r-- | net/mac80211/debugfs.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/net/mac80211/debugfs.c b/net/mac80211/debugfs.c index dfb9f55e2685..207f772bd8ce 100644 --- a/net/mac80211/debugfs.c +++ b/net/mac80211/debugfs.c @@ -673,10 +673,6 @@ void debugfs_hw_add(struct ieee80211_local *local) statsd = debugfs_create_dir("statistics", phyd); - /* if the dir failed, don't put all the other things into the root! */ - if (!statsd) - return; - #ifdef CONFIG_MAC80211_DEBUG_COUNTERS DEBUGFS_STATS_ADD(dot11TransmittedFragmentCount); DEBUGFS_STATS_ADD(dot11MulticastTransmittedFrameCount); |