projects
/
platform
/
kernel
/
linux-starfive.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
580c47d
)
greybus: es1: no need to assign a variable we return directly
author
Greg Kroah-Hartman
<greg@kroah.com>
Wed, 21 Jan 2015 01:47:21 +0000
(09:47 +0800)
committer
Greg Kroah-Hartman
<greg@kroah.com>
Wed, 21 Jan 2015 01:47:21 +0000
(09:47 +0800)
In next_free_urb(), just return usb_alloc_urb(), don't waste the time
assigning it to a local variable that we then return.
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
drivers/staging/greybus/gb-es1.c
patch
|
blob
|
history
diff --git
a/drivers/staging/greybus/gb-es1.c
b/drivers/staging/greybus/gb-es1.c
index
96c7c81
..
2ec5d7b
100644
(file)
--- a/
drivers/staging/greybus/gb-es1.c
+++ b/
drivers/staging/greybus/gb-es1.c
@@
-179,9
+179,7
@@
static struct urb *next_free_urb(struct es1_ap_dev *es1, gfp_t gfp_mask)
*/
dev_err(&es1->usb_dev->dev,
"No free CPort OUT urbs, having to dynamically allocate one!\n");
- urb = usb_alloc_urb(0, gfp_mask);
-
- return urb;
+ return usb_alloc_urb(0, gfp_mask);
}
static void free_urb(struct es1_ap_dev *es1, struct urb *urb)