From: Johan Hedberg Date: Wed, 28 Jan 2015 17:55:59 +0000 (+0200) Subject: Bluetooth: btusb: Remove redundant call to btusb_free_frags() X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=838f66e3ab5fcaa05fbf33acf189541078352685;p=openwrt%2Fstaging%2Fblogic.git Bluetooth: btusb: Remove redundant call to btusb_free_frags() The btusb_disconnect() callback calls hci_unregister_dev() which in turn calls btusb_close() if the HCI device is powered. The btusb_close() function in turn will call btusb_free_frags(). It's therefore unnecessary to have another call to btusb_free_frags() in the btusb_disconnect() function. Besides the redundancy the second call seems to also cause some strange stability issues which this patch then also fixes. Signed-off-by: Johan Hedberg Signed-off-by: Marcel Holtmann --- diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c index 25ca93d13209..70513b8d4d4a 100644 --- a/drivers/bluetooth/btusb.c +++ b/drivers/bluetooth/btusb.c @@ -2767,7 +2767,6 @@ static void btusb_disconnect(struct usb_interface *intf) else if (data->isoc) usb_driver_release_interface(&btusb_driver, data->isoc); - btusb_free_frags(data); hci_free_dev(hdev); }