diff options
| author | Andre Guedes | 2013-03-08 15:20:17 +0100 |
|---|---|---|
| committer | Gustavo Padovan | 2013-03-09 21:11:23 +0100 |
| commit | e348fe6bbab85c513816d2536ffabac4be016442 (patch) | |
| tree | 24a4ec5bbb40b56012e0ef0414de61f5b977ca6d /net/bluetooth/hci_core.c | |
| parent | Bluetooth: HCI request error handling (diff) | |
| download | kernel-qcow2-linux-e348fe6bbab85c513816d2536ffabac4be016442.tar.gz kernel-qcow2-linux-e348fe6bbab85c513816d2536ffabac4be016442.tar.xz kernel-qcow2-linux-e348fe6bbab85c513816d2536ffabac4be016442.zip | |
Bluetooth: Make hci_req_add returning void
Since no one checks the returning value of hci_req_add and HCI
request errors are now handled in hci_req_run, we can make hci_
req_add returning void.
Signed-off-by: Andre Guedes <andre.guedes@openbossa.org>
Acked-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Diffstat (limited to 'net/bluetooth/hci_core.c')
| -rw-r--r-- | net/bluetooth/hci_core.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c index b432baafdf12..1c678757c83a 100644 --- a/net/bluetooth/hci_core.c +++ b/net/bluetooth/hci_core.c @@ -2529,7 +2529,7 @@ int hci_send_cmd(struct hci_dev *hdev, __u16 opcode, __u32 plen, void *param) } /* Queue a command to an asynchronous HCI request */ -int hci_req_add(struct hci_request *req, u16 opcode, u32 plen, void *param) +void hci_req_add(struct hci_request *req, u16 opcode, u32 plen, void *param) { struct hci_dev *hdev = req->hdev; struct sk_buff *skb; @@ -2541,15 +2541,13 @@ int hci_req_add(struct hci_request *req, u16 opcode, u32 plen, void *param) BT_ERR("%s no memory for command (opcode 0x%4.4x)", hdev->name, opcode); req->err = -ENOMEM; - return -ENOMEM; + return; } if (skb_queue_empty(&req->cmd_q)) bt_cb(skb)->req.start = true; skb_queue_tail(&req->cmd_q, skb); - - return 0; } /* Get data from the previously sent command */ |