staging: wlan-ng: Replace hard coded values with MACRO's
authorHari Prasath Gujulan Elango <hgujulan@visteon.com>
Mon, 15 Jun 2015 11:48:53 +0000 (11:48 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 16 Jun 2015 04:56:37 +0000 (21:56 -0700)
This patch replaces hard coded values with global definitions for the
Ethernet IEEE 802.3 interface defined in standard header file.

Signed-off-by: Hari Prasath Gujulan Elango <hgujulan@visteon.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wlan-ng/p80211conv.c

index c0e6ac8..49f2ef8 100644 (file)
@@ -129,7 +129,7 @@ int skb_ether_to_p80211(wlandevice_t *wlandev, u32 ethconv,
        } else {
                /* step 1: classify ether frame, DIX or 802.3? */
                proto = ntohs(e_hdr.type);
-               if (proto <= 1500) {
+               if (proto <= ETH_DATA_LEN) {
                        pr_debug("802.3 len: %d\n", skb->len);
                        /* codes <= 1500 reserved for 802.3 lengths */
                        /* it's 802.3, pass ether payload unchanged,  */
@@ -531,7 +531,7 @@ int p80211_stt_findproto(u16 proto)
           Need to do some testing to confirm.
         */
 
-       if (proto == 0x80f3)    /* APPLETALK */
+       if (proto == ETH_P_AARP)        /* APPLETALK */
                return 1;
 
        return 0;