diff options
author | Marcel Holtmann <marcel@holtmann.org> | 2013-10-11 10:01:01 -0400 |
---|---|---|
committer | Johan Hedberg <johan.hedberg@intel.com> | 2013-10-11 11:00:02 -0400 |
commit | 6b5924bfeac1321118fbc54313fc3c25ba1a2cb5 (patch) | |
tree | 2cd5152e9c711e7bbef71ccf62dc99b99e436208 /drivers/bluetooth/btuart_cs.c | |
parent | 7648d28f28f6a661b660948b34c6ea15e163b772 (diff) |
Bluetooth: Remove pointless parameter check in btuart_hci_send_frame()
The hdev parameter of btuart_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>
Diffstat (limited to 'drivers/bluetooth/btuart_cs.c')
-rw-r--r-- | drivers/bluetooth/btuart_cs.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/bluetooth/btuart_cs.c b/drivers/bluetooth/btuart_cs.c index f567cd8424c3..a03ecc22a561 100644 --- a/drivers/bluetooth/btuart_cs.c +++ b/drivers/bluetooth/btuart_cs.c | |||
@@ -425,14 +425,7 @@ static int btuart_hci_close(struct hci_dev *hdev) | |||
425 | 425 | ||
426 | static int btuart_hci_send_frame(struct hci_dev *hdev, struct sk_buff *skb) | 426 | static int btuart_hci_send_frame(struct hci_dev *hdev, struct sk_buff *skb) |
427 | { | 427 | { |
428 | btuart_info_t *info; | 428 | btuart_info_t *info = hci_get_drvdata(hdev); |
429 | |||
430 | if (!hdev) { | ||
431 | BT_ERR("Frame for unknown HCI device (hdev=NULL)"); | ||
432 | return -ENODEV; | ||
433 | } | ||
434 | |||
435 | info = hci_get_drvdata(hdev); | ||
436 | 429 | ||
437 | switch (bt_cb(skb)->pkt_type) { | 430 | switch (bt_cb(skb)->pkt_type) { |
438 | case HCI_COMMAND_PKT: | 431 | case HCI_COMMAND_PKT: |