can: Remove unnecessary alloc/OOM messages
authorJoe Perches <joe@perches.com>
Sun, 3 Feb 2013 17:28:09 +0000 (17:28 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 4 Feb 2013 18:22:33 +0000 (13:22 -0500)
alloc failures already get standardized OOM
messages and a dump_stack.

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Marc Kleine-Budde <mkl@pengutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/can/sja1000/ems_pci.c
drivers/net/can/sja1000/peak_pci.c
drivers/net/can/sja1000/peak_pcmcia.c
drivers/net/can/sja1000/plx_pci.c
drivers/net/can/usb/ems_usb.c
drivers/net/can/usb/kvaser_usb.c
drivers/net/can/usb/peak_usb/pcan_usb_core.c
drivers/net/can/usb/usb_8dev.c

index 036a326836b29871d8710f23f5e2d57a6c02ac52..36d298da2af6afb4c4ed5b7780c8f9dc11df0311 100644 (file)
@@ -238,7 +238,6 @@ static int ems_pci_add_card(struct pci_dev *pdev,
        /* Allocating card structures to hold addresses, ... */
        card = kzalloc(sizeof(struct ems_pci_card), GFP_KERNEL);
        if (card == NULL) {
-               dev_err(&pdev->dev, "Unable to allocate memory\n");
                pci_disable_device(pdev);
                return -ENOMEM;
        }
index d84888f03d92e28995cd4087b511db1ca3c3a4f1..44406a4839f62dd8ecb8c861692c783cd365f307 100644 (file)
@@ -451,11 +451,8 @@ static int peak_pciec_probe(struct pci_dev *pdev, struct net_device *dev)
        } else {
                /* create the bit banging I2C adapter structure */
                card = kzalloc(sizeof(struct peak_pciec_card), GFP_KERNEL);
-               if (!card) {
-                       dev_err(&pdev->dev,
-                                "failed allocating memory for i2c chip\n");
+               if (!card)
                        return -ENOMEM;
-               }
 
                card->cfg_base = chan->cfg_base;
                card->reg_base = priv->reg_base;
index f1175142b0a079ef61018e8bff78098ba2829a8f..1a7020ba37f5416435f6e8e1475772ec33b5d0db 100644 (file)
@@ -660,7 +660,6 @@ static int pcan_probe(struct pcmcia_device *pdev)
 
        card = kzalloc(sizeof(struct pcan_pccard), GFP_KERNEL);
        if (!card) {
-               dev_err(&pdev->dev, "couldn't allocate card memory\n");
                err = -ENOMEM;
                goto probe_err_2;
        }
index 11d1062a9449f612bf3d6a62c81b26a459427c03..a042cdc260dc720f982c44ae1bf8da315c0faaf6 100644 (file)
@@ -508,7 +508,6 @@ static int plx_pci_add_card(struct pci_dev *pdev,
        /* Allocate card structures to hold addresses, ... */
        card = kzalloc(sizeof(*card), GFP_KERNEL);
        if (!card) {
-               dev_err(&pdev->dev, "Unable to allocate memory\n");
                pci_disable_device(pdev);
                return -ENOMEM;
        }
index c69f0b72b352dc97bfc047a54a5e0d344ec7476b..0e7bde73c3d2a6f69c66da0d640ca81d3f03aec4 100644 (file)
@@ -1014,10 +1014,8 @@ static int ems_usb_probe(struct usb_interface *intf,
        }
 
        dev->intr_in_buffer = kzalloc(INTR_IN_BUFFER_SIZE, GFP_KERNEL);
-       if (!dev->intr_in_buffer) {
-               dev_err(&intf->dev, "Couldn't alloc Intr buffer\n");
+       if (!dev->intr_in_buffer)
                goto cleanup_intr_urb;
-       }
 
        dev->tx_msg_buffer = kzalloc(CPC_HEADER_SIZE +
                                     sizeof(struct ems_cpc_msg), GFP_KERNEL);
index 5b58a4d87397c0c19ce155e8beb5a8a52194f447..45cb9f3c1324817a912c4541358b34eecce818ea 100644 (file)
@@ -561,7 +561,6 @@ static int kvaser_usb_simple_msg_async(struct kvaser_usb_net_priv *priv,
 
        buf = kmalloc(sizeof(struct kvaser_msg), GFP_ATOMIC);
        if (!buf) {
-               netdev_err(netdev, "No memory left for USB buffer\n");
                usb_free_urb(urb);
                return -ENOMEM;
        }
@@ -1268,7 +1267,6 @@ static netdev_tx_t kvaser_usb_start_xmit(struct sk_buff *skb,
 
        buf = kmalloc(sizeof(struct kvaser_msg), GFP_ATOMIC);
        if (!buf) {
-               netdev_err(netdev, "No memory left for USB buffer\n");
                stats->tx_dropped++;
                goto nobufmem;
        }
index d9290ea788e02665d169e34b4fd7704a116622b4..a0f647f92bf55c7388034f9fbd0a377cc1225901 100644 (file)
@@ -386,7 +386,6 @@ static int peak_usb_start(struct peak_usb_device *dev)
 
                buf = kmalloc(dev->adapter->rx_buffer_size, GFP_KERNEL);
                if (!buf) {
-                       netdev_err(netdev, "No memory left for USB buffer\n");
                        usb_free_urb(urb);
                        err = -ENOMEM;
                        break;
@@ -442,7 +441,6 @@ static int peak_usb_start(struct peak_usb_device *dev)
 
                buf = kmalloc(dev->adapter->tx_buffer_size, GFP_KERNEL);
                if (!buf) {
-                       netdev_err(netdev, "No memory left for USB buffer\n");
                        usb_free_urb(urb);
                        err = -ENOMEM;
                        break;
@@ -634,7 +632,6 @@ static int peak_usb_restart(struct peak_usb_device *dev)
        /* also allocate enough space for the commands to send */
        buf = kmalloc(PCAN_USB_MAX_CMD_LEN, GFP_ATOMIC);
        if (!buf) {
-               netdev_err(dev->netdev, "no memory left for async cmd\n");
                usb_free_urb(urb);
                return -ENOMEM;
        }
@@ -729,8 +726,6 @@ static int peak_usb_create_dev(struct peak_usb_adapter *peak_usb_adapter,
        /* allocate a buffer large enough to send commands */
        dev->cmd_buf = kmalloc(PCAN_USB_MAX_CMD_LEN, GFP_KERNEL);
        if (!dev->cmd_buf) {
-               dev_err(&intf->dev, "%s: couldn't alloc cmd buffer\n",
-                       PCAN_USB_DRIVER_NAME);
                err = -ENOMEM;
                goto lbl_set_intf_data;
        }
index f789e6f54a55009f67c9d06bff03e865b9e9e388..6e15ef08f301fe385dec2727bdc6f92dded28e89 100644 (file)
@@ -958,10 +958,8 @@ static int usb_8dev_probe(struct usb_interface *intf,
 
        priv->cmd_msg_buffer = kzalloc(sizeof(struct usb_8dev_cmd_msg),
                                      GFP_KERNEL);
-       if (!priv->cmd_msg_buffer) {
-               netdev_err(netdev, "Couldn't alloc Tx buffer\n");
+       if (!priv->cmd_msg_buffer)
                goto cleanup_candev;
-       }
 
        usb_set_intfdata(intf, priv);