Bluetooth: AMP: Factor out amp_ctrl_add

Add ctrl_id parameter to amp_ctrl_add since we always set it
after function ctrl is created.

Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>

1 files changed, 2 insertions, 6 deletions

diff --git a/net/bluetooth/a2mp.c b/net/bluetooth/a2mp.c
index d4946b591b71..88a4b583d218 100644
--- a/net/bluetooth/a2mp.c
+++ b/net/bluetooth/a2mp.c
@@ -316,12 +316,10 @@ static int a2mp_getinfo_rsp(struct amp_mgr *mgr, struct sk_buff *skb,
         if (rsp->status)
                 return -EINVAL;
 
-        ctrl = amp_ctrl_add(mgr);
+        ctrl = amp_ctrl_add(mgr, rsp->id);
         if (!ctrl)
                 return -ENOMEM;
 
-        ctrl->id = rsp->id;
-
         req.id = rsp->id;
         a2mp_send(mgr, A2MP_GETAMPASSOC_REQ, __next_ident(mgr), sizeof(req),
                   &req);
@@ -461,7 +459,7 @@ static int a2mp_createphyslink_req(struct amp_mgr *mgr, struct sk_buff *skb,
 
         ctrl = amp_ctrl_lookup(mgr, rsp.remote_id);
         if (!ctrl) {
-                ctrl = amp_ctrl_add(mgr);
+                ctrl = amp_ctrl_add(mgr, rsp.remote_id);
                 if (ctrl) {
                         amp_ctrl_get(ctrl);
                 } else {
@@ -474,8 +472,6 @@ static int a2mp_createphyslink_req(struct amp_mgr *mgr, struct sk_buff *skb,
                 size_t assoc_len = le16_to_cpu(hdr->len) - sizeof(*req);
                 u8 *assoc;
 
-                ctrl->id = rsp.remote_id;
-
                 assoc = kzalloc(assoc_len, GFP_KERNEL);
                 if (!assoc) {
                         amp_ctrl_put(ctrl);