greybus: es1: no need to assign a variable we return directly
authorGreg Kroah-Hartman <greg@kroah.com>
Wed, 21 Jan 2015 01:47:21 +0000 (09:47 +0800)
committerGreg 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

index 96c7c81..2ec5d7b 100644 (file)
@@ -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)