OSDN Git Service

staging: rtl8723bs: fix comparsions to true
authorMichael Straube <straube.linux@gmail.com>
Thu, 12 Jul 2018 10:36:10 +0000 (12:36 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 13 Jul 2018 09:41:52 +0000 (11:41 +0200)
Use if(x) instead of if(x == true).

Signed-off-by: Michael Straube <straube.linux@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/core/rtw_ieee80211.c
drivers/staging/rtl8723bs/os_dep/ioctl_linux.c

index 6271804..33f2649 100644 (file)
@@ -95,14 +95,14 @@ bool rtw_is_cckratesonly_included(u8 *rate)
 int rtw_check_network_type(unsigned char *rate, int ratelen, int channel)
 {
        if (channel > 14) {
-               if ((rtw_is_cckrates_included(rate)) == true)
+               if (rtw_is_cckrates_included(rate))
                        return WIRELESS_INVALID;
                else
                        return WIRELESS_11A;
        } else{ /*  could be pure B, pure G, or B/G */
-               if ((rtw_is_cckratesonly_included(rate)) == true)
+               if (rtw_is_cckratesonly_included(rate))
                        return WIRELESS_11B;
-               else if ((rtw_is_cckrates_included(rate)) == true)
+               else if (rtw_is_cckrates_included(rate))
                        return  WIRELESS_11BG;
                else
                        return WIRELESS_11G;
index 63b79a7..c38298d 100644 (file)
@@ -126,26 +126,26 @@ static char *translate_scan(struct adapter *padapter,
 
        /* Add the protocol name */
        iwe.cmd = SIOCGIWNAME;
-       if ((rtw_is_cckratesonly_included((u8 *)&pnetwork->network.SupportedRates)) == true) {
-               if (ht_cap == true)
+       if (rtw_is_cckratesonly_included((u8 *)&pnetwork->network.SupportedRates)) {
+               if (ht_cap)
                        snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11bn");
                else
                snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11b");
-       } else if ((rtw_is_cckrates_included((u8 *)&pnetwork->network.SupportedRates)) == true) {
-               if (ht_cap == true)
+       } else if (rtw_is_cckrates_included((u8 *)&pnetwork->network.SupportedRates)) {
+               if (ht_cap)
                        snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11bgn");
                else
                        snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11bg");
        } else {
                if (pnetwork->network.Configuration.DSConfig > 14) {
-                       if (vht_cap == true)
+                       if (vht_cap)
                                snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11AC");
-                       else if (ht_cap == true)
+                       else if (ht_cap)
                                snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11an");
                        else
                                snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11a");
                } else {
-                       if (ht_cap == true)
+                       if (ht_cap)
                                snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11gn");
                        else
                                snprintf(iwe.u.name, IFNAMSIZ, "IEEE 802.11g");
@@ -785,26 +785,26 @@ static int rtw_wx_get_name(struct net_device *dev,
 
                prates = &pcur_bss->SupportedRates;
 
-               if (rtw_is_cckratesonly_included((u8 *)prates) == true) {
-                       if (ht_cap == true)
+               if (rtw_is_cckratesonly_included((u8 *)prates)) {
+                       if (ht_cap)
                                snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11bn");
                        else
                                snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11b");
-               } else if ((rtw_is_cckrates_included((u8 *)prates)) == true) {
-                       if (ht_cap == true)
+               } else if (rtw_is_cckrates_included((u8 *)prates)) {
+                       if (ht_cap)
                                snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11bgn");
                        else
                                snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11bg");
                } else {
                        if (pcur_bss->Configuration.DSConfig > 14) {
-                               if (vht_cap == true)
+                               if (vht_cap)
                                        snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11AC");
-                               else if (ht_cap == true)
+                               else if (ht_cap)
                                        snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11an");
                                else
                                        snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11a");
                        } else {
-                               if (ht_cap == true)
+                               if (ht_cap)
                                        snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11gn");
                                else
                                        snprintf(wrqu->name, IFNAMSIZ, "IEEE 802.11g");