Re: [PATCH] wil6210: wmi: Remove useless code

[Kalle Valo]

Jiapeng Chong  wrote:

> Fix the following whitescan warning:
> 
> An unsigned value can never be less than 0.
> 
> Reported-by: Abaci Robot 
> Signed-off-by: Jiapeng Chong 

Patch applied to wireless-drivers-next.git, thanks.

5e6087559e85 wil6210: wmi: Remove useless code

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/1617788766-91433-1-git-send-email-jiapeng.ch...@linux.alibaba.com/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

[PATCH] wil6210: wmi: Remove useless code

[Jiapeng Chong]

Fix the following whitescan warning:

An unsigned value can never be less than 0.

Reported-by: Abaci Robot 
Signed-off-by: Jiapeng Chong 
---
 drivers/net/wireless/ath/wil6210/wmi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/wil6210/wmi.c 
b/drivers/net/wireless/ath/wil6210/wmi.c
index 823ec6e..02ad449 100644
--- a/drivers/net/wireless/ath/wil6210/wmi.c
+++ b/drivers/net/wireless/ath/wil6210/wmi.c
@@ -1456,7 +1456,7 @@ static void wil_link_stats_store_basic(struct wil6210_vif 
*vif,
u8 cid = basic->cid;
struct wil_sta_info *sta;
 
-   if (cid < 0 || cid >= wil->max_assoc_sta) {
+   if (cid >= wil->max_assoc_sta) {
wil_err(wil, "invalid cid %d\n", cid);
return;
}
-- 
1.8.3.1