diff options
author | Joe Perches <joe@perches.com> | 2011-06-14 04:57:47 -0400 |
---|---|---|
committer | David S. Miller <davem@conan.davemloft.net> | 2011-06-16 23:35:30 -0400 |
commit | 59deab26c1974b29b7f501e0ace0673d5acadae1 (patch) | |
tree | 60f30d7e1591794b3dea085e5a386a57bd77f83c /drivers/net/gianfar.c | |
parent | ea110733874d5176cb56dcf612a629ffac09dbf0 (diff) |
gianfar: Use pr_<level>, netdev_<level> and netif_<level>
Use the current logging styles.
Add #define DEBUG to get same output for <foo>_dbg messages.
Convert a few bare printks to pr_err.
Fix a likely copy/paste defect where a test was done with RX values:
if (num_rx_qs > MAX_RX_QS) {
but TX limits were emitted:
printk(KERN_ERR "num_rx_qs(=%d) greater than MAX_RX_QS(=%d)\n",
num_tx_qs, MAX_TX_QS);
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@conan.davemloft.net>
Diffstat (limited to 'drivers/net/gianfar.c')
-rw-r--r-- | drivers/net/gianfar.c | 113 |
1 files changed, 48 insertions, 65 deletions
diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c index ff60b23a5b74..0c748328ca7b 100644 --- a/drivers/net/gianfar.c +++ b/drivers/net/gianfar.c | |||
@@ -62,6 +62,9 @@ | |||
62 | * The driver then cleans up the buffer. | 62 | * The driver then cleans up the buffer. |
63 | */ | 63 | */ |
64 | 64 | ||
65 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt | ||
66 | #define DEBUG | ||
67 | |||
65 | #include <linux/kernel.h> | 68 | #include <linux/kernel.h> |
66 | #include <linux/string.h> | 69 | #include <linux/string.h> |
67 | #include <linux/errno.h> | 70 | #include <linux/errno.h> |
@@ -213,8 +216,7 @@ static int gfar_init_bds(struct net_device *ndev) | |||
213 | } else { | 216 | } else { |
214 | skb = gfar_new_skb(ndev); | 217 | skb = gfar_new_skb(ndev); |
215 | if (!skb) { | 218 | if (!skb) { |
216 | pr_err("%s: Can't allocate RX buffers\n", | 219 | netdev_err(ndev, "Can't allocate RX buffers\n"); |
217 | ndev->name); | ||
218 | goto err_rxalloc_fail; | 220 | goto err_rxalloc_fail; |
219 | } | 221 | } |
220 | rx_queue->rx_skbuff[j] = skb; | 222 | rx_queue->rx_skbuff[j] = skb; |
@@ -258,9 +260,8 @@ static int gfar_alloc_skb_resources(struct net_device *ndev) | |||
258 | sizeof(struct rxbd8) * priv->total_rx_ring_size, | 260 | sizeof(struct rxbd8) * priv->total_rx_ring_size, |
259 | &addr, GFP_KERNEL); | 261 | &addr, GFP_KERNEL); |
260 | if (!vaddr) { | 262 | if (!vaddr) { |
261 | if (netif_msg_ifup(priv)) | 263 | netif_err(priv, ifup, ndev, |
262 | pr_err("%s: Could not allocate buffer descriptors!\n", | 264 | "Could not allocate buffer descriptors!\n"); |
263 | ndev->name); | ||
264 | return -ENOMEM; | 265 | return -ENOMEM; |
265 | } | 266 | } |
266 | 267 | ||
@@ -290,9 +291,8 @@ static int gfar_alloc_skb_resources(struct net_device *ndev) | |||
290 | tx_queue->tx_skbuff = kmalloc(sizeof(*tx_queue->tx_skbuff) * | 291 | tx_queue->tx_skbuff = kmalloc(sizeof(*tx_queue->tx_skbuff) * |
291 | tx_queue->tx_ring_size, GFP_KERNEL); | 292 | tx_queue->tx_ring_size, GFP_KERNEL); |
292 | if (!tx_queue->tx_skbuff) { | 293 | if (!tx_queue->tx_skbuff) { |
293 | if (netif_msg_ifup(priv)) | 294 | netif_err(priv, ifup, ndev, |
294 | pr_err("%s: Could not allocate tx_skbuff\n", | 295 | "Could not allocate tx_skbuff\n"); |
295 | ndev->name); | ||
296 | goto cleanup; | 296 | goto cleanup; |
297 | } | 297 | } |
298 | 298 | ||
@@ -306,9 +306,8 @@ static int gfar_alloc_skb_resources(struct net_device *ndev) | |||
306 | rx_queue->rx_ring_size, GFP_KERNEL); | 306 | rx_queue->rx_ring_size, GFP_KERNEL); |
307 | 307 | ||
308 | if (!rx_queue->rx_skbuff) { | 308 | if (!rx_queue->rx_skbuff) { |
309 | if (netif_msg_ifup(priv)) | 309 | netif_err(priv, ifup, ndev, |
310 | pr_err("%s: Could not allocate rx_skbuff\n", | 310 | "Could not allocate rx_skbuff\n"); |
311 | ndev->name); | ||
312 | goto cleanup; | 311 | goto cleanup; |
313 | } | 312 | } |
314 | 313 | ||
@@ -628,9 +627,9 @@ static int gfar_of_init(struct platform_device *ofdev, struct net_device **pdev) | |||
628 | num_tx_qs = tx_queues ? *tx_queues : 1; | 627 | num_tx_qs = tx_queues ? *tx_queues : 1; |
629 | 628 | ||
630 | if (num_tx_qs > MAX_TX_QS) { | 629 | if (num_tx_qs > MAX_TX_QS) { |
631 | printk(KERN_ERR "num_tx_qs(=%d) greater than MAX_TX_QS(=%d)\n", | 630 | pr_err("num_tx_qs(=%d) greater than MAX_TX_QS(=%d)\n", |
632 | num_tx_qs, MAX_TX_QS); | 631 | num_tx_qs, MAX_TX_QS); |
633 | printk(KERN_ERR "Cannot do alloc_etherdev, aborting\n"); | 632 | pr_err("Cannot do alloc_etherdev, aborting\n"); |
634 | return -EINVAL; | 633 | return -EINVAL; |
635 | } | 634 | } |
636 | 635 | ||
@@ -638,9 +637,9 @@ static int gfar_of_init(struct platform_device *ofdev, struct net_device **pdev) | |||
638 | num_rx_qs = rx_queues ? *rx_queues : 1; | 637 | num_rx_qs = rx_queues ? *rx_queues : 1; |
639 | 638 | ||
640 | if (num_rx_qs > MAX_RX_QS) { | 639 | if (num_rx_qs > MAX_RX_QS) { |
641 | printk(KERN_ERR "num_rx_qs(=%d) greater than MAX_RX_QS(=%d)\n", | 640 | pr_err("num_rx_qs(=%d) greater than MAX_RX_QS(=%d)\n", |
642 | num_tx_qs, MAX_TX_QS); | 641 | num_rx_qs, MAX_RX_QS); |
643 | printk(KERN_ERR "Cannot do alloc_etherdev, aborting\n"); | 642 | pr_err("Cannot do alloc_etherdev, aborting\n"); |
644 | return -EINVAL; | 643 | return -EINVAL; |
645 | } | 644 | } |
646 | 645 | ||
@@ -1163,8 +1162,7 @@ static int gfar_probe(struct platform_device *ofdev) | |||
1163 | err = register_netdev(dev); | 1162 | err = register_netdev(dev); |
1164 | 1163 | ||
1165 | if (err) { | 1164 | if (err) { |
1166 | printk(KERN_ERR "%s: Cannot register net device, aborting.\n", | 1165 | pr_err("%s: Cannot register net device, aborting\n", dev->name); |
1167 | dev->name); | ||
1168 | goto register_fail; | 1166 | goto register_fail; |
1169 | } | 1167 | } |
1170 | 1168 | ||
@@ -1215,17 +1213,17 @@ static int gfar_probe(struct platform_device *ofdev) | |||
1215 | gfar_init_sysfs(dev); | 1213 | gfar_init_sysfs(dev); |
1216 | 1214 | ||
1217 | /* Print out the device info */ | 1215 | /* Print out the device info */ |
1218 | printk(KERN_INFO DEVICE_NAME "%pM\n", dev->name, dev->dev_addr); | 1216 | netdev_info(dev, "mac: %pM\n", dev->dev_addr); |
1219 | 1217 | ||
1220 | /* Even more device info helps when determining which kernel */ | 1218 | /* Even more device info helps when determining which kernel */ |
1221 | /* provided which set of benchmarks. */ | 1219 | /* provided which set of benchmarks. */ |
1222 | printk(KERN_INFO "%s: Running with NAPI enabled\n", dev->name); | 1220 | netdev_info(dev, "Running with NAPI enabled\n"); |
1223 | for (i = 0; i < priv->num_rx_queues; i++) | 1221 | for (i = 0; i < priv->num_rx_queues; i++) |
1224 | printk(KERN_INFO "%s: RX BD ring size for Q[%d]: %d\n", | 1222 | netdev_info(dev, "RX BD ring size for Q[%d]: %d\n", |
1225 | dev->name, i, priv->rx_queue[i]->rx_ring_size); | 1223 | i, priv->rx_queue[i]->rx_ring_size); |
1226 | for(i = 0; i < priv->num_tx_queues; i++) | 1224 | for(i = 0; i < priv->num_tx_queues; i++) |
1227 | printk(KERN_INFO "%s: TX BD ring size for Q[%d]: %d\n", | 1225 | netdev_info(dev, "TX BD ring size for Q[%d]: %d\n", |
1228 | dev->name, i, priv->tx_queue[i]->tx_ring_size); | 1226 | i, priv->tx_queue[i]->tx_ring_size); |
1229 | 1227 | ||
1230 | return 0; | 1228 | return 0; |
1231 | 1229 | ||
@@ -1858,34 +1856,30 @@ static int register_grp_irqs(struct gfar_priv_grp *grp) | |||
1858 | * Transmit, and Receive */ | 1856 | * Transmit, and Receive */ |
1859 | if ((err = request_irq(grp->interruptError, gfar_error, 0, | 1857 | if ((err = request_irq(grp->interruptError, gfar_error, 0, |
1860 | grp->int_name_er,grp)) < 0) { | 1858 | grp->int_name_er,grp)) < 0) { |
1861 | if (netif_msg_intr(priv)) | 1859 | netif_err(priv, intr, dev, "Can't get IRQ %d\n", |
1862 | printk(KERN_ERR "%s: Can't get IRQ %d\n", | 1860 | grp->interruptError); |
1863 | dev->name, grp->interruptError); | ||
1864 | 1861 | ||
1865 | goto err_irq_fail; | 1862 | goto err_irq_fail; |
1866 | } | 1863 | } |
1867 | 1864 | ||
1868 | if ((err = request_irq(grp->interruptTransmit, gfar_transmit, | 1865 | if ((err = request_irq(grp->interruptTransmit, gfar_transmit, |
1869 | 0, grp->int_name_tx, grp)) < 0) { | 1866 | 0, grp->int_name_tx, grp)) < 0) { |
1870 | if (netif_msg_intr(priv)) | 1867 | netif_err(priv, intr, dev, "Can't get IRQ %d\n", |
1871 | printk(KERN_ERR "%s: Can't get IRQ %d\n", | 1868 | grp->interruptTransmit); |
1872 | dev->name, grp->interruptTransmit); | ||
1873 | goto tx_irq_fail; | 1869 | goto tx_irq_fail; |
1874 | } | 1870 | } |
1875 | 1871 | ||
1876 | if ((err = request_irq(grp->interruptReceive, gfar_receive, 0, | 1872 | if ((err = request_irq(grp->interruptReceive, gfar_receive, 0, |
1877 | grp->int_name_rx, grp)) < 0) { | 1873 | grp->int_name_rx, grp)) < 0) { |
1878 | if (netif_msg_intr(priv)) | 1874 | netif_err(priv, intr, dev, "Can't get IRQ %d\n", |
1879 | printk(KERN_ERR "%s: Can't get IRQ %d\n", | 1875 | grp->interruptReceive); |
1880 | dev->name, grp->interruptReceive); | ||
1881 | goto rx_irq_fail; | 1876 | goto rx_irq_fail; |
1882 | } | 1877 | } |
1883 | } else { | 1878 | } else { |
1884 | if ((err = request_irq(grp->interruptTransmit, gfar_interrupt, 0, | 1879 | if ((err = request_irq(grp->interruptTransmit, gfar_interrupt, 0, |
1885 | grp->int_name_tx, grp)) < 0) { | 1880 | grp->int_name_tx, grp)) < 0) { |
1886 | if (netif_msg_intr(priv)) | 1881 | netif_err(priv, intr, dev, "Can't get IRQ %d\n", |
1887 | printk(KERN_ERR "%s: Can't get IRQ %d\n", | 1882 | grp->interruptTransmit); |
1888 | dev->name, grp->interruptTransmit); | ||
1889 | goto err_irq_fail; | 1883 | goto err_irq_fail; |
1890 | } | 1884 | } |
1891 | } | 1885 | } |
@@ -2354,9 +2348,7 @@ static int gfar_change_mtu(struct net_device *dev, int new_mtu) | |||
2354 | frame_size += VLAN_HLEN; | 2348 | frame_size += VLAN_HLEN; |
2355 | 2349 | ||
2356 | if ((frame_size < 64) || (frame_size > JUMBO_FRAME_SIZE)) { | 2350 | if ((frame_size < 64) || (frame_size > JUMBO_FRAME_SIZE)) { |
2357 | if (netif_msg_drv(priv)) | 2351 | netif_err(priv, drv, dev, "Invalid MTU setting\n"); |
2358 | printk(KERN_ERR "%s: Invalid MTU setting\n", | ||
2359 | dev->name); | ||
2360 | return -EINVAL; | 2352 | return -EINVAL; |
2361 | } | 2353 | } |
2362 | 2354 | ||
@@ -2776,9 +2768,7 @@ int gfar_clean_rx_ring(struct gfar_priv_rx_q *rx_queue, int rx_work_limit) | |||
2776 | gfar_process_frame(dev, skb, amount_pull); | 2768 | gfar_process_frame(dev, skb, amount_pull); |
2777 | 2769 | ||
2778 | } else { | 2770 | } else { |
2779 | if (netif_msg_rx_err(priv)) | 2771 | netif_warn(priv, rx_err, dev, "Missing skb!\n"); |
2780 | printk(KERN_WARNING | ||
2781 | "%s: Missing skb!\n", dev->name); | ||
2782 | rx_queue->stats.rx_dropped++; | 2772 | rx_queue->stats.rx_dropped++; |
2783 | priv->extra_stats.rx_skbmissing++; | 2773 | priv->extra_stats.rx_skbmissing++; |
2784 | } | 2774 | } |
@@ -2981,10 +2971,9 @@ static void adjust_link(struct net_device *dev) | |||
2981 | ecntrl &= ~(ECNTRL_R100); | 2971 | ecntrl &= ~(ECNTRL_R100); |
2982 | break; | 2972 | break; |
2983 | default: | 2973 | default: |
2984 | if (netif_msg_link(priv)) | 2974 | netif_warn(priv, link, dev, |
2985 | printk(KERN_WARNING | 2975 | "Ack! Speed (%d) is not 10/100/1000!\n", |
2986 | "%s: Ack! Speed (%d) is not 10/100/1000!\n", | 2976 | phydev->speed); |
2987 | dev->name, phydev->speed); | ||
2988 | break; | 2977 | break; |
2989 | } | 2978 | } |
2990 | 2979 | ||
@@ -3189,8 +3178,8 @@ static irqreturn_t gfar_error(int irq, void *grp_id) | |||
3189 | 3178 | ||
3190 | /* Hmm... */ | 3179 | /* Hmm... */ |
3191 | if (netif_msg_rx_err(priv) || netif_msg_tx_err(priv)) | 3180 | if (netif_msg_rx_err(priv) || netif_msg_tx_err(priv)) |
3192 | printk(KERN_DEBUG "%s: error interrupt (ievent=0x%08x imask=0x%08x)\n", | 3181 | netdev_dbg(dev, "error interrupt (ievent=0x%08x imask=0x%08x)\n", |
3193 | dev->name, events, gfar_read(®s->imask)); | 3182 | events, gfar_read(®s->imask)); |
3194 | 3183 | ||
3195 | /* Update the error counters */ | 3184 | /* Update the error counters */ |
3196 | if (events & IEVENT_TXE) { | 3185 | if (events & IEVENT_TXE) { |
@@ -3203,9 +3192,8 @@ static irqreturn_t gfar_error(int irq, void *grp_id) | |||
3203 | if (events & IEVENT_XFUN) { | 3192 | if (events & IEVENT_XFUN) { |
3204 | unsigned long flags; | 3193 | unsigned long flags; |
3205 | 3194 | ||
3206 | if (netif_msg_tx_err(priv)) | 3195 | netif_dbg(priv, tx_err, dev, |
3207 | printk(KERN_DEBUG "%s: TX FIFO underrun, " | 3196 | "TX FIFO underrun, packet dropped\n"); |
3208 | "packet dropped.\n", dev->name); | ||
3209 | dev->stats.tx_dropped++; | 3197 | dev->stats.tx_dropped++; |
3210 | priv->extra_stats.tx_underrun++; | 3198 | priv->extra_stats.tx_underrun++; |
3211 | 3199 | ||
@@ -3218,8 +3206,7 @@ static irqreturn_t gfar_error(int irq, void *grp_id) | |||
3218 | unlock_tx_qs(priv); | 3206 | unlock_tx_qs(priv); |
3219 | local_irq_restore(flags); | 3207 | local_irq_restore(flags); |
3220 | } | 3208 | } |
3221 | if (netif_msg_tx_err(priv)) | 3209 | netif_dbg(priv, tx_err, dev, "Transmit Error\n"); |
3222 | printk(KERN_DEBUG "%s: Transmit Error\n", dev->name); | ||
3223 | } | 3210 | } |
3224 | if (events & IEVENT_BSY) { | 3211 | if (events & IEVENT_BSY) { |
3225 | dev->stats.rx_errors++; | 3212 | dev->stats.rx_errors++; |
@@ -3227,29 +3214,25 @@ static irqreturn_t gfar_error(int irq, void *grp_id) | |||
3227 | 3214 | ||
3228 | gfar_receive(irq, grp_id); | 3215 | gfar_receive(irq, grp_id); |
3229 | 3216 | ||
3230 | if (netif_msg_rx_err(priv)) | 3217 | netif_dbg(priv, rx_err, dev, "busy error (rstat: %x)\n", |
3231 | printk(KERN_DEBUG "%s: busy error (rstat: %x)\n", | 3218 | gfar_read(®s->rstat)); |
3232 | dev->name, gfar_read(®s->rstat)); | ||
3233 | } | 3219 | } |
3234 | if (events & IEVENT_BABR) { | 3220 | if (events & IEVENT_BABR) { |
3235 | dev->stats.rx_errors++; | 3221 | dev->stats.rx_errors++; |
3236 | priv->extra_stats.rx_babr++; | 3222 | priv->extra_stats.rx_babr++; |
3237 | 3223 | ||
3238 | if (netif_msg_rx_err(priv)) | 3224 | netif_dbg(priv, rx_err, dev, "babbling RX error\n"); |
3239 | printk(KERN_DEBUG "%s: babbling RX error\n", dev->name); | ||
3240 | } | 3225 | } |
3241 | if (events & IEVENT_EBERR) { | 3226 | if (events & IEVENT_EBERR) { |
3242 | priv->extra_stats.eberr++; | 3227 | priv->extra_stats.eberr++; |
3243 | if (netif_msg_rx_err(priv)) | 3228 | netif_dbg(priv, rx_err, dev, "bus error\n"); |
3244 | printk(KERN_DEBUG "%s: bus error\n", dev->name); | ||
3245 | } | 3229 | } |
3246 | if ((events & IEVENT_RXC) && netif_msg_rx_status(priv)) | 3230 | if (events & IEVENT_RXC) |
3247 | printk(KERN_DEBUG "%s: control frame\n", dev->name); | 3231 | netif_dbg(priv, rx_status, dev, "control frame\n"); |
3248 | 3232 | ||
3249 | if (events & IEVENT_BABT) { | 3233 | if (events & IEVENT_BABT) { |
3250 | priv->extra_stats.tx_babt++; | 3234 | priv->extra_stats.tx_babt++; |
3251 | if (netif_msg_tx_err(priv)) | 3235 | netif_dbg(priv, tx_err, dev, "babbling TX error\n"); |
3252 | printk(KERN_DEBUG "%s: babbling TX error\n", dev->name); | ||
3253 | } | 3236 | } |
3254 | return IRQ_HANDLED; | 3237 | return IRQ_HANDLED; |
3255 | } | 3238 | } |