diff options
author | Joe Perches <joe@perches.com> | 2013-02-07 06:46:27 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-02-08 17:44:39 -0500 |
commit | 14f8dc49532f765968ff37c3b99edbeb99004ace (patch) | |
tree | a0c2f42ca545b3587a071ad05427c7351c8244e0 /drivers/net/can/usb | |
parent | e9ba103931950da7bc87d58ffe39612df17d47af (diff) |
drivers: net: Remove remaining alloc/OOM messages
alloc failures already get standardized OOM
messages and a dump_stack.
For the affected mallocs around these OOM messages:
Converted kmallocs with multiplies to kmalloc_array.
Converted a kmalloc/memcpy to kmemdup.
Removed now unused stack variables.
Removed unnecessary parentheses.
Neatened alignment.
Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Arend van Spriel <arend@broadcom.com>
Acked-by: Marc Kleine-Budde <mkl@pengutronix.de>
Acked-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/can/usb')
-rw-r--r-- | drivers/net/can/usb/ems_usb.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/can/usb/ems_usb.c b/drivers/net/can/usb/ems_usb.c index 0e7bde73c3d2..5f9a7ad9b964 100644 --- a/drivers/net/can/usb/ems_usb.c +++ b/drivers/net/can/usb/ems_usb.c | |||
@@ -1019,10 +1019,8 @@ static int ems_usb_probe(struct usb_interface *intf, | |||
1019 | 1019 | ||
1020 | dev->tx_msg_buffer = kzalloc(CPC_HEADER_SIZE + | 1020 | dev->tx_msg_buffer = kzalloc(CPC_HEADER_SIZE + |
1021 | sizeof(struct ems_cpc_msg), GFP_KERNEL); | 1021 | sizeof(struct ems_cpc_msg), GFP_KERNEL); |
1022 | if (!dev->tx_msg_buffer) { | 1022 | if (!dev->tx_msg_buffer) |
1023 | dev_err(&intf->dev, "Couldn't alloc Tx buffer\n"); | ||
1024 | goto cleanup_intr_in_buffer; | 1023 | goto cleanup_intr_in_buffer; |
1025 | } | ||
1026 | 1024 | ||
1027 | usb_set_intfdata(intf, dev); | 1025 | usb_set_intfdata(intf, dev); |
1028 | 1026 | ||