linux-stable.git/drivers/bluetooth/bfusb.c, branch linux-4.8.y Linux kernel stable tree Bluetooth: bfusb: Fix the return error code 2015-12-22T14:25:33+00:00 Syam Sidhardhan s.syam@samsung.com 2015-12-22T14:00:20+00:00 7ddb69222d0fdc0bc83b60dbb21d9864ebf2907e -ENOMEM is the appropriate error code instead of -EIO. Signed-off-by: Syam Sidhardhan <s.syam@samsung.com> Signed-off-by: Marcel Holtmann <marcel@holtmann.org> 
-ENOMEM is the appropriate error code instead of -EIO.

Signed-off-by: Syam Sidhardhan <s.syam@samsung.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Bluetooth: bfusb: Remove redundant error message 2015-12-22T14:25:33+00:00 Syam Sidhardhan s.syam@samsung.com 2015-12-22T14:00:19+00:00 f71e823b8776b3ce0073087d7b8ec8a7d31fe818 devm_kzalloc prints its own OOM message upon failure. Signed-off-by: Syam Sidhardhan <s.syam@samsung.com> Signed-off-by: Marcel Holtmann <marcel@holtmann.org> 
devm_kzalloc prints its own OOM message upon failure.

Signed-off-by: Syam Sidhardhan <s.syam@samsung.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Bluetooth: Use new hci_skb_pkt_* wrappers for drivers 2015-11-19T16:50:27+00:00 Marcel Holtmann marcel@holtmann.org 2015-11-05T06:33:56+00:00 618e8bc228cda7b8c517caac40a45ee909b8672d The new hci_skb_pkt_* wrappers are mainly intented for drivers to require less knowledge about bt_cb(sbk) handling. So after converting the core packet handling, convert all drivers. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> 
The new hci_skb_pkt_* wrappers are mainly intented for drivers to
require less knowledge about bt_cb(sbk) handling. So after converting
the core packet handling, convert all drivers.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Bluetooth: Move handling of HCI_RUNNING flag into core 2015-10-05T07:30:25+00:00 Marcel Holtmann marcel@holtmann.org 2015-10-04T21:34:02+00:00 e9ca8bf157f2b45f8f670517c96da313083ee9b2 Setting and clearing of HCI_RUNNING flag in each and every driver is just duplicating the same code all over the place. So instead of having the driver do it in their hdev->open and hdev->close callbacks, set it globally in the core transport handling. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> 
Setting and clearing of HCI_RUNNING flag in each and every driver is
just duplicating the same code all over the place. So instead of having
the driver do it in their hdev->open and hdev->close callbacks, set it
globally in the core transport handling.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Bluetooth: Move HCI_RUNNING check into hci_send_frame 2015-10-05T07:30:10+00:00 Marcel Holtmann marcel@holtmann.org 2015-10-04T21:34:01+00:00 73d0d3c8671190ea982a8e79a7c79fbfe88f8f47 In all callbacks for hdev->send the status of HCI_RUNNING is checked. So instead of repeating that code in every driver, move the check into the hci_send_frame function before calling hdev->send. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> 
In all callbacks for hdev->send the status of HCI_RUNNING is checked. So
instead of repeating that code in every driver, move the check into the
hci_send_frame function before calling hdev->send.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Bluetooth: bfusb: Coding style fix reported by coccinelle 2015-07-23T15:10:49+00:00 Prasanna Karthik mkarthi3@visteon.com 2015-07-05T09:49:27+00:00 a03e33da5f1f10cc93bd5de140a131dac1db97f0 Removed semicolon at the end of switch case statement Signed-off-by: Prasanna Karthik <mkarthi3@visteon.com> Signed-off-by: Marcel Holtmann <marcel@holtmann.org> 
Removed semicolon at the end of switch case statement

Signed-off-by: Prasanna Karthik <mkarthi3@visteon.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Bluetooth: bfusb: Set the HCI_QUIRK_BROKEN_LOCAL_COMMANDS quirk 2014-12-26T18:16:12+00:00 Marcel Holtmann marcel@holtmann.org 2014-12-26T03:42:32+00:00 9ef80d40e901feb875660101dabdeddaa27e837c The AVM BlueFritz! 1.0 USB dongles do not support the HCI command for reading the local supported commands. So set this quirk to let the core know about it. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> 
The AVM BlueFritz! 1.0 USB dongles do not support the HCI command for
reading the local supported commands. So set this quirk to let the
core know about it.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Bluetooth: Remove assignments in if-statements 2014-03-05T18:52:17+00:00 Valentin Ilie valentin.ilie@gmail.com 2013-08-12T15:46:00+00:00 a08b15e66e8ec700992641cf8ec015032e8365c8 Remove assignment in if-statements to be consistent with the coding style. Signed-off-by: Valentin Ilie <valentin.ilie@gmail.com> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> 
Remove assignment in if-statements to be consistent with the coding
style.

Signed-off-by: Valentin Ilie <valentin.ilie@gmail.com>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Bluetooth: Declare bfusb_table[] as const 2013-10-11T15:05:20+00:00 Marcel Holtmann marcel@holtmann.org 2013-10-11T14:46:19+00:00 9712d59ab60141fa05b276fe85dce736f329ad00 The bfusb_table[] device table can be declared as const Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> 
The bfusb_table[] device table can be declared as const

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Bluetooth: Remove pointless parameter check in bfusb_send_frame() 2013-10-11T14:52:37+00:00 Marcel Holtmann marcel@holtmann.org 2013-10-11T14:00:57+00:00 aae26277121a99509300c8024cf88d313f18f367 The hdev parameter of bfusb_send_frame() is always valid. If it were not valid, then it would have crashed earlier in the call chain. Signed-off-by: Marcel Holtmann <marcel@holtmann.org> Signed-off-by: Johan Hedberg <johan.hedberg@intel.com> 
The hdev parameter of bfusb_send_frame() is always valid. If it were
not valid, then it would have crashed earlier in the call chain.

Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>