Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f7b4fb22 authored by Joe Perches's avatar Joe Perches Committed by David S. Miller
Browse files

ethernet: Convert dev_printk(KERN_<LEVEL> to dev_<level>( 



dev_<level> calls take less code than dev_printk(KERN_<LEVEL>
and reducing object size is good.
Coalesce formats for easier grep.

Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 855ddb56

drivers/net/ethernet/chelsio/cxgb3/common.h

+3 −4
Original line number Diff line number Diff line
@@ -44,8 +44,7 @@ 


#define CH_ERR(adap, fmt, ...)   dev_err(&adap->pdev->dev, fmt, ##__VA_ARGS__)

#define CH_WARN(adap, fmt, ...)  dev_warn(&adap->pdev->dev, fmt, ##__VA_ARGS__)

#define CH_ALERT(adap, fmt, ...) \

	dev_printk(KERN_ALERT, &adap->pdev->dev, fmt, ## __VA_ARGS__)

#define CH_ALERT(adap, fmt, ...) dev_alert(&adap->pdev->dev, fmt, ##__VA_ARGS__)



/*

 * More powerful macro that selectively prints messages based on msg_enable.

drivers/net/ethernet/marvell/pxa168_eth.c

+3 −4
Original line number Diff line number Diff line
@@ -1131,7 +1131,7 @@ static int pxa168_eth_open(struct net_device *dev) 
	err = request_irq(dev->irq, pxa168_eth_int_handler,

			  IRQF_DISABLED, dev->name, dev);

	if (err) {

		dev_printk(KERN_ERR, &dev->dev, "can't assign irq\n");

		dev_err(&dev->dev, "can't assign irq\n");

		return -EAGAIN;

	}

	pep->rx_resource_err = 0;
@@ -1201,9 +1201,8 @@ static int pxa168_eth_change_mtu(struct net_device *dev, int mtu) 
	 */

	pxa168_eth_stop(dev);

	if (pxa168_eth_open(dev)) {

		dev_printk(KERN_ERR, &dev->dev,

			   "fatal error on re-opening device after "

			   "MTU change\n");

		dev_err(&dev->dev,

			"fatal error on re-opening device after MTU change\n");

	}



	return 0;