On 2024/4/19 23:09, Jeff Johnson wrote:
On 4/17/2024 1:17 AM, Su Hui wrote:
u8 peer_debug_trigger;
- int ret;
+ int ret = 0;
this is unnecessary since this will be written in all paths that lead to the
return that reads it
Yes, this is my fault. I will remove this in v2
On 4/17/2024 1:17 AM, Su Hui wrote:
> Clang Static Checker (scan-build) Warning:
> 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.
> By the way, remove some useless code.
Clang Static Checker (scan-build) Warning:
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.
By the way, remove some useless code.
Fixes: ee8b08a1be82 ("ath10k: add debugfs