staging: rtl8188eu: ternary operator (?:) replaced by min_t kernel macro
authorIvan Safonov <insafonov@gmail.com>
Tue, 27 Oct 2015 15:19:50 +0000 (22:19 +0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 29 Oct 2015 00:09:08 +0000 (09:09 +0900)
min_t macro better conveys the meaning of the code.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_efuse.c
drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
drivers/staging/rtl8188eu/os_dep/ioctl_linux.c

index 685ce98620f1a51ab317617f446fb8d1d14c6fef..eb894233a78585dddd379fb02a56bc3b88ba7066 100644 (file)
@@ -268,7 +268,7 @@ static void efuse_read_phymap_from_txpktbuf(
                        aaa = le16_to_cpup((__le16 *)&lo32);
                        len = le16_to_cpu(*((__le16 *)&lo32));
 
-                       limit = (len-2 < limit) ? len-2 : limit;
+                       limit = min_t(u16, len-2, limit);
 
                        DBG_88E("%s len:%u, lenbak:%u, aaa:%u, aaabak:%u\n", __func__, len, lenbak, aaa, aaabak);
 
index cc5806f9229ace24d5d75306bae82e91ad4255a6..4e6f1a87d625a2d67b49837ab3664b9736658b89 100644 (file)
@@ -3239,7 +3239,7 @@ static unsigned int OnAssocReq(struct adapter *padapter,
                        pstat->flags |= WLAN_STA_WPS;
                        copy_len = 0;
                } else {
-                       copy_len = ((wpa_ie_len+2) > sizeof(pstat->wpa_ie)) ? (sizeof(pstat->wpa_ie)) : (wpa_ie_len+2);
+                       copy_len = min_t(int, wpa_ie_len + 2, sizeof(pstat->wpa_ie));
                }
                if (copy_len > 0)
                        memcpy(pstat->wpa_ie, wpa_ie-2, copy_len);
index 9c11ef4bc88fdb8421933a129c3cc2e10842f6a7..a076ede50b2245003e4d41d9f3e054fe7d031d8e 100644 (file)
@@ -2669,7 +2669,7 @@ static int rtw_get_sta_wpaie(struct net_device *dev, struct ieee_param *param)
                        int copy_len;
 
                        wpa_ie_len = psta->wpa_ie[1];
-                       copy_len = ((wpa_ie_len+2) > sizeof(psta->wpa_ie)) ? (sizeof(psta->wpa_ie)) : (wpa_ie_len+2);
+                       copy_len = min_t(int, wpa_ie_len + 2, sizeof(psta->wpa_ie));
                        param->u.wpa_ie.len = copy_len;
                        memcpy(param->u.wpa_ie.reserved, psta->wpa_ie, copy_len);
                } else {