[NET]: Introduce and use print_mac() and DECLARE_MAC_BUF()

This is nicer than the MAC_FMT stuff.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 3 insertions, 8 deletions

diff --git a/drivers/net/myri10ge/myri10ge.c b/drivers/net/myri10ge/myri10ge.c
index 2f8864e70ca9..38b03f538e95 100644
--- a/drivers/net/myri10ge/myri10ge.c
+++ b/drivers/net/myri10ge/myri10ge.c
@@ -2395,6 +2395,7 @@ static void myri10ge_set_multicast_list(struct net_device *dev)
         struct dev_mc_list *mc_list;
         __be32 data[2] = { 0, 0 };
         int err;
+        DECLARE_MAC_BUF(mac);
 
         mgp = netdev_priv(dev);
         /* can be called from atomic contexts,
@@ -2442,14 +2443,8 @@ static void myri10ge_set_multicast_list(struct net_device *dev)
                         printk(KERN_ERR "myri10ge: %s: Failed "
                                "MXGEFW_JOIN_MULTICAST_GROUP, error status:"
                                "%d\t", dev->name, err);
-                        printk(KERN_ERR "MAC %02x:%02x:%02x:%02x:%02x:%02x\n",
-                               ((unsigned char *)&mc_list->dmi_addr)[0],
-                               ((unsigned char *)&mc_list->dmi_addr)[1],
-                               ((unsigned char *)&mc_list->dmi_addr)[2],
-                               ((unsigned char *)&mc_list->dmi_addr)[3],
-                               ((unsigned char *)&mc_list->dmi_addr)[4],
-                               ((unsigned char *)&mc_list->dmi_addr)[5]
-                            );
+                        printk(KERN_ERR "MAC %s\n",
+                               print_mac(mac, mc_list->dmi_addr));
                         goto abort;
                 }
         }