diff options
| author | Marcel Holtmann <marcel@holtmann.org> | 2013-10-11 10:00:59 -0400 |
|---|---|---|
| committer | Johan Hedberg <johan.hedberg@intel.com> | 2013-10-11 10:57:27 -0400 |
| commit | 2e1745583581500097a1a61dfb4e42b83f1a2c45 (patch) | |
| tree | 602765fb86c27630ae653d58e222ba64903c6c71 | |
| parent | 3c69454afa29c8214f86534ac95565b902a0888c (diff) | |
Bluetooth: Remove pointless parameter check in bt3c_hci_send_frame()
The hdev parameter of bt3c_hci_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>
| -rw-r--r-- | drivers/bluetooth/bt3c_cs.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/bluetooth/bt3c_cs.c b/drivers/bluetooth/bt3c_cs.c index fcd5fe993ad0..73d87994d028 100644 --- a/drivers/bluetooth/bt3c_cs.c +++ b/drivers/bluetooth/bt3c_cs.c | |||
| @@ -417,16 +417,9 @@ static int bt3c_hci_close(struct hci_dev *hdev) | |||
| 417 | 417 | ||
| 418 | static int bt3c_hci_send_frame(struct hci_dev *hdev, struct sk_buff *skb) | 418 | static int bt3c_hci_send_frame(struct hci_dev *hdev, struct sk_buff *skb) |
| 419 | { | 419 | { |
| 420 | bt3c_info_t *info; | 420 | bt3c_info_t *info = hci_get_drvdata(hdev); |
| 421 | unsigned long flags; | 421 | unsigned long flags; |
| 422 | 422 | ||
| 423 | if (!hdev) { | ||
| 424 | BT_ERR("Frame for unknown HCI device (hdev=NULL)"); | ||
| 425 | return -ENODEV; | ||
| 426 | } | ||
| 427 | |||
| 428 | info = hci_get_drvdata(hdev); | ||
| 429 | |||
| 430 | switch (bt_cb(skb)->pkt_type) { | 423 | switch (bt_cb(skb)->pkt_type) { |
| 431 | case HCI_COMMAND_PKT: | 424 | case HCI_COMMAND_PKT: |
| 432 | hdev->stat.cmd_tx++; | 425 | hdev->stat.cmd_tx++; |
