brcmsmac: Remove always false 'channel < 0' statement
authorAustin Kim <austindh.kim@gmail.com>
Wed, 27 Nov 2019 05:43:58 +0000 (14:43 +0900)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 18 Dec 2019 18:27:40 +0000 (20:27 +0200)
As 'channel' is declared as u16, the following expression is always false.
   channel < 0

So we can remove unnecessary 'always false' statement.

Signed-off-by: Austin Kim <austindh.kim@gmail.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c

index 3f09d89..7f2c15c 100644 (file)
@@ -5408,7 +5408,7 @@ int brcms_c_set_channel(struct brcms_c_info *wlc, u16 channel)
 {
        u16 chspec = ch20mhz_chspec(channel);
 
-       if (channel < 0 || channel > MAXCHANNEL)
+       if (channel > MAXCHANNEL)
                return -EINVAL;
 
        if (!brcms_c_valid_chanspec_db(wlc->cmi, chspec))