wifi: ath10k: Fix an error code problem in ath10k_dbg_sta_write_peer_debug_trigger()
authorSu Hui <suhui@nfschina.com>
Mon, 22 Apr 2024 03:42:44 +0000 (11:42 +0800)
committerKalle Valo <quic_kvalo@quicinc.com>
Wed, 24 Apr 2024 15:40:09 +0000 (18:40 +0300)
Clang Static Checker (scan-build) warns:

drivers/net/wireless/ath/ath10k/debugfs_sta.c:line 429, column 3
Value stored to 'ret' is never read.

Return 'ret' rather than 'count' when 'ret' stores an error code.

Fixes: ee8b08a1be82 ("ath10k: add debugfs support to get per peer tids log via tracing")
Signed-off-by: Su Hui <suhui@nfschina.com>
Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://msgid.link/20240422034243.938962-1-suhui@nfschina.com
drivers/net/wireless/ath/ath10k/debugfs_sta.c

index 394bf3c32abff6be8a8ff8e4ac71cca3315a163b..0f6de862c3a9ba8995a72ef451e90371dce0a978 100644 (file)
@@ -439,7 +439,7 @@ ath10k_dbg_sta_write_peer_debug_trigger(struct file *file,
        }
 out:
        mutex_unlock(&ar->conf_mutex);
-       return count;
+       return ret ?: count;
 }
 
 static const struct file_operations fops_peer_debug_trigger = {