Message ID | 1382650645-23973-1-git-send-email-zanchm@rpi.edu |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
On Thu, 2013-10-24 at 21:37 +0000, Matt Zanchelli wrote: > Updated prinkt() statement in mac_sonic_probe() function to match current > convention of using KERN style leading argument. s/prinkt/printk/ > diff --git a/drivers/net/ethernet/natsemi/macsonic.c b/drivers/net/ethernet/natsemi/macsonic.c [] > @@ -601,7 +601,7 @@ found: > if (err) > goto out; > > - printk("%s: MAC %pM IRQ %d\n", dev->name, dev->dev_addr, dev->irq); > + printk(KERN_INFO "%s: MAC %pM IRQ %d\n", dev->name, dev->dev_addr, dev->irq); It'd be nicer to use: netdev_info(dev, "MAC %pM IRQ %d\n", dev->dev_addr, dev->irq); Nicer still to convert all the printks in this file to netdev_<level> where possible. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/natsemi/macsonic.c b/drivers/net/ethernet/natsemi/macsonic.c index 346a4e0..3eca756 100644 --- a/drivers/net/ethernet/natsemi/macsonic.c +++ b/drivers/net/ethernet/natsemi/macsonic.c @@ -601,7 +601,7 @@ found: if (err) goto out; - printk("%s: MAC %pM IRQ %d\n", dev->name, dev->dev_addr, dev->irq); + printk(KERN_INFO "%s: MAC %pM IRQ %d\n", dev->name, dev->dev_addr, dev->irq); return 0;