diff options
author | Andrei Emeltchenko <andrei.emeltchenko@intel.com> | 2012-11-14 10:39:30 -0500 |
---|---|---|
committer | Gustavo Padovan <gustavo.padovan@collabora.co.uk> | 2012-11-19 16:30:00 -0500 |
commit | a514b17fab51c1433db920d76cf8ddda959e5da0 (patch) | |
tree | aedaf1cdddec615c2938813b57c7e5fc18c9199a /net/bluetooth/amp.c | |
parent | 522db70286fcce420e5fc1be107912fbd3806323 (diff) |
Bluetooth: Refactor locking in amp_physical_cfm
Remove locking from l2cap_physical_cfm and lock chan inside
amp_physical_cfm.
Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Diffstat (limited to 'net/bluetooth/amp.c')
-rw-r--r-- | net/bluetooth/amp.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/net/bluetooth/amp.c b/net/bluetooth/amp.c index 4b2fea6c1c2a..eaf473ffeef9 100644 --- a/net/bluetooth/amp.c +++ b/net/bluetooth/amp.c | |||
@@ -386,13 +386,17 @@ void amp_physical_cfm(struct hci_conn *bredr_hcon, struct hci_conn *hs_hcon) | |||
386 | 386 | ||
387 | bredr_chan = mgr->bredr_chan; | 387 | bredr_chan = mgr->bredr_chan; |
388 | 388 | ||
389 | l2cap_chan_lock(bredr_chan); | ||
390 | |||
389 | set_bit(FLAG_EFS_ENABLE, &bredr_chan->flags); | 391 | set_bit(FLAG_EFS_ENABLE, &bredr_chan->flags); |
390 | bredr_chan->remote_amp_id = hs_hcon->remote_id; | 392 | bredr_chan->remote_amp_id = hs_hcon->remote_id; |
391 | bredr_chan->hs_hcon = hs_hcon; | 393 | bredr_chan->hs_hcon = hs_hcon; |
392 | bredr_chan->conn->mtu = hs_hcon->hdev->block_mtu; | 394 | bredr_chan->conn->mtu = hs_hcon->hdev->block_mtu; |
393 | bredr_chan->fcs = L2CAP_FCS_NONE; | 395 | bredr_chan->fcs = L2CAP_FCS_NONE; |
394 | 396 | ||
395 | l2cap_physical_cfm(bredr_chan, 0); | 397 | __l2cap_physical_cfm(bredr_chan, 0); |
398 | |||
399 | l2cap_chan_unlock(bredr_chan); | ||
396 | 400 | ||
397 | hci_dev_put(bredr_hdev); | 401 | hci_dev_put(bredr_hdev); |
398 | } | 402 | } |