On Mon, Nov 11, 2013 at 02:06:21AM -0200, Luca Bartolacci wrote: > I'm not sure if the change is justified. > An other question is. Can i "clean" a complete directory (Like > /bluetooth) an put it all in one patch? That way is more easy for you? > Im asking all these questions to understand what is better to avoid > spam to the others. > Im sure the these questions are dumb. But i really want to help to this project. > This change introduces bugs. No signed-off-by line either, btw. regards, dan carpenter > index 3138699..08fba91 100644 > --- a/drivers/bluetooth/bfusb.c > +++ b/drivers/bluetooth/bfusb.c > @@ -131,8 +131,10 @@ static int bfusb_send_bulk(struct bfusb_data *data, struct > > BT_DBG("bfusb %p skb %p len %d", data, skb, skb->len); > > - if (!urb && !(urb = usb_alloc_urb(0, GFP_ATOMIC))) > - return -ENOMEM; > + if (!urb) > + urb = usb_alloc_urb(0, GFP_ATOMIC); > + if (!urb) > + return -ENOMEM; > > pipe = usb_sndbulkpipe(data->udev, data->bulk_out_ep); > > > @@ -218,8 +220,10 @@ static int bfusb_rx_submit(struct bfusb_data *data, struct > > BT_DBG("bfusb %p urb %p", data, urb); > > - if (!urb && !(urb = usb_alloc_urb(0, GFP_ATOMIC))) > - return -ENOMEM; > + if (!urb) > + urb = usb_alloc_urb(0, GFP_ATOMIC); > + if (!urb) > + return -ENOMEM; > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html