Commit 98900a80 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by David S. Miller

net:hns: print MAC with %pM

printf() has a dedicated specifier to print MAC addresses. Use it instead of
pushing each byte via stack.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 946973a3
...@@ -1259,12 +1259,8 @@ int hns_dsaf_set_mac_uc_entry( ...@@ -1259,12 +1259,8 @@ int hns_dsaf_set_mac_uc_entry(
if (MAC_IS_ALL_ZEROS(mac_entry->addr) || if (MAC_IS_ALL_ZEROS(mac_entry->addr) ||
MAC_IS_BROADCAST(mac_entry->addr) || MAC_IS_BROADCAST(mac_entry->addr) ||
MAC_IS_MULTICAST(mac_entry->addr)) { MAC_IS_MULTICAST(mac_entry->addr)) {
dev_err(dsaf_dev->dev, dev_err(dsaf_dev->dev, "set_uc %s Mac %pM err!\n",
"set_uc %s Mac %02x:%02x:%02x:%02x:%02x:%02x err!\n", dsaf_dev->ae_dev.name, mac_entry->addr);
dsaf_dev->ae_dev.name, mac_entry->addr[0],
mac_entry->addr[1], mac_entry->addr[2],
mac_entry->addr[3], mac_entry->addr[4],
mac_entry->addr[5]);
return -EINVAL; return -EINVAL;
} }
...@@ -1331,12 +1327,8 @@ int hns_dsaf_set_mac_mc_entry( ...@@ -1331,12 +1327,8 @@ int hns_dsaf_set_mac_mc_entry(
/* mac addr check */ /* mac addr check */
if (MAC_IS_ALL_ZEROS(mac_entry->addr)) { if (MAC_IS_ALL_ZEROS(mac_entry->addr)) {
dev_err(dsaf_dev->dev, dev_err(dsaf_dev->dev, "set uc %s Mac %pM err!\n",
"set uc %s Mac %02x:%02x:%02x:%02x:%02x:%02x err!\n", dsaf_dev->ae_dev.name, mac_entry->addr);
dsaf_dev->ae_dev.name, mac_entry->addr[0],
mac_entry->addr[1], mac_entry->addr[2],
mac_entry->addr[3],
mac_entry->addr[4], mac_entry->addr[5]);
return -EINVAL; return -EINVAL;
} }
...@@ -1410,11 +1402,8 @@ int hns_dsaf_add_mac_mc_port(struct dsaf_device *dsaf_dev, ...@@ -1410,11 +1402,8 @@ int hns_dsaf_add_mac_mc_port(struct dsaf_device *dsaf_dev,
/*chechk mac addr */ /*chechk mac addr */
if (MAC_IS_ALL_ZEROS(mac_entry->addr)) { if (MAC_IS_ALL_ZEROS(mac_entry->addr)) {
dev_err(dsaf_dev->dev, dev_err(dsaf_dev->dev, "set_entry failed,addr %pM!\n",
"set_entry failed,addr %02x:%02x:%02x:%02x:%02x:%02x!\n", mac_entry->addr);
mac_entry->addr[0], mac_entry->addr[1],
mac_entry->addr[2], mac_entry->addr[3],
mac_entry->addr[4], mac_entry->addr[5]);
return -EINVAL; return -EINVAL;
} }
...@@ -1497,9 +1486,8 @@ int hns_dsaf_del_mac_entry(struct dsaf_device *dsaf_dev, u16 vlan_id, ...@@ -1497,9 +1486,8 @@ int hns_dsaf_del_mac_entry(struct dsaf_device *dsaf_dev, u16 vlan_id,
/*check mac addr */ /*check mac addr */
if (MAC_IS_ALL_ZEROS(addr) || MAC_IS_BROADCAST(addr)) { if (MAC_IS_ALL_ZEROS(addr) || MAC_IS_BROADCAST(addr)) {
dev_err(dsaf_dev->dev, dev_err(dsaf_dev->dev, "del_entry failed,addr %pM!\n",
"del_entry failed,addr %02x:%02x:%02x:%02x:%02x:%02x!\n", addr);
addr[0], addr[1], addr[2], addr[3], addr[4], addr[5]);
return -EINVAL; return -EINVAL;
} }
...@@ -1563,11 +1551,8 @@ int hns_dsaf_del_mac_mc_port(struct dsaf_device *dsaf_dev, ...@@ -1563,11 +1551,8 @@ int hns_dsaf_del_mac_mc_port(struct dsaf_device *dsaf_dev,
/*check mac addr */ /*check mac addr */
if (MAC_IS_ALL_ZEROS(mac_entry->addr)) { if (MAC_IS_ALL_ZEROS(mac_entry->addr)) {
dev_err(dsaf_dev->dev, dev_err(dsaf_dev->dev, "del_port failed, addr %pM!\n",
"del_port failed, addr %02x:%02x:%02x:%02x:%02x:%02x!\n", mac_entry->addr);
mac_entry->addr[0], mac_entry->addr[1],
mac_entry->addr[2], mac_entry->addr[3],
mac_entry->addr[4], mac_entry->addr[5]);
return -EINVAL; return -EINVAL;
} }
...@@ -1644,11 +1629,8 @@ int hns_dsaf_get_mac_uc_entry(struct dsaf_device *dsaf_dev, ...@@ -1644,11 +1629,8 @@ int hns_dsaf_get_mac_uc_entry(struct dsaf_device *dsaf_dev,
/* check macaddr */ /* check macaddr */
if (MAC_IS_ALL_ZEROS(mac_entry->addr) || if (MAC_IS_ALL_ZEROS(mac_entry->addr) ||
MAC_IS_BROADCAST(mac_entry->addr)) { MAC_IS_BROADCAST(mac_entry->addr)) {
dev_err(dsaf_dev->dev, dev_err(dsaf_dev->dev, "get_entry failed,addr %pM\n",
"get_entry failed,addr %02x:%02x:%02x:%02x:%02x:%02x\n", mac_entry->addr);
mac_entry->addr[0], mac_entry->addr[1],
mac_entry->addr[2], mac_entry->addr[3],
mac_entry->addr[4], mac_entry->addr[5]);
return -EINVAL; return -EINVAL;
} }
...@@ -1695,11 +1677,8 @@ int hns_dsaf_get_mac_mc_entry(struct dsaf_device *dsaf_dev, ...@@ -1695,11 +1677,8 @@ int hns_dsaf_get_mac_mc_entry(struct dsaf_device *dsaf_dev,
/*check mac addr */ /*check mac addr */
if (MAC_IS_ALL_ZEROS(mac_entry->addr) || if (MAC_IS_ALL_ZEROS(mac_entry->addr) ||
MAC_IS_BROADCAST(mac_entry->addr)) { MAC_IS_BROADCAST(mac_entry->addr)) {
dev_err(dsaf_dev->dev, dev_err(dsaf_dev->dev, "get_entry failed,addr %pM\n",
"get_entry failed,addr %02x:%02x:%02x:%02x:%02x:%02x\n", mac_entry->addr);
mac_entry->addr[0], mac_entry->addr[1],
mac_entry->addr[2], mac_entry->addr[3],
mac_entry->addr[4], mac_entry->addr[5]);
return -EINVAL; return -EINVAL;
} }
......
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