OSDN Git Service

Replace BT_ADDR with bd_addr_t in bta_sys related code
authorJakub Pawlowski <jpawlowski@google.com>
Fri, 16 Jun 2017 19:52:23 +0000 (12:52 -0700)
committerJakub Pawlowski <jpawlowski@google.com>
Tue, 20 Jun 2017 02:44:10 +0000 (19:44 -0700)
Test: compilation test
Change-Id: Idab6e9c03621c5540b4a3a7fa537dff71a25ce3f

22 files changed:
bta/ag/bta_ag_act.cc
bta/ag/bta_ag_cmd.cc
bta/ag/bta_ag_int.h
bta/ag/bta_ag_main.cc
bta/ag/bta_ag_sco.cc
bta/av/bta_av_aact.cc
bta/av/bta_av_act.cc
bta/av/bta_av_main.cc
bta/dm/bta_dm_act.cc
bta/gatt/bta_gattc_act.cc
bta/gatt/bta_gatts_act.cc
bta/hd/bta_hd_act.cc
bta/hf_client/bta_hf_client_act.cc
bta/hf_client/bta_hf_client_int.h
bta/hf_client/bta_hf_client_main.cc
bta/hf_client/bta_hf_client_sco.cc
bta/hh/bta_hh_act.cc
bta/hl/bta_hl_act.cc
bta/jv/bta_jv_act.cc
bta/pan/bta_pan_act.cc
bta/sys/bta_sys.h
bta/sys/bta_sys_conn.cc

index 4179a57..56670d6 100644 (file)
@@ -195,7 +195,7 @@ void bta_ag_start_open(tBTA_AG_SCB* p_scb, tBTA_AG_DATA* p_data) {
     /* Let the incoming connection goes through.                        */
     /* Issue collision for this scb for now.                            */
     /* We will decide what to do when we find incoming connetion later. */
-    bta_ag_collision_cback(0, BTA_ID_AG, 0, to_BD_ADDR(p_scb->peer_addr));
+    bta_ag_collision_cback(0, BTA_ID_AG, 0, &p_scb->peer_addr);
     return;
   }
 
