diff options
| author | Marcel Holtmann <marcel@holtmann.org> | 2013-10-11 07:01:02 -0700 |
|---|---|---|
| committer | Johan Hedberg <johan.hedberg@intel.com> | 2013-10-11 17:00:08 +0200 |
| commit | 03b31868a463c48d6ebd8825326ee8310fce62c3 (patch) | |
| tree | a2bc240dd99922bda8ae052a7c06eaf16f9492cc | |
| parent | 6b5924bfeac1321118fbc54313fc3c25ba1a2cb5 (diff) | |
| download | op-kernel-dev-03b31868a463c48d6ebd8825326ee8310fce62c3.zip op-kernel-dev-03b31868a463c48d6ebd8825326ee8310fce62c3.tar.gz | |
Bluetooth: Remove pointless parameter check in dtl1_hci_send_frame()
The hdev parameter of dtl1_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/dtl1_cs.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/bluetooth/dtl1_cs.c b/drivers/bluetooth/dtl1_cs.c index ad1a2f9..52eed1f 100644 --- a/drivers/bluetooth/dtl1_cs.c +++ b/drivers/bluetooth/dtl1_cs.c @@ -384,17 +384,10 @@ static int dtl1_hci_close(struct hci_dev *hdev) static int dtl1_hci_send_frame(struct hci_dev *hdev, struct sk_buff *skb) { - dtl1_info_t *info; + dtl1_info_t *info = hci_get_drvdata(hdev); struct sk_buff *s; nsh_t nsh; - if (!hdev) { - BT_ERR("Frame for unknown HCI device (hdev=NULL)"); - return -ENODEV; - } - - info = hci_get_drvdata(hdev); - switch (bt_cb(skb)->pkt_type) { case HCI_COMMAND_PKT: hdev->stat.cmd_tx++; |
