diff options
author | Gustavo F. Padovan <padovan@profusion.mobi> | 2011-06-17 11:58:41 -0400 |
---|---|---|
committer | Gustavo F. Padovan <padovan@profusion.mobi> | 2011-12-18 14:07:55 -0500 |
commit | b9cc553f12d14b692d0fcb607d28db783da68139 (patch) | |
tree | f1218f45f6389b1aa970ec34526d587ad2d58c68 | |
parent | db323f2fff0ded058f033df6235e8c2be4146bfd (diff) |
Bluetooth: hci_conn_auto_accept() doesn't need locking
It doesn't really touch any sensitive information about hdev. So no need
to lock here.
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
-rw-r--r-- | net/bluetooth/hci_conn.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index 7d88a6142092..e6d8a220b5d9 100644 --- a/net/bluetooth/hci_conn.c +++ b/net/bluetooth/hci_conn.c | |||
@@ -362,12 +362,8 @@ static void hci_conn_auto_accept(unsigned long arg) | |||
362 | struct hci_conn *conn = (void *) arg; | 362 | struct hci_conn *conn = (void *) arg; |
363 | struct hci_dev *hdev = conn->hdev; | 363 | struct hci_dev *hdev = conn->hdev; |
364 | 364 | ||
365 | hci_dev_lock(hdev); | ||
366 | |||
367 | hci_send_cmd(hdev, HCI_OP_USER_CONFIRM_REPLY, sizeof(conn->dst), | 365 | hci_send_cmd(hdev, HCI_OP_USER_CONFIRM_REPLY, sizeof(conn->dst), |
368 | &conn->dst); | 366 | &conn->dst); |
369 | |||
370 | hci_dev_unlock(hdev); | ||
371 | } | 367 | } |
372 | 368 | ||
373 | struct hci_conn *hci_conn_add(struct hci_dev *hdev, int type, bdaddr_t *dst) | 369 | struct hci_conn *hci_conn_add(struct hci_dev *hdev, int type, bdaddr_t *dst) |