Browse Source

staging:rtl8192u: Rename IsACValid and add parenthesis - Style

The macro IsACValid is renamed to resolve the checkpatch issue with
CamelCase naming. In addition the parameter has parenthesis added to
clear the checkpatch issue with precedence issues.

These changes are coding style changes and as such should have not
impact on runtime code execution.

Signed-off-by: John Whitmore <johnfwhitmore@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
John Whitmore 7 years ago
parent
commit
0b0251ad9b

+ 1 - 1
drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h

@@ -77,6 +77,6 @@ struct octet_string {
 	u16             length;
 	u16             length;
 };
 };
 
 
-#define IsACValid(ac)			((ac <= 7) ? true : false)
+#define is_ac_valid(ac)			(((ac) <= 7) ? true : false)
 
 
 #endif // #ifndef __INC_QOS_TYPE_H
 #endif // #ifndef __INC_QOS_TYPE_H

+ 1 - 1
drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c

@@ -309,7 +309,7 @@ bool GetTs(
 		UP = 0;
 		UP = 0;
 	} else {
 	} else {
 		// In WMM case: we use 4 TID only
 		// In WMM case: we use 4 TID only
-		if (!IsACValid(TID)) {
+		if (!is_ac_valid(TID)) {
 			IEEE80211_DEBUG(IEEE80211_DL_ERR, " in %s(), TID(%d) is not valid\n", __func__, TID);
 			IEEE80211_DEBUG(IEEE80211_DL_ERR, " in %s(), TID(%d) is not valid\n", __func__, TID);
 			return false;
 			return false;
 		}
 		}