Jelajahi Sumber

staging: ks7010: replace some custom defines with the ones in uapi/linux/if_ether.h

This commit reviews some custom defines changing them for the
globals defined in if_ether header file.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Sergio Paracuellos 7 tahun lalu
induk
melakukan
38fc31e73c
2 mengubah file dengan 5 tambahan dan 11 penghapusan
  1. 1 7
      drivers/staging/ks7010/eap_packet.h
  2. 4 4
      drivers/staging/ks7010/ks_hostif.c

+ 1 - 7
drivers/staging/ks7010/eap_packet.h

@@ -3,13 +3,10 @@
 #define EAP_PACKET_H
 #define EAP_PACKET_H
 
 
 #include <linux/compiler.h>
 #include <linux/compiler.h>
+#include <uapi/linux/if_ether.h>
 
 
 #define WBIT(n) (1 << (n))
 #define WBIT(n) (1 << (n))
 
 
-#ifndef ETH_ALEN
-#define ETH_ALEN 6
-#endif
-
 #define ETHER_HDR_SIZE 20
 #define ETHER_HDR_SIZE 20
 
 
 struct ether_hdr {
 struct ether_hdr {
@@ -20,9 +17,6 @@ struct ether_hdr {
 	unsigned char h_command;
 	unsigned char h_command;
 	unsigned char h_vendor_id[3];
 	unsigned char h_vendor_id[3];
 	__be16 h_proto;	/* packet type ID field */
 	__be16 h_proto;	/* packet type ID field */
-#define ETHER_PROTOCOL_TYPE_EAP		0x888e
-#define ETHER_PROTOCOL_TYPE_IP		0x0800
-#define ETHER_PROTOCOL_TYPE_ARP		0x0806
 	/* followed by length octets of data */
 	/* followed by length octets of data */
 } __packed;
 } __packed;
 
 

+ 4 - 4
drivers/staging/ks7010/ks_hostif.c

@@ -1152,7 +1152,7 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *skb)
 	eth_proto = ntohs(eth_hdr->h_proto);
 	eth_proto = ntohs(eth_hdr->h_proto);
 
 
 	/* for MIC FAILURE REPORT check */
 	/* for MIC FAILURE REPORT check */
-	if (eth_proto == ETHER_PROTOCOL_TYPE_EAP &&
+	if (eth_proto == ETH_P_PAE &&
 	    priv->wpa.mic_failure.failure > 0) {
 	    priv->wpa.mic_failure.failure > 0) {
 		aa1x_hdr = (struct ieee802_1x_hdr *)(eth_hdr + 1);
 		aa1x_hdr = (struct ieee802_1x_hdr *)(eth_hdr + 1);
 		if (aa1x_hdr->type == IEEE802_1X_TYPE_EAPOL_KEY) {
 		if (aa1x_hdr->type == IEEE802_1X_TYPE_EAPOL_KEY) {
@@ -1162,7 +1162,7 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *skb)
 	}
 	}
 
 
 	if (priv->wpa.rsn_enabled && priv->wpa.key[0].key_len) {
 	if (priv->wpa.rsn_enabled && priv->wpa.key[0].key_len) {
-		if (eth_proto == ETHER_PROTOCOL_TYPE_EAP &&
+		if (eth_proto == ETH_P_PAE &&
 		    priv->wpa.key[1].key_len == 0 &&
 		    priv->wpa.key[1].key_len == 0 &&
 		    priv->wpa.key[2].key_len == 0 &&
 		    priv->wpa.key[2].key_len == 0 &&
 		    priv->wpa.key[3].key_len == 0) {
 		    priv->wpa.key[3].key_len == 0) {
@@ -1189,7 +1189,7 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *skb)
 			}
 			}
 		}
 		}
 	} else {
 	} else {
-		if (eth_proto == ETHER_PROTOCOL_TYPE_EAP)
+		if (eth_proto == ETH_P_PAE)
 			pp->auth_type = cpu_to_le16((uint16_t)TYPE_AUTH);
 			pp->auth_type = cpu_to_le16((uint16_t)TYPE_AUTH);
 		else
 		else
 			pp->auth_type = cpu_to_le16((uint16_t)TYPE_DATA);
 			pp->auth_type = cpu_to_le16((uint16_t)TYPE_DATA);
@@ -1206,7 +1206,7 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *skb)
 			       send_packet_complete, skb);
 			       send_packet_complete, skb);
 
 
 	/* MIC FAILURE REPORT check */
 	/* MIC FAILURE REPORT check */
-	if (eth_proto == ETHER_PROTOCOL_TYPE_EAP &&
+	if (eth_proto == ETH_P_PAE &&
 	    priv->wpa.mic_failure.failure > 0) {
 	    priv->wpa.mic_failure.failure > 0) {
 		if (keyinfo & WPA_KEY_INFO_ERROR &&
 		if (keyinfo & WPA_KEY_INFO_ERROR &&
 		    keyinfo & WPA_KEY_INFO_REQUEST) {
 		    keyinfo & WPA_KEY_INFO_REQUEST) {