[PATCH 3/8] macb: convert printk to pr_ and friends
Joe Perches
joe at perches.com
Thu Mar 10 17:48:04 EST 2011
On Thu, 2011-03-10 at 10:10 +0000, Jamie Iles wrote:
> macb is already using the dev_dbg() and friends helpers so use pr_foo()
> along with a pr_fmt() definition to make the printing a little cleaner.
> Signed-off-by: Jamie Iles <jamie at jamieiles.com>
Hi Jamie.
Because these conversions use a struct net_device,
they would be better as:
netdev_<foo>(struct net_device *dev, const char *fmt, ...)
> diff --git a/drivers/net/macb.c b/drivers/net/macb.c
> @@ -175,11 +176,11 @@ static void macb_handle_link_change(struct net_device *dev)
>
> if (status_change) {
> if (phydev->link)
> - printk(KERN_INFO "%s: link up (%d/%s)\n",
> + pr_info("%s: link up (%d/%s)\n",
> dev->name, phydev->speed,
> DUPLEX_FULL == phydev->duplex ? "Full":"Half");
netdev_info(dev, "link up (%d/%s)\n",
phydev->speed,
phydev->duplex == DUPLEX_FULL ? "Full" : "Half");
> else
> - printk(KERN_INFO "%s: link down\n", dev->name);
> + pr_info("%s: link down\n", dev->name);
netdev_info(dev, "link down\n");
> @@ -193,7 +194,7 @@ static int macb_mii_probe(struct net_device *dev)
>
> phydev = phy_find_first(bp->mii_bus);
> if (!phydev) {
> - printk (KERN_ERR "%s: no PHY found\n", dev->name);
> + pr_err("%s: no PHY found\n", dev->name);
netdev_err(dev, "no PHY found\n");
> @@ -206,7 +207,7 @@ static int macb_mii_probe(struct net_device *dev)
> PHY_INTERFACE_MODE_RMII :
> PHY_INTERFACE_MODE_MII);
> if (ret) {
> - printk(KERN_ERR "%s: Could not attach to PHY\n", dev->name);
> + pr_err("%s: Could not attach to PHY\n", dev->name);
netdev_err(dev, "could not attach to PHY\n");
etc...
More information about the linux-arm-kernel
mailing list