kmalloc will print enough information in case of failure.

Signed-off-by: Wolfram Sang <wsa-...@sang-engineering.com>
---
 drivers/net/can/usb/gs_usb.c | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/drivers/net/can/usb/gs_usb.c b/drivers/net/can/usb/gs_usb.c
index 6f0cbc38782e68..77e3cc06a30c8c 100644
--- a/drivers/net/can/usb/gs_usb.c
+++ b/drivers/net/can/usb/gs_usb.c
@@ -493,10 +493,8 @@ static netdev_tx_t gs_can_start_xmit(struct sk_buff *skb,
 
        /* create a URB, and a buffer for it */
        urb = usb_alloc_urb(0, GFP_ATOMIC);
-       if (!urb) {
-               netdev_err(netdev, "No memory left for URB\n");
+       if (!urb)
                goto nomem_urb;
-       }
 
        hf = usb_alloc_coherent(dev->udev, sizeof(*hf), GFP_ATOMIC,
                                &urb->transfer_dma);
@@ -600,11 +598,8 @@ static int gs_can_open(struct net_device *netdev)
 
                        /* alloc rx urb */
                        urb = usb_alloc_urb(0, GFP_KERNEL);
-                       if (!urb) {
-                               netdev_err(netdev,
-                                          "No memory left for URB\n");
+                       if (!urb)
                                return -ENOMEM;
-                       }
 
                        /* alloc rx buffer */
                        buf = usb_alloc_coherent(dev->udev,
-- 
2.8.1

Reply via email to