@@ -397,7 +397,7 @@ void bta_ag_rfc_close(tBTA_AG_SCB* p_scb, UNUSED_ATTR tBTA_AG_DATA* p_data) {
   close.hdr.app_id = p_scb->app_id;
   close.bd_addr = p_scb->peer_addr;
 
-  bta_sys_conn_close(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_conn_close(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
   /* call close call-out */
   bta_ag_co_data_close(close.hdr.handle);
@@ -430,7 +430,7 @@ void bta_ag_rfc_close(tBTA_AG_SCB* p_scb, UNUSED_ATTR tBTA_AG_DATA* p_data) {
     }
 
     if (!num_active_conn) {
-      bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
     }
 
   }
@@ -473,7 +473,7 @@ void bta_ag_rfc_open(tBTA_AG_SCB* p_scb, tBTA_AG_DATA* p_data) {
   bta_ag_co_data_open(bta_ag_scb_to_idx(p_scb),
                       bta_ag_svc_id[p_scb->conn_service]);
 
-  bta_sys_conn_open(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_conn_open(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
   bta_ag_cback_open(p_scb, NULL, BTA_AG_SUCCESS);
 
@@ -604,14 +604,14 @@ void bta_ag_rfc_data(tBTA_AG_SCB* p_scb, UNUSED_ATTR tBTA_AG_DATA* p_data) {
     }
 
     /* run AT command interpreter on data */
-    bta_sys_busy(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_busy(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
     bta_ag_at_parse(&p_scb->at_cb, buf, len);
     if ((p_scb->sco_idx != BTM_INVALID_SCO_INDEX) &&
         bta_ag_sco_is_open(p_scb)) {
       APPL_TRACE_DEBUG("%s change link policy for SCO", __func__);
-      bta_sys_sco_open(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_sco_open(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
     } else {
-      bta_sys_idle(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_idle(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
     }
 
     /* no more data to read, we're done */
@@ -760,7 +760,7 @@ void bta_ag_svc_conn_open(tBTA_AG_SCB* p_scb,
 
     if ((p_scb->call_ind != BTA_AG_CALL_INACTIVE) ||
         (p_scb->callsetup_ind != BTA_AG_CALLSETUP_NONE)) {
-      bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
     }
 
     (*bta_ag_cb.p_cback)(BTA_AG_CONN_EVT, (tBTA_AG*)&evt);
@@ -784,13 +784,13 @@ void bta_ag_ci_rx_data(tBTA_AG_SCB* p_scb, tBTA_AG_DATA* p_data) {
 
   APPL_TRACE_DEBUG("bta_ag_ci_rx_data:");
   /* send to RFCOMM */
-  bta_sys_busy(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_busy(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
   PORT_WriteData(p_scb->conn_handle, p_data_area, strlen(p_data_area), &len);
   if ((p_scb->sco_idx != BTM_INVALID_SCO_INDEX) && bta_ag_sco_is_open(p_scb)) {
     APPL_TRACE_DEBUG("bta_ag_rfc_data, change link policy for SCO");
-    bta_sys_sco_open(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_sco_open(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
   } else {
-    bta_sys_idle(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_idle(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
   }
 }
 
index ba409f3..da8bf01 100644 (file)
@@ -1304,7 +1304,7 @@ void bta_ag_hsp_result(tBTA_AG_SCB* p_scb, tBTA_AG_API_RESULT* p_result) {
 
     case BTA_AG_IN_CALL_RES:
       /* tell sys to stop av if any */
-      bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
       /* if sco already opened or no inband ring send ring now */
       if (bta_ag_sco_is_open(p_scb) || !bta_ag_inband_enabled(p_scb) ||
@@ -1352,8 +1352,7 @@ void bta_ag_hsp_result(tBTA_AG_SCB* p_scb, tBTA_AG_API_RESULT* p_result) {
         bta_ag_sco_close(p_scb, (tBTA_AG_DATA*)p_result);
       } else {
         /* if av got suspended by this call, let it resume. */
-        bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id,
-                          to_BD_ADDR(p_scb->peer_addr));
+        bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
       }
       break;
 
@@ -1401,7 +1400,7 @@ void bta_ag_hfp_result(tBTA_AG_SCB* p_scb, tBTA_AG_API_RESULT* p_result) {
 
     case BTA_AG_IN_CALL_RES:
       /* tell sys to stop av if any */
-      bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
       /* store caller id string.
        * append type info at the end.
@@ -1528,8 +1527,7 @@ void bta_ag_hfp_result(tBTA_AG_SCB* p_scb, tBTA_AG_API_RESULT* p_result) {
         bta_ag_send_call_inds(p_scb, p_result->result);
 
         /* if av got suspended by this call, let it resume. */
-        bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id,
-                          to_BD_ADDR(p_scb->peer_addr));
+        bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
       }
       break;
 
index 8614e52..dbe26dd 100644 (file)
@@ -331,7 +331,8 @@ extern void bta_ag_sm_execute(tBTA_AG_SCB* p_scb, uint16_t event,
                               tBTA_AG_DATA* p_data);
 extern bool bta_ag_hdl_event(BT_HDR* p_msg);
 extern void bta_ag_collision_cback(tBTA_SYS_CONN_STATUS status, uint8_t id,
-                                   uint8_t app_id, BD_ADDR peer_addr);
+                                   uint8_t app_id,
+                                   const bt_bdaddr_t* peer_addr);
 extern void bta_ag_resume_open(tBTA_AG_SCB* p_scb);
 
 /* SDP functions */
index 6aca837..d36c072 100644 (file)
@@ -511,12 +511,13 @@ static void bta_ag_collision_timer_cback(void* data) {
  *
  ******************************************************************************/
 void bta_ag_collision_cback(UNUSED_ATTR tBTA_SYS_CONN_STATUS status, uint8_t id,
-                            UNUSED_ATTR uint8_t app_id, BD_ADDR peer_addr) {
+                            UNUSED_ATTR uint8_t app_id,
+                            const bt_bdaddr_t* peer_addr) {
   uint16_t handle;
   tBTA_AG_SCB* p_scb;
 
   /* Check if we have opening scb for the peer device. */
-  handle = bta_ag_idx_by_bdaddr(&from_BD_ADDR(peer_addr));
+  handle = bta_ag_idx_by_bdaddr(peer_addr);
   p_scb = bta_ag_scb_by_idx(handle);
 
   if (p_scb && (p_scb->state == BTA_AG_OPENING_ST)) {
index 1adfb5a..6d4ad31 100644 (file)
@@ -421,7 +421,7 @@ static void bta_ag_create_sco(tBTA_AG_SCB* p_scb, bool is_orig) {
     p_scb->inuse_codec = esco_codec;
 
     /* tell sys to stop av if any */
-    bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
     /* Send pending commands to create SCO connection to peer */
     bta_ag_create_pending_sco(p_scb, bta_ag_cb.sco.is_local);
@@ -552,7 +552,7 @@ void bta_ag_codec_negotiate(tBTA_AG_SCB* p_scb) {
   if ((p_scb->codec_updated || p_scb->codec_fallback) &&
       (p_scb->peer_features & BTA_AG_PEER_FEAT_CODEC)) {
     /* Change the power mode to Active until SCO open is completed. */
-    bta_sys_busy(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_busy(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
     /* Send +BCS to the peer */
     bta_ag_send_bcs(p_scb, NULL);
@@ -1252,7 +1252,7 @@ void bta_ag_sco_conn_open(tBTA_AG_SCB* p_scb,
                           UNUSED_ATTR tBTA_AG_DATA* p_data) {
   bta_ag_sco_event(p_scb, BTA_AG_SCO_CONN_OPEN_E);
 
-  bta_sys_sco_open(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_sco_open(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
 #if (BTM_SCO_HCI_INCLUDED == TRUE)
   /* open SCO codec if SCO is routed through transport */
@@ -1294,14 +1294,14 @@ void bta_ag_sco_conn_close(tBTA_AG_SCB* p_scb,
     /* Indicate if the closing of audio is because of transfer */
     bta_ag_sco_event(p_scb, BTA_AG_SCO_CONN_CLOSE_E);
 
-    bta_sys_sco_close(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_sco_close(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
 
     /* if av got suspended by this call, let it resume. */
     /* In case call stays alive regardless of sco, av should not be affected. */
     if (((p_scb->call_ind == BTA_AG_CALL_INACTIVE) &&
          (p_scb->callsetup_ind == BTA_AG_CALLSETUP_NONE)) ||
         (p_scb->post_sco == BTA_AG_POST_SCO_CALL_END)) {
-      bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_sco_unuse(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
     }
 
     /* call app callback */
@@ -1333,7 +1333,7 @@ void bta_ag_sco_conn_rsp(tBTA_AG_SCB* p_scb,
       bta_ag_cb.sco.state == BTA_AG_SCO_CLOSE_XFER_ST ||
       bta_ag_cb.sco.state == BTA_AG_SCO_OPEN_XFER_ST) {
     /* tell sys to stop av if any */
-    bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_sco_use(BTA_ID_AG, p_scb->app_id, p_scb->peer_addr);
     /* When HS initiated SCO, it cannot be WBS. */
 #if (BTM_SCO_HCI_INCLUDED == TRUE)
     /* Configure the transport being used */
index 2d678b3..143285b 100644 (file)
@@ -818,8 +818,7 @@ void bta_av_role_res(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
       p_scb->wait &= ~BTA_AV_WAIT_ROLE_SW_BITS;
       if (p_data->role_res.hci_status != HCI_SUCCESS) {
         p_scb->role &= ~BTA_AV_ROLE_START_INT;
-        bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt,
-                     to_BD_ADDR(p_scb->peer_addr));
+        bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt, p_scb->peer_addr);
         /* start failed because of role switch. */
         start.chnl = p_scb->chnl;
         start.status = BTA_AV_FAIL_ROLE;
@@ -961,7 +960,7 @@ void bta_av_do_disc_a2dp(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
   p_scb->sec_mask = p_data->api_open.sec_mask;
   p_scb->use_rc = p_data->api_open.use_rc;
 
-  bta_sys_app_open(BTA_ID_AV, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_app_open(BTA_ID_AV, p_scb->app_id, p_scb->peer_addr);
 
   if (p_scb->skip_sdp == true) {
     tA2DP_Service a2dp_ser;
@@ -1268,7 +1267,7 @@ void bta_av_setconfig_rsp(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
         num > 1) {
       /* if SBC is used by the SNK as INT, discover req is not sent in
        * bta_av_config_ind.
-                 * call disc_res now */
+       * call disc_res now */
       /* this is called in A2DP SRC path only, In case of SINK we don't need it
        */
       if (local_sep == AVDT_TSEP_SRC)
@@ -1335,7 +1334,7 @@ void bta_av_str_opened(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
   L2CA_SetTxPriority(p_scb->l2c_cid, L2CAP_CHNL_PRIORITY_MEDIUM);
   L2CA_SetChnlFlushability(p_scb->l2c_cid, true);
 
-  bta_sys_conn_open(BTA_ID_AV, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_conn_open(BTA_ID_AV, p_scb->app_id, p_scb->peer_addr);
   memset(&p_scb->q_info, 0, sizeof(tBTA_AV_Q_INFO));
 
   p_scb->l2c_bufs = 0;
@@ -1892,13 +1891,12 @@ void bta_av_do_start(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
     policy |= HCI_ENABLE_MASTER_SLAVE_SWITCH;
   }
 
-  bta_sys_clear_policy(BTA_ID_AV, policy, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_clear_policy(BTA_ID_AV, policy, p_scb->peer_addr);
 
   if ((p_scb->started == false) &&
       ((p_scb->role & BTA_AV_ROLE_START_INT) == 0)) {
     p_scb->role |= BTA_AV_ROLE_START_INT;
-    bta_sys_busy(BTA_ID_AV, bta_av_cb.audio_open_cnt,
-                 to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_busy(BTA_ID_AV, bta_av_cb.audio_open_cnt, p_scb->peer_addr);
 
     AVDT_StartReq(&p_scb->avdt_handle, 1);
   } else if (p_scb->started) {
@@ -1934,12 +1932,11 @@ void bta_av_str_stopped(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
   APPL_TRACE_ERROR("%s: audio_open_cnt=%d, p_data %p", __func__,
                    bta_av_cb.audio_open_cnt, p_data);
 
-  bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt,
-               to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt, p_scb->peer_addr);
   if ((bta_av_cb.features & BTA_AV_FEAT_MASTER) == 0 ||
       bta_av_cb.audio_open_cnt == 1)
     policy |= HCI_ENABLE_MASTER_SLAVE_SWITCH;
-  bta_sys_set_policy(BTA_ID_AV, policy, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_set_policy(BTA_ID_AV, policy, p_scb->peer_addr);
 
   if (p_scb->co_started) {
     /* TODO(eisenbach): RE-IMPLEMENT USING VSC OR HAL EXTENSION
@@ -2259,8 +2256,7 @@ void bta_av_start_ok(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
   if (p_data && (p_data->hdr.offset != BTA_AV_RS_NONE)) {
     p_scb->wait &= ~BTA_AV_WAIT_ROLE_SW_BITS;
     if (p_data->hdr.offset == BTA_AV_RS_FAIL) {
-      bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt,
-                   to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt, p_scb->peer_addr);
       start.chnl = p_scb->chnl;
       start.status = BTA_AV_FAIL_ROLE;
       start.hndl = p_scb->hndl;
@@ -2297,15 +2293,14 @@ void bta_av_start_ok(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
      * first bit of p_scb->wait is cleared hence it ensures bt_av_start_ok is
      * not called
      * again from bta_av_save_caps.
-    */
+     */
     p_scb->wait &= ~BTA_AV_WAIT_ACP_CAPS_ON;
   }
 
   /* tell role manager to check M/S role */
-  bta_sys_conn_open(BTA_ID_AV, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_conn_open(BTA_ID_AV, p_scb->app_id, p_scb->peer_addr);
 
-  bta_sys_busy(BTA_ID_AV, bta_av_cb.audio_open_cnt,
-               to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_busy(BTA_ID_AV, bta_av_cb.audio_open_cnt, p_scb->peer_addr);
 
   if (p_scb->media_type == AVDT_MEDIA_TYPE_AUDIO) {
     /* in normal logic, conns should be bta_av_cb.audio_count - 1,
@@ -2340,15 +2335,15 @@ void bta_av_start_ok(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
     /* If sink starts stream, disable sniff mode here */
     if (!initiator) {
       /* If souce is the master role, disable role switch during streaming.
-      * Otherwise allow role switch, if source is slave.
-      * Because it would not hurt source, if the peer device wants source to be
-      * master */
+       * Otherwise allow role switch, if source is slave.
+       * Because it would not hurt source, if the peer device wants source to be
+       * master */
       if ((BTM_GetRole(p_scb->peer_addr, &cur_role) == BTM_SUCCESS) &&
           (cur_role == BTM_ROLE_MASTER)) {
         policy |= HCI_ENABLE_MASTER_SLAVE_SWITCH;
       }
 
-      bta_sys_clear_policy(BTA_ID_AV, policy, to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_clear_policy(BTA_ID_AV, policy, p_scb->peer_addr);
     }
 
     p_scb->role = new_role;
@@ -2395,14 +2390,13 @@ void bta_av_start_ok(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
  ******************************************************************************/
 void bta_av_start_failed(tBTA_AV_SCB* p_scb, UNUSED_ATTR tBTA_AV_DATA* p_data) {
   if (p_scb->started == false && p_scb->co_started == false) {
-    bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt,
-                 to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt, p_scb->peer_addr);
     notify_start_failed(p_scb);
   }
 
   bta_sys_set_policy(BTA_ID_AV,
                      (HCI_ENABLE_SNIFF_MODE | HCI_ENABLE_MASTER_SLAVE_SWITCH),
-                     to_BD_ADDR(p_scb->peer_addr));
+                     p_scb->peer_addr);
   p_scb->sco_suspend = false;
 }
 
@@ -2423,7 +2417,7 @@ void bta_av_str_closed(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
   if ((bta_av_cb.features & BTA_AV_FEAT_MASTER) == 0 ||
       bta_av_cb.audio_open_cnt == 1)
     policy |= HCI_ENABLE_MASTER_SLAVE_SWITCH;
-  bta_sys_set_policy(BTA_ID_AV, policy, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_set_policy(BTA_ID_AV, policy, p_scb->peer_addr);
   if (bta_av_cb.audio_open_cnt <= 1) {
     /* last connection - restore the allow switch flag */
     L2CA_SetDesireRole(L2CAP_ROLE_ALLOW_SWITCH);
@@ -2444,7 +2438,7 @@ void bta_av_str_closed(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
     event = BTA_AV_OPEN_EVT;
     p_scb->open_status = BTA_AV_SUCCESS;
 
-    bta_sys_conn_close(BTA_ID_AV, p_scb->app_id, to_BD_ADDR(p_scb->peer_addr));
+    bta_sys_conn_close(BTA_ID_AV, p_scb->app_id, p_scb->peer_addr);
     bta_av_cleanup(p_scb, p_data);
     (*bta_av_cb.p_cback)(event, &data);
   } else {
@@ -2459,8 +2453,7 @@ void bta_av_str_closed(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
       data.close.hndl = p_scb->hndl;
       event = BTA_AV_CLOSE_EVT;
 
-      bta_sys_conn_close(BTA_ID_AV, p_scb->app_id,
-                         to_BD_ADDR(p_scb->peer_addr));
+      bta_sys_conn_close(BTA_ID_AV, p_scb->app_id, p_scb->peer_addr);
       bta_av_cleanup(p_scb, p_data);
       (*bta_av_cb.p_cback)(event, &data);
     }
@@ -2499,9 +2492,9 @@ void bta_av_suspend_cfm(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
 
   if (p_scb->started == false) {
     /* handle the condition where there is a collision of SUSPEND req from
-    *either side
-    ** Second SUSPEND req could be rejected. Do not treat this as a failure
-    */
+     *either side
+     ** Second SUSPEND req could be rejected. Do not treat this as a failure
+     */
     APPL_TRACE_WARNING("%s: already suspended, ignore, err_code %d", __func__,
                        err_code);
     return;
@@ -2529,12 +2522,11 @@ void bta_av_suspend_cfm(tBTA_AV_SCB* p_scb, tBTA_AV_DATA* p_data) {
     p_scb->cong = false;
   }
 
-  bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt,
-               to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_idle(BTA_ID_AV, bta_av_cb.audio_open_cnt, p_scb->peer_addr);
   if ((bta_av_cb.features & BTA_AV_FEAT_MASTER) == 0 ||
       bta_av_cb.audio_open_cnt == 1)
     policy |= HCI_ENABLE_MASTER_SLAVE_SWITCH;
-  bta_sys_set_policy(BTA_ID_AV, policy, to_BD_ADDR(p_scb->peer_addr));
+  bta_sys_set_policy(BTA_ID_AV, policy, p_scb->peer_addr);
 
   /* in case that we received suspend_ind, we may need to call co_stop here */
   if (p_scb->co_started) {
index c53eb3f..c98e627 100644 (file)
@@ -1425,7 +1425,7 @@ void bta_av_sig_chg(tBTA_AV_DATA* p_data) {
 
             APPL_TRACE_DEBUG("%s: Re-start timer for AVDTP service", __func__);
             bta_sys_conn_open(BTA_ID_AV, p_cb->p_scb[xx]->app_id,
-                              to_BD_ADDR(p_cb->p_scb[xx]->peer_addr));
+                              p_cb->p_scb[xx]->peer_addr);
             /* Possible collision : need to avoid outgoing processing while the
              * timer is running */
             p_cb->p_scb[xx]->coll_mask = BTA_AV_COLL_INC_TMR;
@@ -1467,7 +1467,7 @@ void bta_av_sig_chg(tBTA_AV_DATA* p_data) {
             p_cb->p_scb[xx]->peer_addr == p_data->str_msg.bd_addr) {
           APPL_TRACE_DEBUG("%s: Closing timer for AVDTP service", __func__);
           bta_sys_conn_close(BTA_ID_AV, p_cb->p_scb[xx]->app_id,
-                             to_BD_ADDR(p_cb->p_scb[xx]->peer_addr));
+                             p_cb->p_scb[xx]->peer_addr);
         }
         mask = 1 << (xx + 1);
         if (((mask & p_lcb->conn_msk) || bta_av_cb.conn_lcb) &&
index cee5255..d85957b 100644 (file)
@@ -167,9 +167,9 @@ static void bta_av_rpc_conn(tBTA_AV_DATA* p_data);
 static void bta_av_api_to_ssm(tBTA_AV_DATA* p_data);
 
 static void bta_av_sco_chg_cback(tBTA_SYS_CONN_STATUS status, uint8_t id,
-                                 uint8_t app_id, BD_ADDR peer_addr);
+                                 uint8_t app_id, const bt_bdaddr_t* peer_addr);
 static void bta_av_sys_rs_cback(tBTA_SYS_CONN_STATUS status, uint8_t id,
-                                uint8_t app_id, BD_ADDR peer_addr);
+                                uint8_t app_id, const bt_bdaddr_t* peer_addr);
 
 /* action functions */
 const tBTA_AV_NSM_ACT bta_av_nsm_act[] = {
@@ -816,7 +816,7 @@ void bta_av_restore_switch(void) {
     if (p_cb->conn_audio == mask) {
       if (p_cb->p_scb[i]) {
         bta_sys_set_policy(BTA_ID_AV, HCI_ENABLE_MASTER_SLAVE_SWITCH,
-                           to_BD_ADDR(p_cb->p_scb[i]->peer_addr));
+                           p_cb->p_scb[i]->peer_addr);
       }
       break;
     }
@@ -833,7 +833,8 @@ void bta_av_restore_switch(void) {
  *
  ******************************************************************************/
 static void bta_av_sys_rs_cback(UNUSED_ATTR tBTA_SYS_CONN_STATUS status,
-                                uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+                                uint8_t id, uint8_t app_id,
+                                const bt_bdaddr_t* peer_addr) {
   int i;
   tBTA_AV_SCB* p_scb = NULL;
   uint8_t cur_role;
@@ -845,7 +846,7 @@ static void bta_av_sys_rs_cback(UNUSED_ATTR tBTA_SYS_CONN_STATUS status,
      * role change event */
     /* note that more than one SCB (a2dp & vdp) maybe waiting for this event */
     p_scb = bta_av_cb.p_scb[i];
-    if (p_scb && p_scb->peer_addr == from_BD_ADDR(peer_addr)) {
+    if (p_scb && p_scb->peer_addr == *peer_addr) {
       tBTA_AV_ROLE_RES* p_buf =
           (tBTA_AV_ROLE_RES*)osi_malloc(sizeof(tBTA_AV_ROLE_RES));
       APPL_TRACE_DEBUG("new_role:%d, hci_status:x%x hndl: x%x", id, app_id,
@@ -869,9 +870,9 @@ static void bta_av_sys_rs_cback(UNUSED_ATTR tBTA_SYS_CONN_STATUS status,
 
   /* restore role switch policy, if role switch failed */
   if ((HCI_SUCCESS != app_id) &&
-      (BTM_GetRole(from_BD_ADDR(peer_addr), &cur_role) == BTM_SUCCESS) &&
+      (BTM_GetRole(*peer_addr, &cur_role) == BTM_SUCCESS) &&
       (cur_role == BTM_ROLE_SLAVE)) {
-    bta_sys_set_policy(BTA_ID_AV, HCI_ENABLE_MASTER_SLAVE_SWITCH, peer_addr);
+    bta_sys_set_policy(BTA_ID_AV, HCI_ENABLE_MASTER_SLAVE_SWITCH, *peer_addr);
   }
 
   /* if BTA_AvOpen() was called for other device, which caused the role switch
@@ -911,7 +912,7 @@ static void bta_av_sys_rs_cback(UNUSED_ATTR tBTA_SYS_CONN_STATUS status,
  ******************************************************************************/
 static void bta_av_sco_chg_cback(tBTA_SYS_CONN_STATUS status, uint8_t id,
                                  UNUSED_ATTR uint8_t app_id,
-                                 UNUSED_ATTR BD_ADDR peer_addr) {
+                                 UNUSED_ATTR const bt_bdaddr_t* peer_addr) {
   tBTA_AV_SCB* p_scb;
   int i;
   tBTA_AV_API_STOP stop;
@@ -980,7 +981,7 @@ bool bta_av_switch_if_needed(tBTA_AV_SCB* p_scb) {
       if (BTM_ROLE_MASTER != role) {
         if (bta_av_cb.features & BTA_AV_FEAT_MASTER)
           bta_sys_clear_policy(BTA_ID_AV, HCI_ENABLE_MASTER_SLAVE_SWITCH,
-                               to_BD_ADDR(p_scbi->peer_addr));
+                               p_scbi->peer_addr);
         if (BTM_CMD_STARTED !=
             BTM_SwitchRole(p_scbi->peer_addr, BTM_ROLE_MASTER, NULL)) {
           /* can not switch role on SCBI
@@ -1022,7 +1023,7 @@ bool bta_av_link_role_ok(tBTA_AV_SCB* p_scb, uint8_t bits) {
          (bta_av_cb.features & BTA_AV_FEAT_MASTER))) {
       if (bta_av_cb.features & BTA_AV_FEAT_MASTER)
         bta_sys_clear_policy(BTA_ID_AV, HCI_ENABLE_MASTER_SLAVE_SWITCH,
-                             to_BD_ADDR(p_scb->peer_addr));
+                             p_scb->peer_addr);
 
       if (BTM_CMD_STARTED !=
           BTM_SwitchRole(p_scb->peer_addr, BTM_ROLE_MASTER, NULL)) {
index 76d4f2f..dc2ffb2 100644 (file)
@@ -2961,7 +2961,7 @@ void bta_dm_acl_change(tBTA_DM_MSG* p_data) {
            * if so, make sure the AV link is master */
           bta_dm_check_av(0);
         }
-        bta_sys_notify_role_chg(to_BD_ADDR(p_data->acl_change.bd_addr),
+        bta_sys_notify_role_chg(p_data->acl_change.bd_addr,
                                 p_data->acl_change.new_role,
                                 p_data->acl_change.hci_status);
         conn.role_chg.bd_addr = p_bda;
@@ -2974,7 +2974,7 @@ void bta_dm_acl_change(tBTA_DM_MSG* p_data) {
 
   /* Collision report from Stack: Notify profiles */
   if (p_data->acl_change.event == BTM_BL_COLLISION_EVT) {
-    bta_sys_notify_collision(to_BD_ADDR(p_bda));
+    bta_sys_notify_collision(p_bda);
     return;
   }
 
index d13313a..27b72c0 100644 (file)
@@ -599,7 +599,7 @@ void bta_gattc_conn(tBTA_GATTC_CLCB* p_clcb, tBTA_GATTC_DATA* p_data) {
   if (p_clcb->p_rcb) {
     /* there is no RM for GATT */
     if (p_clcb->transport == BTA_TRANSPORT_BR_EDR)
-      bta_sys_conn_open(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda.address);
+      bta_sys_conn_open(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda);
 
     bta_gattc_send_open_cback(p_clcb->p_rcb, BTA_GATT_OK, p_clcb->bda,
                               p_clcb->bta_conn_id, p_clcb->transport,
@@ -652,7 +652,7 @@ void bta_gattc_close(tBTA_GATTC_CLCB* p_clcb, tBTA_GATTC_DATA* p_data) {
   cb_data.close.remote_bda = p_clcb->bda;
 
   if (p_clcb->transport == BTA_TRANSPORT_BR_EDR)
-    bta_sys_conn_close(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda.address);
+    bta_sys_conn_close(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda);
 
   bta_gattc_clcb_dealloc(p_clcb);
 
@@ -1042,8 +1042,8 @@ void bta_gattc_confirm(tBTA_GATTC_CLCB* p_clcb, tBTA_GATTC_DATA* p_data) {
   } else {
     /* if over BR_EDR, inform PM for mode change */
     if (p_clcb->transport == BTA_TRANSPORT_BR_EDR) {
-      bta_sys_busy(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda.address);
-      bta_sys_idle(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda.address);
+      bta_sys_busy(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda);
+      bta_sys_idle(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda);
     }
   }
 }
@@ -1627,8 +1627,8 @@ static void bta_gattc_cmpl_cback(uint16_t conn_id, tGATTC_OPTYPE op,
 
   /* if over BR_EDR, inform PM for mode change */
   if (p_clcb->transport == BTA_TRANSPORT_BR_EDR) {
-    bta_sys_busy(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda.address);
-    bta_sys_idle(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda.address);
+    bta_sys_busy(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda);
+    bta_sys_idle(BTA_ID_GATTC, BTA_ALL_APP_ID, p_clcb->bda);
   }
 
   bta_gattc_cmpl_sendmsg(conn_id, op, status, p_data);
index 5a1f466..47fedf4 100644 (file)
@@ -397,8 +397,8 @@ void bta_gatts_indicate_handle(tBTA_GATTS_CB* p_cb, tBTA_GATTS_DATA* p_msg) {
 
       /* if over BR_EDR, inform PM for mode change */
       if (transport == BTA_TRANSPORT_BR_EDR) {
-        bta_sys_busy(BTA_ID_GATTS, BTA_ALL_APP_ID, remote_bda.address);
-        bta_sys_idle(BTA_ID_GATTS, BTA_ALL_APP_ID, remote_bda.address);
+        bta_sys_busy(BTA_ID_GATTS, BTA_ALL_APP_ID, remote_bda);
+        bta_sys_idle(BTA_ID_GATTS, BTA_ALL_APP_ID, remote_bda);
       }
     } else {
       APPL_TRACE_ERROR("Unknown connection ID: %d fail sending notification",
@@ -510,7 +510,7 @@ void bta_gatts_close(UNUSED_ATTR tBTA_GATTS_CB* p_cb, tBTA_GATTS_DATA* p_msg) {
 
     if (p_rcb && p_rcb->p_cback) {
       if (transport == BTA_TRANSPORT_BR_EDR)
-        bta_sys_conn_close(BTA_ID_GATTS, BTA_ALL_APP_ID, remote_bda.address);
+        bta_sys_conn_close(BTA_ID_GATTS, BTA_ALL_APP_ID, remote_bda);
 
       (*p_rcb->p_cback)(BTA_GATTS_CLOSE_EVT, (tBTA_GATTS*)&status);
     }
@@ -548,10 +548,8 @@ static void bta_gatts_send_request_cback(uint16_t conn_id, uint32_t trans_id,
     if (p_rcb && p_rcb->p_cback) {
       /* if over BR_EDR, inform PM for mode change */
       if (transport == BTA_TRANSPORT_BR_EDR) {
-        bta_sys_busy(BTA_ID_GATTS, BTA_ALL_APP_ID,
-                     cb_data.req_data.remote_bda.address);
-        bta_sys_idle(BTA_ID_GATTS, BTA_ALL_APP_ID,
-                     cb_data.req_data.remote_bda.address);
+        bta_sys_busy(BTA_ID_GATTS, BTA_ALL_APP_ID, cb_data.req_data.remote_bda);
+        bta_sys_idle(BTA_ID_GATTS, BTA_ALL_APP_ID, cb_data.req_data.remote_bda);
       }
 
       cb_data.req_data.conn_id = conn_id;
@@ -601,9 +599,9 @@ static void bta_gatts_conn_cback(tGATT_IF gatt_if, const bt_bdaddr_t& bdaddr,
     /* there is no RM for GATT */
     if (transport == BTA_TRANSPORT_BR_EDR) {
       if (connected)
-        bta_sys_conn_open(BTA_ID_GATTS, BTA_ALL_APP_ID, to_BD_ADDR(bdaddr));
+        bta_sys_conn_open(BTA_ID_GATTS, BTA_ALL_APP_ID, bdaddr);
       else
-        bta_sys_conn_close(BTA_ID_GATTS, BTA_ALL_APP_ID, to_BD_ADDR(bdaddr));
+        bta_sys_conn_close(BTA_ID_GATTS, BTA_ALL_APP_ID, bdaddr);
     }
 
     cb_data.conn.conn_id = conn_id;
index 39f8065..4458464 100644 (file)
@@ -377,8 +377,8 @@ extern void bta_hd_send_report_act(tBTA_HD_DATA* p_data) {
                     p_report->data);
 
   /* trigger PM */
-  bta_sys_busy(BTA_ID_HD, 1, to_BD_ADDR(bta_hd_cb.bd_addr));
-  bta_sys_idle(BTA_ID_HD, 1, to_BD_ADDR(bta_hd_cb.bd_addr));
+  bta_sys_busy(BTA_ID_HD, 1, bta_hd_cb.bd_addr);
+  bta_sys_idle(BTA_ID_HD, 1, bta_hd_cb.bd_addr);
 }
 
 /*******************************************************************************
@@ -427,8 +427,8 @@ extern void bta_hd_vc_unplug_act(UNUSED_ATTR tBTA_HD_DATA* p_data) {
   }
 
   /* trigger PM */
-  bta_sys_busy(BTA_ID_HD, 1, to_BD_ADDR(bta_hd_cb.bd_addr));
-  bta_sys_idle(BTA_ID_HD, 1, to_BD_ADDR(bta_hd_cb.bd_addr));
+  bta_sys_busy(BTA_ID_HD, 1, bta_hd_cb.bd_addr);
+  bta_sys_idle(BTA_ID_HD, 1, bta_hd_cb.bd_addr);
 }
 
 /*******************************************************************************
@@ -447,7 +447,7 @@ extern void bta_hd_open_act(tBTA_HD_DATA* p_data) {
   APPL_TRACE_API("%s", __func__);
 
   HID_DevPlugDevice(p_cback->addr);
-  bta_sys_conn_open(BTA_ID_HD, 1, to_BD_ADDR(p_cback->addr));
+  bta_sys_conn_open(BTA_ID_HD, 1, p_cback->addr);
 
   cback_data.conn.bda = p_cback->addr;
   bta_hd_cb.bd_addr = p_cback->addr;
@@ -471,7 +471,7 @@ extern void bta_hd_close_act(tBTA_HD_DATA* p_data) {
 
   APPL_TRACE_API("%s", __func__);
 
-  bta_sys_conn_close(BTA_ID_HD, 1, to_BD_ADDR(p_cback->addr));
+  bta_sys_conn_close(BTA_ID_HD, 1, p_cback->addr);
 
   if (bta_hd_cb.vc_unplug) {
     bta_hd_cb.vc_unplug = FALSE;
@@ -624,7 +624,7 @@ extern void bta_hd_vc_unplug_done_act(tBTA_HD_DATA* p_data) {
 
   APPL_TRACE_API("%s", __func__);
 
-  bta_sys_conn_close(BTA_ID_HD, 1, to_BD_ADDR(p_cback->addr));
+  bta_sys_conn_close(BTA_ID_HD, 1, p_cback->addr);
 
   HID_DevUnplugDevice(p_cback->addr);
 
@@ -648,7 +648,7 @@ extern void bta_hd_suspend_act(tBTA_HD_DATA* p_data) {
 
   APPL_TRACE_API("%s", __func__);
 
-  bta_sys_idle(BTA_ID_HD, 1, to_BD_ADDR(p_cback->addr));
+  bta_sys_idle(BTA_ID_HD, 1, p_cback->addr);
 }
 
 /*******************************************************************************
@@ -665,8 +665,8 @@ extern void bta_hd_exit_suspend_act(tBTA_HD_DATA* p_data) {
 
   APPL_TRACE_API("%s", __func__);
 
-  bta_sys_busy(BTA_ID_HD, 1, to_BD_ADDR(p_cback->addr));
-  bta_sys_idle(BTA_ID_HD, 1, to_BD_ADDR(p_cback->addr));
+  bta_sys_busy(BTA_ID_HD, 1, p_cback->addr);
+  bta_sys_idle(BTA_ID_HD, 1, p_cback->addr);
 }
 
 /*******************************************************************************
index cc143ae..7c99124 100644 (file)
@@ -108,8 +108,7 @@ void bta_hf_client_start_open(tBTA_HF_CLIENT_DATA* p_data) {
     /* Let the incoming connection goes through.                        */
     /* Issue collision for now.                                         */
     /* We will decide what to do when we find incoming connection later.*/
-    bta_hf_client_collision_cback(0, BTA_ID_HS, 0,
-                                  to_BD_ADDR(client_cb->peer_addr));
+    bta_hf_client_collision_cback(0, BTA_ID_HS, 0, &client_cb->peer_addr);
     return;
   }
 
@@ -140,7 +139,7 @@ void bta_hf_client_rfc_open(tBTA_HF_CLIENT_DATA* p_data) {
     return;
   }
 
-  bta_sys_conn_open(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+  bta_sys_conn_open(BTA_ID_HS, 1, client_cb->peer_addr);
 
   /* start SLC procedure */
   bta_hf_client_slc_seq(client_cb, false);
@@ -296,7 +295,7 @@ void bta_hf_client_rfc_close(tBTA_HF_CLIENT_DATA* p_data) {
 
   bta_hf_client_at_reset(client_cb);
 
-  bta_sys_conn_close(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+  bta_sys_conn_close(BTA_ID_HS, 1, client_cb->peer_addr);
 
   /* call close cback */
   tBTA_HF_CLIENT evt;
@@ -308,7 +307,7 @@ void bta_hf_client_rfc_close(tBTA_HF_CLIENT_DATA* p_data) {
     /* Make sure SCO is shutdown */
     bta_hf_client_sco_shutdown(client_cb);
 
-    bta_sys_sco_unuse(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+    bta_sys_sco_unuse(BTA_ID_HS, 1, client_cb->peer_addr);
   }
   /* else close port and deallocate scb */
   else {
index e080cf2..d8bec2f 100644 (file)
@@ -218,7 +218,7 @@ extern bool bta_hf_client_allocate_handle(const bt_bdaddr_t& bd_addr,
 extern void bta_hf_client_app_callback(uint16_t event, tBTA_HF_CLIENT* data);
 extern void bta_hf_client_collision_cback(tBTA_SYS_CONN_STATUS status,
                                           uint8_t id, uint8_t app_id,
-                                          BD_ADDR peer_addr);
+                                          const bt_bdaddr_t* peer_addr);
 extern void bta_hf_client_resume_open(tBTA_HF_CLIENT_CB* client_cb);
 extern tBTA_STATUS bta_hf_client_api_enable(tBTA_HF_CLIENT_CBACK* p_cback,
                                             tBTA_SEC sec_mask,
index dcd714a..748959a 100644 (file)
@@ -366,9 +366,8 @@ static void bta_hf_client_collision_timer_cback(void* data) {
  ******************************************************************************/
 void bta_hf_client_collision_cback(UNUSED_ATTR tBTA_SYS_CONN_STATUS status,
                                    uint8_t id, UNUSED_ATTR uint8_t app_id,
-                                   BD_ADDR peer_addr) {
-  tBTA_HF_CLIENT_CB* client_cb =
-      bta_hf_client_find_cb_by_bda(from_BD_ADDR(peer_addr));
+                                   const bt_bdaddr_t* peer_addr) {
+  tBTA_HF_CLIENT_CB* client_cb = bta_hf_client_find_cb_by_bda(*peer_addr);
   if (client_cb != NULL && client_cb->state == BTA_HF_CLIENT_OPENING_ST) {
     if (id == BTA_ID_SYS) /* ACL collision */
     {
index 4b2fec0..a6d0104 100644 (file)
@@ -118,7 +118,7 @@ static void bta_hf_client_sco_conn_rsp(tBTA_HF_CLIENT_CB* client_cb,
     }
 
     /* tell sys to stop av if any */
-    bta_sys_sco_use(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+    bta_sys_sco_use(BTA_ID_HS, 1, client_cb->peer_addr);
   } else {
     hci_status = HCI_ERR_HOST_REJECT_DEVICE;
   }
@@ -237,7 +237,7 @@ static void bta_hf_client_sco_create(tBTA_HF_CLIENT_CB* client_cb,
   if (is_orig) {
     BTM_SetEScoMode(&params);
     /* tell sys to stop av if any */
-    bta_sys_sco_use(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+    bta_sys_sco_use(BTA_ID_HS, 1, client_cb->peer_addr);
   }
 
   status = BTM_CreateSco(&client_cb->peer_addr, is_orig, params.packet_types,
@@ -552,7 +552,7 @@ void bta_hf_client_sco_conn_open(tBTA_HF_CLIENT_DATA* p_data) {
 
   bta_hf_client_sco_event(client_cb, BTA_HF_CLIENT_SCO_CONN_OPEN_E);
 
-  bta_sys_sco_open(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+  bta_sys_sco_open(BTA_ID_HS, 1, client_cb->peer_addr);
 
   if (client_cb->negotiated_codec == BTM_SCO_CODEC_MSBC) {
     bta_hf_client_cback_sco(client_cb, BTA_HF_CLIENT_AUDIO_MSBC_OPEN_EVT);
@@ -587,9 +587,9 @@ void bta_hf_client_sco_conn_close(tBTA_HF_CLIENT_DATA* p_data) {
 
   bta_hf_client_sco_event(client_cb, BTA_HF_CLIENT_SCO_CONN_CLOSE_E);
 
-  bta_sys_sco_close(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+  bta_sys_sco_close(BTA_ID_HS, 1, client_cb->peer_addr);
 
-  bta_sys_sco_unuse(BTA_ID_HS, 1, to_BD_ADDR(client_cb->peer_addr));
+  bta_sys_sco_unuse(BTA_ID_HS, 1, client_cb->peer_addr);
 
   /* call app callback */
   bta_hf_client_cback_sco(client_cb, BTA_HF_CLIENT_AUDIO_CLOSE_EVT);
index 4dec631..d8eaa82 100644 (file)
@@ -526,7 +526,7 @@ void bta_hh_open_cmpl_act(tBTA_HH_DEV_CB* p_cb, tBTA_HH_DATA* p_data) {
 #endif
   {
     /* inform role manager */
-    bta_sys_conn_open(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
+    bta_sys_conn_open(BTA_ID_HH, p_cb->app_id, p_cb->addr);
   }
   /* set protocol mode when not default report mode */
   if (p_cb->mode != BTA_HH_PROTO_RPT_MODE
@@ -679,7 +679,7 @@ void bta_hh_handsk_act(tBTA_HH_DEV_CB* p_cb, tBTA_HH_DATA* p_data) {
   }
 
   /* transaction achknoledgement received, inform PM for mode change */
-  bta_sys_idle(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
+  bta_sys_idle(BTA_ID_HH, p_cb->app_id, p_cb->addr);
   return;
 }
 /*******************************************************************************
@@ -739,8 +739,8 @@ void bta_hh_ctrl_dat_act(tBTA_HH_DEV_CB* p_cb, tBTA_HH_DATA* p_data) {
   }
 
   /* inform PM for mode change */
-  bta_sys_busy(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
-  bta_sys_idle(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
+  bta_sys_busy(BTA_ID_HH, p_cb->app_id, p_cb->addr);
+  bta_sys_idle(BTA_ID_HH, p_cb->app_id, p_cb->addr);
 
   (*bta_hh_cb.p_cback)(p_cb->w4_evt, (tBTA_HH*)&hs_data);
 
@@ -839,7 +839,7 @@ void bta_hh_close_act(tBTA_HH_DEV_CB* p_cb, tBTA_HH_DATA* p_data) {
     /* finaliza device driver */
     bta_hh_co_close(p_cb->hid_handle, p_cb->app_id);
     /* inform role manager */
-    bta_sys_conn_close(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
+    bta_sys_conn_close(BTA_ID_HH, p_cb->app_id, p_cb->addr);
     /* update total conn number */
     bta_hh_cb.cnt_num--;
 
@@ -1063,12 +1063,12 @@ void bta_hh_write_dev_act(tBTA_HH_DEV_CB* p_cb, tBTA_HH_DATA* p_data) {
       /* if not control type transaction, notify PM for energy control */
       if (p_data->api_sndcmd.t_type != HID_TRANS_CONTROL) {
         /* inform PM for mode change */
-        bta_sys_busy(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
-        bta_sys_idle(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
+        bta_sys_busy(BTA_ID_HH, p_cb->app_id, p_cb->addr);
+        bta_sys_idle(BTA_ID_HH, p_cb->app_id, p_cb->addr);
       } else if (p_data->api_sndcmd.param == BTA_HH_CTRL_SUSPEND) {
-        bta_sys_sco_close(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
+        bta_sys_sco_close(BTA_ID_HH, p_cb->app_id, p_cb->addr);
       } else if (p_data->api_sndcmd.param == BTA_HH_CTRL_EXIT_SUSPEND) {
-        bta_sys_busy(BTA_ID_HH, p_cb->app_id, to_BD_ADDR(p_cb->addr));
+        bta_sys_busy(BTA_ID_HH, p_cb->app_id, p_cb->addr);
       }
     }
   }
index fde02f2..224313c 100644 (file)
@@ -1987,7 +1987,7 @@ void bta_hl_cch_close_cmpl(uint8_t app_idx, uint8_t mcl_idx,
 #if (BTA_HL_DEBUG == TRUE)
   APPL_TRACE_DEBUG("bta_hl_cch_close_cmpl");
 #endif
-  bta_sys_conn_close(BTA_ID_HL, p_acb->app_id, to_BD_ADDR(p_mcb->bd_addr));
+  bta_sys_conn_close(BTA_ID_HL, p_acb->app_id, p_mcb->bd_addr);
 
   if (p_mcb->cch_oper == BTA_HL_CCH_OP_LOCAL_CLOSE &&
       p_mcb->force_close_local_cch_opening) {
@@ -2135,7 +2135,7 @@ void bta_hl_cch_mca_connect(uint8_t app_idx, uint8_t mcl_idx,
   p_mcb->bd_addr = from_BD_ADDR(p_data->mca_evt.mca_data.connect_ind.bd_addr);
   p_mcb->cch_mtu = p_data->mca_evt.mca_data.connect_ind.mtu;
 
-  bta_sys_conn_open(BTA_ID_HL, p_acb->app_id, to_BD_ADDR(p_mcb->bd_addr));
+  bta_sys_conn_open(BTA_ID_HL, p_acb->app_id, p_mcb->bd_addr);
   switch (p_mcb->cch_oper) {
     case BTA_HL_CCH_OP_LOCAL_OPEN:
       bta_hl_build_cch_open_cfm(&evt_data, p_mcb->app_id, p_acb->app_handle,
index 9c2c4e7..1e9a03b 100644 (file)
@@ -397,8 +397,7 @@ static void bta_jv_clear_pm_cb(tBTA_JV_PM_CB* p_pm_cb, bool close_conn) {
   /* needs to be called if registered with bta pm, otherwise we may run out of
    * dm pm slots! */
   if (close_conn)
-    bta_sys_conn_close(BTA_ID_JV, p_pm_cb->app_id,
-                       to_BD_ADDR(p_pm_cb->peer_bd_addr));
+    bta_sys_conn_close(BTA_ID_JV, p_pm_cb->app_id, p_pm_cb->peer_bd_addr);
   p_pm_cb->state = BTA_JV_PM_FREE_ST;
   p_pm_cb->app_id = BTA_JV_PM_ALL;
   p_pm_cb->handle = BTA_JV_PM_HANDLE_CLEAR;
@@ -2112,41 +2111,37 @@ static void bta_jv_pm_state_change(tBTA_JV_PM_CB* p_cb,
 
   switch (state) {
     case BTA_JV_CONN_OPEN:
-      bta_sys_conn_open(BTA_ID_JV, p_cb->app_id,
-                        to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_conn_open(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     case BTA_JV_CONN_CLOSE:
-      bta_sys_conn_close(BTA_ID_JV, p_cb->app_id,
-                         to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_conn_close(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     case BTA_JV_APP_OPEN:
-      bta_sys_app_open(BTA_ID_JV, p_cb->app_id, to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_app_open(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     case BTA_JV_APP_CLOSE:
-      bta_sys_app_close(BTA_ID_JV, p_cb->app_id,
-                        to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_app_close(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     case BTA_JV_SCO_OPEN:
-      bta_sys_sco_open(BTA_ID_JV, p_cb->app_id, to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_sco_open(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     case BTA_JV_SCO_CLOSE:
-      bta_sys_sco_close(BTA_ID_JV, p_cb->app_id,
-                        to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_sco_close(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     case BTA_JV_CONN_IDLE:
       p_cb->state = BTA_JV_PM_IDLE_ST;
-      bta_sys_idle(BTA_ID_JV, p_cb->app_id, to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_idle(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     case BTA_JV_CONN_BUSY:
       p_cb->state = BTA_JV_PM_BUSY_ST;
-      bta_sys_busy(BTA_ID_JV, p_cb->app_id, to_BD_ADDR(p_cb->peer_bd_addr));
+      bta_sys_busy(BTA_ID_JV, p_cb->app_id, p_cb->peer_bd_addr);
       break;
 
     default:
index 22c4fcc..5a87d74 100644 (file)
@@ -55,7 +55,7 @@
  ******************************************************************************/
 static void bta_pan_pm_conn_busy(tBTA_PAN_SCB* p_scb) {
   if ((p_scb != NULL) && (p_scb->state != BTA_PAN_IDLE_ST))
-    bta_sys_busy(BTA_ID_PAN, p_scb->app_id, to_BD_ADDR(p_scb->bd_addr));
+    bta_sys_busy(BTA_ID_PAN, p_scb->app_id, p_scb->bd_addr);
 }
 
 /*******************************************************************************
@@ -71,7 +71,7 @@ static void bta_pan_pm_conn_busy(tBTA_PAN_SCB* p_scb) {
  ******************************************************************************/
 static void bta_pan_pm_conn_idle(tBTA_PAN_SCB* p_scb) {
   if ((p_scb != NULL) && (p_scb->state != BTA_PAN_IDLE_ST))
-    bta_sys_idle(BTA_ID_PAN, p_scb->app_id, to_BD_ADDR(p_scb->bd_addr));
+    bta_sys_idle(BTA_ID_PAN, p_scb->app_id, p_scb->bd_addr);
 }
 
 /*******************************************************************************
@@ -513,7 +513,7 @@ void bta_pan_conn_open(tBTA_PAN_SCB* p_scb, tBTA_PAN_DATA* p_data) {
     bta_pan.open.status = BTA_PAN_SUCCESS;
     p_scb->pan_flow_enable = true;
     p_scb->app_flow_enable = true;
-    bta_sys_conn_open(BTA_ID_PAN, p_scb->app_id, to_BD_ADDR(p_scb->bd_addr));
+    bta_sys_conn_open(BTA_ID_PAN, p_scb->app_id, p_scb->bd_addr);
   } else {
     bta_pan_scb_dealloc(p_scb);
     bta_pan.open.status = BTA_PAN_FAIL;
@@ -531,7 +531,7 @@ void bta_pan_conn_open(tBTA_PAN_SCB* p_scb, tBTA_PAN_DATA* p_data) {
     p_scb->app_id = BTA_APP_ID_PAN_MULTI;
   }
 
-  bta_sys_conn_open(BTA_ID_PAN, p_scb->app_id, to_BD_ADDR(p_scb->bd_addr));
+  bta_sys_conn_open(BTA_ID_PAN, p_scb->app_id, p_scb->bd_addr);
   bta_pan_cb.p_cback(BTA_PAN_OPEN_EVT, &bta_pan);
 }
 
@@ -552,7 +552,7 @@ void bta_pan_conn_close(tBTA_PAN_SCB* p_scb, tBTA_PAN_DATA* p_data) {
 
   bta_pan.close.handle = p_data->hdr.layer_specific;
 
-  bta_sys_conn_close(BTA_ID_PAN, p_scb->app_id, to_BD_ADDR(p_scb->bd_addr));
+  bta_sys_conn_close(BTA_ID_PAN, p_scb->app_id, p_scb->bd_addr);
 
   /* free all queued up data buffers */
   while ((p_buf = (BT_HDR*)fixed_queue_try_dequeue(p_scb->data_queue)) != NULL)
index 093952e..80d3793 100644 (file)
@@ -136,7 +136,7 @@ typedef uint8_t tBTA_SYS_PREF_ROLES;
 
 /* conn callback for role / low power manager*/
 typedef void(tBTA_SYS_CONN_CBACK)(tBTA_SYS_CONN_STATUS status, uint8_t id,
-                                  uint8_t app_id, BD_ADDR peer_addr);
+                                  uint8_t app_id, const bt_bdaddr_t* peer_addr);
 
 /* conn callback for role / low power manager*/
 typedef void(tBTA_SYS_SSR_CFG_CBACK)(uint8_t id, uint8_t app_id,
@@ -235,16 +235,26 @@ extern void bta_sys_pm_register(tBTA_SYS_CONN_CBACK* p_cback);
 extern void bta_sys_policy_register(tBTA_SYS_CONN_CBACK* p_cback);
 extern void bta_sys_sco_register(tBTA_SYS_CONN_CBACK* p_cback);
 
-extern void bta_sys_conn_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_conn_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_app_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_app_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_sco_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_sco_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_sco_use(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_sco_unuse(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_idle(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
-extern void bta_sys_busy(uint8_t id, uint8_t app_id, BD_ADDR peer_addr);
+extern void bta_sys_conn_open(uint8_t id, uint8_t app_id,
+                              const bt_bdaddr_t& peer_addr);
+extern void bta_sys_conn_close(uint8_t id, uint8_t app_id,
+                               const bt_bdaddr_t& peer_addr);
+extern void bta_sys_app_open(uint8_t id, uint8_t app_id,
+                             const bt_bdaddr_t& peer_addr);
+extern void bta_sys_app_close(uint8_t id, uint8_t app_id,
+                              const bt_bdaddr_t& peer_addr);
+extern void bta_sys_sco_open(uint8_t id, uint8_t app_id,
+                             const bt_bdaddr_t& peer_addr);
+extern void bta_sys_sco_close(uint8_t id, uint8_t app_id,
+                              const bt_bdaddr_t& peer_addr);
+extern void bta_sys_sco_use(uint8_t id, uint8_t app_id,
+                            const bt_bdaddr_t& peer_addr);
+extern void bta_sys_sco_unuse(uint8_t id, uint8_t app_id,
+                              const bt_bdaddr_t& peer_addr);
+extern void bta_sys_idle(uint8_t id, uint8_t app_id,
+                         const bt_bdaddr_t& peer_addr);
+extern void bta_sys_busy(uint8_t id, uint8_t app_id,
+                         const bt_bdaddr_t& peer_addr);
 
 #if (BTM_SSR_INCLUDED == TRUE)
 extern void bta_sys_ssr_cfg_register(tBTA_SYS_SSR_CFG_CBACK* p_cback);
@@ -253,11 +263,11 @@ extern void bta_sys_chg_ssr_config(uint8_t id, uint8_t app_id,
 #endif
 
 extern void bta_sys_role_chg_register(tBTA_SYS_CONN_CBACK* p_cback);
-extern void bta_sys_notify_role_chg(BD_ADDR_PTR p_bda, uint8_t new_role,
+extern void bta_sys_notify_role_chg(const bt_bdaddr_t& p_bda, uint8_t new_role,
                                     uint8_t hci_status);
 extern void bta_sys_collision_register(uint8_t bta_id,
                                        tBTA_SYS_CONN_CBACK* p_cback);
-extern void bta_sys_notify_collision(BD_ADDR_PTR p_bda);
+extern void bta_sys_notify_collision(const bt_bdaddr_t& p_bda);
 
 #if (BTA_EIR_CANNED_UUID_LIST != TRUE)
 extern void bta_sys_eir_register(tBTA_SYS_EIR_CBACK* p_cback);
@@ -269,8 +279,10 @@ extern void bta_sys_remove_uuid(uint16_t uuid16);
 #define bta_sys_remove_uuid(ut)
 #endif
 
-extern void bta_sys_set_policy(uint8_t id, uint8_t policy, BD_ADDR peer_addr);
-extern void bta_sys_clear_policy(uint8_t id, uint8_t policy, BD_ADDR peer_addr);
+extern void bta_sys_set_policy(uint8_t id, uint8_t policy,
+                               const bt_bdaddr_t& peer_addr);
+extern void bta_sys_clear_policy(uint8_t id, uint8_t policy,
+                                 const bt_bdaddr_t& peer_addr);
 extern void bta_sys_set_default_policy(uint8_t id, uint8_t policy);
 extern void bta_sys_clear_default_policy(uint8_t id, uint8_t policy);
 
index 27967cc..60c6332 100644 (file)
@@ -97,10 +97,10 @@ void bta_sys_ssr_cfg_register(tBTA_SYS_SSR_CFG_CBACK* p_cback) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_notify_role_chg(BD_ADDR_PTR p_bda, uint8_t new_role,
+void bta_sys_notify_role_chg(const bt_bdaddr_t& p_bda, uint8_t new_role,
                              uint8_t hci_status) {
   if (bta_sys_cb.p_role_cb) {
-    bta_sys_cb.p_role_cb(BTA_SYS_ROLE_CHANGE, new_role, hci_status, p_bda);
+    bta_sys_cb.p_role_cb(BTA_SYS_ROLE_CHANGE, new_role, hci_status, &p_bda);
   }
 }
 
@@ -137,13 +137,13 @@ void bta_sys_collision_register(uint8_t bta_id, tBTA_SYS_CONN_CBACK* p_cback) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_notify_collision(BD_ADDR_PTR p_bda) {
+void bta_sys_notify_collision(const bt_bdaddr_t& p_bda) {
   uint8_t index;
 
   for (index = 0; index < MAX_COLLISION_REG; index++) {
     if ((bta_sys_cb.colli_reg.id[index] != 0) &&
         (bta_sys_cb.colli_reg.p_coll_cback[index] != NULL)) {
-      bta_sys_cb.colli_reg.p_coll_cback[index](0, BTA_ID_SYS, 0, p_bda);
+      bta_sys_cb.colli_reg.p_coll_cback[index](0, BTA_ID_SYS, 0, &p_bda);
     }
   }
 }
@@ -187,13 +187,14 @@ void bta_sys_pm_register(tBTA_SYS_CONN_CBACK* p_cback) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_conn_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_conn_open(uint8_t id, uint8_t app_id,
+                       const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.prm_cb) {
-    bta_sys_cb.prm_cb(BTA_SYS_CONN_OPEN, id, app_id, peer_addr);
+    bta_sys_cb.prm_cb(BTA_SYS_CONN_OPEN, id, app_id, &peer_addr);
   }
 
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_CONN_OPEN, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_CONN_OPEN, id, app_id, &peer_addr);
   }
 }
 
@@ -208,13 +209,14 @@ void bta_sys_conn_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_conn_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_conn_close(uint8_t id, uint8_t app_id,
+                        const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.prm_cb) {
-    bta_sys_cb.prm_cb(BTA_SYS_CONN_CLOSE, id, app_id, peer_addr);
+    bta_sys_cb.prm_cb(BTA_SYS_CONN_CLOSE, id, app_id, &peer_addr);
   }
 
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_CONN_CLOSE, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_CONN_CLOSE, id, app_id, &peer_addr);
   }
 }
 
@@ -229,9 +231,10 @@ void bta_sys_conn_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_app_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_app_open(uint8_t id, uint8_t app_id,
+                      const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_APP_OPEN, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_APP_OPEN, id, app_id, &peer_addr);
   }
 }
 
@@ -245,9 +248,10 @@ void bta_sys_app_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_app_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_app_close(uint8_t id, uint8_t app_id,
+                       const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_APP_CLOSE, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_APP_CLOSE, id, app_id, &peer_addr);
   }
 }
 
@@ -261,15 +265,16 @@ void bta_sys_app_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_sco_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_sco_open(uint8_t id, uint8_t app_id,
+                      const bt_bdaddr_t& peer_addr) {
   /* AG triggers p_sco_cb by bta_sys_sco_use. */
   if ((id != BTA_ID_AG) && (bta_sys_cb.p_sco_cb)) {
     /* without querying BTM_GetNumScoLinks() */
-    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_OPEN, 1, app_id, peer_addr);
+    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_OPEN, 1, app_id, &peer_addr);
   }
 
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_SCO_OPEN, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_SCO_OPEN, id, app_id, &peer_addr);
   }
 }
 
@@ -283,16 +288,17 @@ void bta_sys_sco_open(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_sco_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_sco_close(uint8_t id, uint8_t app_id,
+                       const bt_bdaddr_t& peer_addr) {
   uint8_t num_sco_links;
 
   if ((id != BTA_ID_AG) && (bta_sys_cb.p_sco_cb)) {
     num_sco_links = BTM_GetNumScoLinks();
-    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_CLOSE, num_sco_links, app_id, peer_addr);
+    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_CLOSE, num_sco_links, app_id, &peer_addr);
   }
 
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_SCO_CLOSE, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_SCO_CLOSE, id, app_id, &peer_addr);
   }
 }
 
@@ -307,11 +313,11 @@ void bta_sys_sco_close(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
  *
  ******************************************************************************/
 void bta_sys_sco_use(UNUSED_ATTR uint8_t id, uint8_t app_id,
-                     BD_ADDR peer_addr) {
+                     const bt_bdaddr_t& peer_addr) {
   /* AV streaming need to be suspended before SCO is connected. */
   if (bta_sys_cb.p_sco_cb) {
     /* without querying BTM_GetNumScoLinks() */
-    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_OPEN, 1, app_id, peer_addr);
+    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_OPEN, 1, app_id, &peer_addr);
   }
 }
 
@@ -325,13 +331,11 @@ void bta_sys_sco_use(UNUSED_ATTR uint8_t id, uint8_t app_id,
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_sco_unuse(UNUSED_ATTR uint8_t id, uint8_t app_id,
-                       BD_ADDR peer_addr) {
-  uint8_t num_sco_links;
-
+void bta_sys_sco_unuse(uint8_t id, uint8_t app_id,
+                       const bt_bdaddr_t& peer_addr) {
   if ((bta_sys_cb.p_sco_cb)) {
-    num_sco_links = BTM_GetNumScoLinks();
-    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_CLOSE, num_sco_links, app_id, peer_addr);
+    uint8_t num_sco_links = BTM_GetNumScoLinks();
+    bta_sys_cb.p_sco_cb(BTA_SYS_SCO_CLOSE, num_sco_links, app_id, &peer_addr);
   }
 }
 /*******************************************************************************
@@ -362,9 +366,10 @@ void bta_sys_chg_ssr_config(uint8_t id, uint8_t app_id, uint16_t max_latency,
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_set_policy(uint8_t id, uint8_t policy, BD_ADDR peer_addr) {
+void bta_sys_set_policy(uint8_t id, uint8_t policy,
+                        const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.p_policy_cb) {
-    bta_sys_cb.p_policy_cb(BTA_SYS_PLCY_SET, id, policy, peer_addr);
+    bta_sys_cb.p_policy_cb(BTA_SYS_PLCY_SET, id, policy, &peer_addr);
   }
 }
 
@@ -378,9 +383,10 @@ void bta_sys_set_policy(uint8_t id, uint8_t policy, BD_ADDR peer_addr) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_clear_policy(uint8_t id, uint8_t policy, BD_ADDR peer_addr) {
+void bta_sys_clear_policy(uint8_t id, uint8_t policy,
+                          const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.p_policy_cb) {
-    bta_sys_cb.p_policy_cb(BTA_SYS_PLCY_CLR, id, policy, peer_addr);
+    bta_sys_cb.p_policy_cb(BTA_SYS_PLCY_CLR, id, policy, &peer_addr);
   }
 }
 
@@ -426,13 +432,13 @@ void bta_sys_clear_default_policy(uint8_t id, uint8_t policy) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_idle(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_idle(uint8_t id, uint8_t app_id, const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.prm_cb) {
-    bta_sys_cb.prm_cb(BTA_SYS_CONN_IDLE, id, app_id, peer_addr);
+    bta_sys_cb.prm_cb(BTA_SYS_CONN_IDLE, id, app_id, &peer_addr);
   }
 
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_CONN_IDLE, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_CONN_IDLE, id, app_id, &peer_addr);
   }
 }
 
@@ -446,13 +452,13 @@ void bta_sys_idle(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
  * Returns          void
  *
  ******************************************************************************/
-void bta_sys_busy(uint8_t id, uint8_t app_id, BD_ADDR peer_addr) {
+void bta_sys_busy(uint8_t id, uint8_t app_id, const bt_bdaddr_t& peer_addr) {
   if (bta_sys_cb.prm_cb) {
-    bta_sys_cb.prm_cb(BTA_SYS_CONN_BUSY, id, app_id, peer_addr);
+    bta_sys_cb.prm_cb(BTA_SYS_CONN_BUSY, id, app_id, &peer_addr);
   }
 
   if (bta_sys_cb.ppm_cb) {
-    bta_sys_cb.ppm_cb(BTA_SYS_CONN_BUSY, id, app_id, peer_addr);
+    bta_sys_cb.ppm_cb(BTA_SYS_CONN_BUSY, id, app_id, &peer_addr);
   }
 }