brcmfmac: use define instead of hardcoded values.
authorHante Meuleman <meuleman@broadcom.com>
Thu, 27 Sep 2012 12:17:49 +0000 (14:17 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 28 Sep 2012 17:54:08 +0000 (13:54 -0400)
brcmf_parse_tlvs was using hardcoded values.

Reviewed-by: Arend Van Spriel <arend@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Hante Meuleman <meuleman@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c

index d0a2fa0..8674d34 100644 (file)
@@ -48,6 +48,8 @@
 #define BRCMF_PNO_SCAN_COMPLETE                1
 #define BRCMF_PNO_SCAN_INCOMPLETE      0
 
+#define TLV_HDR_LEN                    2       /* header length */
+
 #define BRCMF_ASSOC_PARAMS_FIXED_SIZE \
        (sizeof(struct brcmf_assoc_params_le) - sizeof(u16))
 
@@ -2608,15 +2610,15 @@ static struct brcmf_tlv *brcmf_parse_tlvs(void *buf, int buflen, uint key)
        totlen = buflen;
 
        /* find tagged parameter */
-       while (totlen >= 2) {
+       while (totlen >= TLV_HDR_LEN) {
                int len = elt->len;
 
                /* validate remaining totlen */
-               if ((elt->id == key) && (totlen >= (len + 2)))
+               if ((elt->id == key) && (totlen >= (len + TLV_HDR_LEN)))
                        return elt;
 
-               elt = (struct brcmf_tlv *) ((u8 *) elt + (len + 2));
-               totlen -= (len + 2);
+               elt = (struct brcmf_tlv *) ((u8 *) elt + (len + TLV_HDR_LEN));
+               totlen -= (len + TLV_HDR_LEN);
        }
 
        return NULL;