The union ba_param_set contains the member variable charData which is
unused. The variable has been removed.
Additionally the member variable shortData has been renamed to clear
a checkpatch issue with CamelCase naming.
These are coding style changes which should not impact runtime code
execution.
Signed-off-by: John Whitmore <johnfwhitmore@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
 };
 
 union ba_param_set {
-       u8 charData[2];
-       u16 shortData;
+       u16 short_data;
        struct {
                u16 AMSDU_Support:1;
                u16 BAPolicy:1;
 
 void ResetBaEntry(PBA_RECORD pBA)
 {
        pBA->bValid                     = false;
-       pBA->BaParamSet.shortData       = 0;
+       pBA->BaParamSet.short_data      = 0;
        pBA->BaTimeoutValue             = 0;
        pBA->DialogToken                = 0;
        pBA->BaStartSeqCtrl.short_data  = 0;
        }
        // BA Parameter Set
 
-       put_unaligned_le16(pBA->BaParamSet.shortData, tag);
+       put_unaligned_le16(pBA->BaParamSet.short_data, tag);
        tag += 2;
        // BA Timeout Value