staging:rtl8192u: Rename sHTCLng - Style
authorJohn Whitmore <johnfwhitmore@gmail.com>
Fri, 17 Aug 2018 18:34:39 +0000 (19:34 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 27 Aug 2018 17:22:39 +0000 (19:22 +0200)
The constant sHTCLng causes a checkpatch issue, due to its use of
CamelCase naming. To correct the issue the constant has been renamed
to HTCLNG.

I'm not sure this is a good name as it communicates very little and
contradicts the block comment above its definition. MCS_LEN might
be a better name if the block comment is correct.

Additionally the block comment has been changed to the recommended
style.

This is a coding style change which should have no 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/ieee80211_rx.c
drivers/staging/rtl8192u/ieee80211/rtl819x_HT.h

index e1897bd..5147f7c 100644 (file)
@@ -794,7 +794,7 @@ static u8 parse_subframe(struct sk_buff *skb,
        }
 
        if (rx_stats->bContainHTC) {
-               LLCOffset += sHTCLng;
+               LLCOffset += HTCLNG;
        }
        // Null packet, don't indicate it to upper layer
        ChkLength = LLCOffset;/* + (Frame_WEP(frame)!=0 ?Adapter->MgntInfo.SecurityInfo.EncryptionHeadOverhead:0);*/
index 7d54a7c..0c6d6b3 100644 (file)
 //
 #define MIMO_PS_STATIC                         0
 
-//
-//     There should be 128 bits to cover all of the MCS rates. However, since
-//     8190 does not support too much rates, one integer is quite enough.
-//
-
-#define sHTCLng        4
+/*
+ * There should be 128 bits to cover all of the MCS rates. However, since
+ * 8190 does not support too much rates, one integer is quite enough.
+ */
+#define HTCLNG 4
 
 
 #define HT_SUPPORTED_MCS_1SS_BITMAP                                    0x000000ff