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>

diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h b/drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h
index 59a60f86607c61dda4e54e8786fa4d78d355a377..77f0e26f5d2d67bbc1b85ea5fc8d6c69997289b7 100644 (file)
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h
@@ -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

diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c
index e0ccabadec22c5744720eb616c5f59aed62f2a87..de06cd11ab65434bd084fb9d4e2fe8c1813e8a70 100644 (file)
--- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c
+++ b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c
@@ -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;
                }