diff options
author | Joe Perches <joe@perches.com> | 2013-03-14 09:07:21 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-03-15 08:56:58 -0400 |
commit | d0320f750093d012d3ed69fc1e8b385f654523d5 (patch) | |
tree | 1f9c25c892c33275fb85a993e9a274dc5618370b /drivers/net/ethernet/aeroflex/greth.c | |
parent | 68c45a2da34cb44962c6a48f8e474ec6b7853641 (diff) |
drivers:net: Remove dma_alloc_coherent OOM messages
I believe these error messages are already logged
on allocation failure by warn_alloc_failed and so
get a dump_stack on OOM.
Remove the unnecessary additional error logging.
Around these deletions:
o Alignment neatening.
o Remove unnecessary casts of dma_alloc_coherent.
o Hoist assigns from ifs.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/aeroflex/greth.c')
-rw-r--r-- | drivers/net/ethernet/aeroflex/greth.c | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/drivers/net/ethernet/aeroflex/greth.c b/drivers/net/ethernet/aeroflex/greth.c index 0be2195e5034..3a9fbacc3729 100644 --- a/drivers/net/ethernet/aeroflex/greth.c +++ b/drivers/net/ethernet/aeroflex/greth.c | |||
@@ -1464,14 +1464,10 @@ static int greth_of_probe(struct platform_device *ofdev) | |||
1464 | } | 1464 | } |
1465 | 1465 | ||
1466 | /* Allocate TX descriptor ring in coherent memory */ | 1466 | /* Allocate TX descriptor ring in coherent memory */ |
1467 | greth->tx_bd_base = (struct greth_bd *) dma_alloc_coherent(greth->dev, | 1467 | greth->tx_bd_base = dma_alloc_coherent(greth->dev, 1024, |
1468 | 1024, | 1468 | &greth->tx_bd_base_phys, |
1469 | &greth->tx_bd_base_phys, | 1469 | GFP_KERNEL); |
1470 | GFP_KERNEL); | ||
1471 | |||
1472 | if (!greth->tx_bd_base) { | 1470 | if (!greth->tx_bd_base) { |
1473 | if (netif_msg_probe(greth)) | ||
1474 | dev_err(&dev->dev, "could not allocate descriptor memory.\n"); | ||
1475 | err = -ENOMEM; | 1471 | err = -ENOMEM; |
1476 | goto error3; | 1472 | goto error3; |
1477 | } | 1473 | } |
@@ -1479,14 +1475,10 @@ static int greth_of_probe(struct platform_device *ofdev) | |||
1479 | memset(greth->tx_bd_base, 0, 1024); | 1475 | memset(greth->tx_bd_base, 0, 1024); |
1480 | 1476 | ||
1481 | /* Allocate RX descriptor ring in coherent memory */ | 1477 | /* Allocate RX descriptor ring in coherent memory */ |
1482 | greth->rx_bd_base = (struct greth_bd *) dma_alloc_coherent(greth->dev, | 1478 | greth->rx_bd_base = dma_alloc_coherent(greth->dev, 1024, |
1483 | 1024, | 1479 | &greth->rx_bd_base_phys, |
1484 | &greth->rx_bd_base_phys, | 1480 | GFP_KERNEL); |
1485 | GFP_KERNEL); | ||
1486 | |||
1487 | if (!greth->rx_bd_base) { | 1481 | if (!greth->rx_bd_base) { |
1488 | if (netif_msg_probe(greth)) | ||
1489 | dev_err(greth->dev, "could not allocate descriptor memory.\n"); | ||
1490 | err = -ENOMEM; | 1482 | err = -ENOMEM; |
1491 | goto error4; | 1483 | goto error4; |
1492 | } | 1484 | } |