Commit 7a98abcc authored by Pontus Fuchs's avatar Pontus Fuchs Committed by Greg Kroah-Hartman

staging: ks7010: Use printk format specifier for MAC addresses

Convert to %pM instead of custom code.
Signed-off-by: default avatarPontus Fuchs <pontus.fuchs@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 0af45479
...@@ -187,13 +187,7 @@ int get_current_ap(struct ks_wlan_private *priv, struct link_ap_info_t *ap_info) ...@@ -187,13 +187,7 @@ int get_current_ap(struct ks_wlan_private *priv, struct link_ap_info_t *ap_info)
memcpy(wrqu.ap_addr.sa_data, memcpy(wrqu.ap_addr.sa_data,
&(priv->current_ap.bssid[0]), ETH_ALEN); &(priv->current_ap.bssid[0]), ETH_ALEN);
DPRINTK(3, DPRINTK(3,
"IWEVENT: connect bssid=%02x:%02x:%02x:%02x:%02x:%02x\n", "IWEVENT: connect bssid=%pM\n", wrqu.ap_addr.sa_data);
(unsigned char)wrqu.ap_addr.sa_data[0],
(unsigned char)wrqu.ap_addr.sa_data[1],
(unsigned char)wrqu.ap_addr.sa_data[2],
(unsigned char)wrqu.ap_addr.sa_data[3],
(unsigned char)wrqu.ap_addr.sa_data[4],
(unsigned char)wrqu.ap_addr.sa_data[5]);
wireless_send_event(netdev, SIOCGIWAP, &wrqu, NULL); wireless_send_event(netdev, SIOCGIWAP, &wrqu, NULL);
} }
DPRINTK(4, "\n Link AP\n"); DPRINTK(4, "\n Link AP\n");
...@@ -420,16 +414,11 @@ void hostif_data_indication(struct ks_wlan_private *priv) ...@@ -420,16 +414,11 @@ void hostif_data_indication(struct ks_wlan_private *priv)
/* needed parameters: count, keyid, key type, TSC */ /* needed parameters: count, keyid, key type, TSC */
sprintf(buf, sprintf(buf,
"MLME-MICHAELMICFAILURE.indication(keyid=%d %scast addr=" "MLME-MICHAELMICFAILURE.indication(keyid=%d %scast addr="
"%02x:%02x:%02x:%02x:%02x:%02x)", "%pM)",
auth_type - 1, auth_type - 1,
eth_hdr-> eth_hdr->
h_dest[0] & 0x01 ? "broad" : h_dest[0] & 0x01 ? "broad" :
"uni", eth_hdr->h_source[0], "uni", eth_hdr->h_source);
eth_hdr->h_source[1],
eth_hdr->h_source[2],
eth_hdr->h_source[3],
eth_hdr->h_source[4],
eth_hdr->h_source[5]);
memset(&wrqu, 0, sizeof(wrqu)); memset(&wrqu, 0, sizeof(wrqu));
wrqu.data.length = strlen(buf); wrqu.data.length = strlen(buf);
DPRINTK(4, DPRINTK(4,
...@@ -560,10 +549,7 @@ void hostif_mib_get_confirm(struct ks_wlan_private *priv) ...@@ -560,10 +549,7 @@ void hostif_mib_get_confirm(struct ks_wlan_private *priv)
dev->dev_addr[5] = priv->eth_addr[5]; dev->dev_addr[5] = priv->eth_addr[5];
dev->dev_addr[6] = 0x00; dev->dev_addr[6] = 0x00;
dev->dev_addr[7] = 0x00; dev->dev_addr[7] = 0x00;
printk(KERN_INFO printk(KERN_INFO "ks_wlan: MAC ADDRESS = %pM\n", priv->eth_addr);
"ks_wlan: MAC ADDRESS = %02x:%02x:%02x:%02x:%02x:%02x\n",
priv->eth_addr[0], priv->eth_addr[1], priv->eth_addr[2],
priv->eth_addr[3], priv->eth_addr[4], priv->eth_addr[5]);
break; break;
case DOT11_PRODUCT_VERSION: case DOT11_PRODUCT_VERSION:
/* firmware version */ /* firmware version */
...@@ -1184,9 +1170,7 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *packet) ...@@ -1184,9 +1170,7 @@ int hostif_data_request(struct ks_wlan_private *priv, struct sk_buff *packet)
eth = (struct ethhdr *)packet->data; eth = (struct ethhdr *)packet->data;
if (memcmp(&priv->eth_addr[0], eth->h_source, ETH_ALEN)) { if (memcmp(&priv->eth_addr[0], eth->h_source, ETH_ALEN)) {
DPRINTK(1, "invalid mac address !!\n"); DPRINTK(1, "invalid mac address !!\n");
DPRINTK(1, "ethernet->h_source=%02X:%02X:%02X:%02X:%02X:%02X\n", DPRINTK(1, "ethernet->h_source=%pM\n", eth->h_source);
eth->h_source[0], eth->h_source[1], eth->h_source[2],
eth->h_source[3], eth->h_source[4], eth->h_source[5]);
dev_kfree_skb(packet); dev_kfree_skb(packet);
kfree(pp); kfree(pp);
return -3; return -3;
...@@ -2217,10 +2201,7 @@ void hostif_sme_mode_setup(struct ks_wlan_private *priv) ...@@ -2217,10 +2201,7 @@ void hostif_sme_mode_setup(struct ks_wlan_private *priv)
} else { } else {
hostif_infrastructure_set2_request(priv); hostif_infrastructure_set2_request(priv);
DPRINTK(2, DPRINTK(2,
"Infra bssid = %02x:%02x:%02x:%02x:%02x:%02x\n", "Infra bssid = %pM\n", priv->reg.bssid);
priv->reg.bssid[0], priv->reg.bssid[1],
priv->reg.bssid[2], priv->reg.bssid[3],
priv->reg.bssid[4], priv->reg.bssid[5]);
} }
break; break;
case MODE_ADHOC: case MODE_ADHOC:
...@@ -2230,10 +2211,7 @@ void hostif_sme_mode_setup(struct ks_wlan_private *priv) ...@@ -2230,10 +2211,7 @@ void hostif_sme_mode_setup(struct ks_wlan_private *priv)
} else { } else {
hostif_adhoc_set2_request(priv); hostif_adhoc_set2_request(priv);
DPRINTK(2, DPRINTK(2,
"Adhoc bssid = %02x:%02x:%02x:%02x:%02x:%02x\n", "Adhoc bssid = %pM\n", priv->reg.bssid);
priv->reg.bssid[0], priv->reg.bssid[1],
priv->reg.bssid[2], priv->reg.bssid[3],
priv->reg.bssid[4], priv->reg.bssid[5]);
} }
break; break;
default: default:
......
...@@ -399,9 +399,7 @@ static int ks_wlan_set_wap(struct net_device *dev, struct iw_request_info *info, ...@@ -399,9 +399,7 @@ static int ks_wlan_set_wap(struct net_device *dev, struct iw_request_info *info,
return -EOPNOTSUPP; return -EOPNOTSUPP;
} }
DPRINTK(2, "bssid = %02x:%02x:%02x:%02x:%02x:%02x\n", DPRINTK(2, "bssid = %pM\n", priv->reg.bssid);
priv->reg.bssid[0], priv->reg.bssid[1], priv->reg.bssid[2],
priv->reg.bssid[3], priv->reg.bssid[4], priv->reg.bssid[5]);
/* Write it to the card */ /* Write it to the card */
if (priv->need_commit) { if (priv->need_commit) {
...@@ -3321,10 +3319,7 @@ int ks_wlan_set_mac_address(struct net_device *dev, void *addr) ...@@ -3321,10 +3319,7 @@ int ks_wlan_set_mac_address(struct net_device *dev, void *addr)
priv->mac_address_valid = 0; priv->mac_address_valid = 0;
hostif_sme_enqueue(priv, SME_MACADDRESS_SET_REQUEST); hostif_sme_enqueue(priv, SME_MACADDRESS_SET_REQUEST);
netdev_info(dev, netdev_info(dev, "ks_wlan: MAC ADDRESS = %pM\n", priv->eth_addr);
"ks_wlan: MAC ADDRESS = %02x:%02x:%02x:%02x:%02x:%02x\n",
priv->eth_addr[0], priv->eth_addr[1], priv->eth_addr[2],
priv->eth_addr[3], priv->eth_addr[4], priv->eth_addr[5]);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment