OSDN Git Service

staging:rtl8192u: Rename IsACValid and add parenthesis - Style
authorJohn Whitmore <johnfwhitmore@gmail.com>
Tue, 31 Jul 2018 21:00:04 +0000 (22:00 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Aug 2018 07:46:01 +0000 (09:46 +0200)
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>
drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h
drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c

index 59a60f8..77f0e26 100644 (file)
@@ -77,6 +77,6 @@ struct octet_string {
        u16             length;
 };
 
-#define IsACValid(ac)                  ((ac <= 7) ? true : false)
+#define is_ac_valid(ac)                        (((ac) <= 7) ? true : false)
 
 #endif // #ifndef __INC_QOS_TYPE_H
index e0ccaba..de06cd1 100644 (file)
@@ -309,7 +309,7 @@ bool GetTs(
                UP = 0;
        } else {
                // 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);
                        return false;
                }