From 2ec515bfff5b13c4efd0d260683b1213f3069ab4 Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman Date: Wed, 21 Jan 2015 09:47:21 +0800 Subject: [PATCH] greybus: es1: no need to assign a variable we return directly 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 --- drivers/staging/greybus/gb-es1.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/greybus/gb-es1.c b/drivers/staging/greybus/gb-es1.c index 96c7c816b590..2ec5d7b403a3 100644 --- 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)