Commit eb74bfcf authored by Yingsha Xu's avatar Yingsha Xu Committed by Johannes Berg
Browse files

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: default avatarYingsha Xu <ysxu@hust.edu.cn>
Reviewed-by: default avatarDongliang Mu <dzm91@hust.edu.cn>
Link: https://lore.kernel.org/r/20230419104548.30124-1-ysxu@hust.edu.cn


Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 22b68fc6

net/mac80211/debugfs.c

+0 −4
Original line number Diff line number Diff line
@@ -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);