aboutsummaryrefslogtreecommitdiffstats
path: root/net/bluetooth/amp.c
diff options
context:
space:
mode:
authorAndrei Emeltchenko <andrei.emeltchenko@intel.com>2012-11-14 10:39:30 -0500
committerGustavo Padovan <gustavo.padovan@collabora.co.uk>2012-11-19 16:30:00 -0500
commita514b17fab51c1433db920d76cf8ddda959e5da0 (patch)
treeaedaf1cdddec615c2938813b57c7e5fc18c9199a /net/bluetooth/amp.c
parent522db70286fcce420e5fc1be107912fbd3806323 (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.c6
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}