* [PATCH] net-next-2.6 : TI DaVinci EMAC: Remove print_mac, DECLARE_MAC_BUF
@ 2009-06-02 11:14 Chaithrika U S
2009-06-04 4:54 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Chaithrika U S @ 2009-06-02 11:14 UTC (permalink / raw)
To: netdev; +Cc: davinci-linux-open-source, Chaithrika U S
Use printk format specifier for MAC address.
Remove DECALRE_MAC_BUF and print_mac() usage in the driver.
Signed-off-by: Chaithrika U S <chaithrika@ti.com>
---
drivers/net/davinci_emac.c | 10 ++++------
1 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c
index cf689a0..0e9b9f9 100644
--- a/drivers/net/davinci_emac.c
+++ b/drivers/net/davinci_emac.c
@@ -1819,7 +1819,6 @@ static int emac_dev_setmac_addr(struct net_device *ndev, void *addr)
struct emac_rxch *rxch = priv->rxch[EMAC_DEF_RX_CH];
struct device *emac_dev = &priv->ndev->dev;
struct sockaddr *sa = addr;
- DECLARE_MAC_BUF(mac);
/* Store mac addr in priv and rx channel and set it in EMAC hw */
memcpy(priv->mac_addr, sa->sa_data, ndev->addr_len);
@@ -1828,8 +1827,8 @@ static int emac_dev_setmac_addr(struct net_device *ndev, void *addr)
emac_setmac(priv, EMAC_DEF_RX_CH, rxch->mac_addr);
if (netif_msg_drv(priv))
- dev_notice(emac_dev, "DaVinci EMAC: emac_dev_setmac_addr %s\n",
- print_mac(mac, priv->mac_addr));
+ dev_notice(emac_dev, "DaVinci EMAC: emac_dev_setmac_addr %pM\n",
+ priv->mac_addr);
return 0;
}
@@ -2683,11 +2682,10 @@ static int __devinit davinci_emac_probe(struct platform_device *pdev)
ndev->irq = res->start;
if (!is_valid_ether_addr(priv->mac_addr)) {
- DECLARE_MAC_BUF(buf);
/* Use random MAC if none passed */
random_ether_addr(priv->mac_addr);
- printk(KERN_WARNING "%s: using random MAC addr: %s\n",
- __func__, print_mac(buf, priv->mac_addr));
+ printk(KERN_WARNING "%s: using random MAC addr: %pM\n",
+ __func__, priv->mac_addr);
}
ndev->netdev_ops = &emac_netdev_ops;
--
1.5.6
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] net-next-2.6 : TI DaVinci EMAC: Remove print_mac, DECLARE_MAC_BUF
2009-06-02 11:14 [PATCH] net-next-2.6 : TI DaVinci EMAC: Remove print_mac, DECLARE_MAC_BUF Chaithrika U S
@ 2009-06-04 4:54 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2009-06-04 4:54 UTC (permalink / raw)
To: chaithrika; +Cc: netdev, davinci-linux-open-source
From: Chaithrika U S <chaithrika@ti.com>
Date: Tue, 2 Jun 2009 07:14:41 -0400
> Use printk format specifier for MAC address.
> Remove DECALRE_MAC_BUF and print_mac() usage in the driver.
>
> Signed-off-by: Chaithrika U S <chaithrika@ti.com>
Patch applied, thank you.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-06-04 4:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-06-02 11:14 [PATCH] net-next-2.6 : TI DaVinci EMAC: Remove print_mac, DECLARE_MAC_BUF Chaithrika U S
2009-06-04 4:54 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).