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.
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
