Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c3835e37 authored by Chris Manton's avatar Chris Manton
Browse files

tBTM_INQUIRY_VAR_ST:GAP [1-7] Use longform btm_ble_start_inquiry 

Bug: 315853986
Test: m .
Flag: EXEMPT, mechanical refactor

I26e1e2eeff53022e357a79f5d423c0c26522c1f3
If69e44fd2a6c1bb8f17644db756180b79f4ab99c
I6532eaaed020a0a34757dda40f3bec0e8d4a2800
I6772d44bf2a5b6482b459cb618f8aec476a90b62
Iec421b3448d2041295b779344ab2b6bc9ffefaad
Ibec36f5fd42cf1c3936f5ac638b884dc9517d103
I7e63831276eb74098f396a554a71c09e3d3a49da

Change-Id: Ica705a78eaa75202810da58bafeb01bff254241b
parent 86f1df99

system/stack/btm/btm_ble_gap.cc

+29 −30
Original line number Diff line number Diff line
@@ -2038,7 +2038,6 @@ static void btm_ble_scan_filt_param_cfg_evt(uint8_t avbl_space, 
 ******************************************************************************/

tBTM_STATUS btm_ble_start_inquiry(uint8_t duration) {

  tBTM_BLE_CB* p_ble_cb = &btm_cb.ble_ctr_cb;

  tBTM_INQUIRY_VAR_ST* p_inq = &btm_cb.btm_inq_vars;



  LOG_VERBOSE("btm_ble_start_inquiry: inq_active = 0x%02x",

              btm_cb.btm_inq_vars.inq_active);
@@ -2088,10 +2087,11 @@ tBTM_STATUS btm_ble_start_inquiry(uint8_t duration) { 
    btm_send_hci_scan_enable(BTM_BLE_SCAN_ENABLE, BTM_BLE_DUPLICATE_DISABLE);

  }



  p_inq->inq_active |= BTM_BLE_GENERAL_INQUIRY;

  btm_cb.btm_inq_vars.inq_active |= BTM_BLE_GENERAL_INQUIRY;

  p_ble_cb->set_ble_inquiry_active();



  LOG_VERBOSE("btm_ble_start_inquiry inq_active = 0x%02x", p_inq->inq_active);

  LOG_VERBOSE("btm_ble_start_inquiry inq_active = 0x%02x",

              btm_cb.btm_inq_vars.inq_active);



  if (duration != 0) {

    /* start inquiry timer */
@@ -2151,8 +2151,6 @@ void btm_ble_read_remote_name_cmpl(bool status, const RawAddress& bda, 
 ******************************************************************************/

tBTM_STATUS btm_ble_read_remote_name(const RawAddress& remote_bda,

                                     tBTM_NAME_CMPL_CB* p_cb) {

  tBTM_INQUIRY_VAR_ST* p_inq = &btm_cb.btm_inq_vars;



  if (!controller_get_interface()->supports_ble()) return BTM_ERR_PROCESSING;



  tINQ_DB_ENT* p_i = btm_inq_db_find(remote_bda);
@@ -2162,16 +2160,17 @@ tBTM_STATUS btm_ble_read_remote_name(const RawAddress& remote_bda, 
  }



  /* read remote device name using GATT procedure */

  if (p_inq->remname_active) return BTM_BUSY;

  if (btm_cb.btm_inq_vars.remname_active) return BTM_BUSY;



  if (!GAP_BleReadPeerDevName(remote_bda, btm_ble_read_remote_name_cmpl))

    return BTM_BUSY;



  p_inq->p_remname_cmpl_cb = p_cb;

  p_inq->remname_active = true;

  p_inq->remname_bda = remote_bda;

  btm_cb.btm_inq_vars.p_remname_cmpl_cb = p_cb;

  btm_cb.btm_inq_vars.remname_active = true;

  btm_cb.btm_inq_vars.remname_bda = remote_bda;



  alarm_set_on_mloop(p_inq->remote_name_timer, BTM_EXT_BLE_RMT_NAME_TIMEOUT_MS,

  alarm_set_on_mloop(btm_cb.btm_inq_vars.remote_name_timer,

                     BTM_EXT_BLE_RMT_NAME_TIMEOUT_MS,

                     btm_inq_remote_name_timer_timeout, NULL);



  return BTM_CMD_STARTED;
@@ -2189,14 +2188,13 @@ tBTM_STATUS btm_ble_read_remote_name(const RawAddress& remote_bda, 
 *

 ******************************************************************************/

bool btm_ble_cancel_remote_name(const RawAddress& remote_bda) {

  tBTM_INQUIRY_VAR_ST* p_inq = &btm_cb.btm_inq_vars;

  bool status;



  status = GAP_BleCancelReadPeerDevName(remote_bda);



  p_inq->remname_active = false;

  p_inq->remname_bda = RawAddress::kEmpty;

  alarm_cancel(p_inq->remote_name_timer);

  btm_cb.btm_inq_vars.remname_active = false;

  btm_cb.btm_inq_vars.remname_bda = RawAddress::kEmpty;

  alarm_cancel(btm_cb.btm_inq_vars.remote_name_timer);



  return status;

}
@@ -2454,7 +2452,6 @@ void btm_ble_update_inq_result(tINQ_DB_ENT* p_i, uint8_t addr_type, 
                               std::vector<uint8_t> const& data) {

  tBTM_INQ_RESULTS* p_cur = &p_i->inq_info.results;

  uint8_t len;

  tBTM_INQUIRY_VAR_ST* p_inq = &btm_cb.btm_inq_vars;



  /* Save the info */

  p_cur->inq_result_type |= BTM_INQ_RESULT_BLE;
@@ -2473,14 +2470,15 @@ void btm_ble_update_inq_result(tINQ_DB_ENT* p_i, uint8_t addr_type, 
  } else

    p_i->scan_rsp = true;



  if (p_i->inq_count != p_inq->inq_counter)

  if (p_i->inq_count != btm_cb.btm_inq_vars.inq_counter)

    p_cur->device_type = BT_DEVICE_TYPE_BLE;

  else

    p_cur->device_type |= BT_DEVICE_TYPE_BLE;



  if (evt_type != BTM_BLE_SCAN_RSP_EVT) p_cur->ble_evt_type = evt_type;



  p_i->inq_count = p_inq->inq_counter; /* Mark entry for current inquiry */

  p_i->inq_count =

      btm_cb.btm_inq_vars.inq_counter; /* Mark entry for current inquiry */



  bool has_advertising_flags = false;

  if (!data.empty()) {
@@ -2751,7 +2749,6 @@ void btm_ble_process_adv_pkt_cont(uint16_t evt_type, tBLE_ADDR_TYPE addr_type, 
                                  int8_t rssi, uint16_t periodic_adv_int,

                                  uint8_t data_len, const uint8_t* data,

                                  const RawAddress& original_bda) {

  tBTM_INQUIRY_VAR_ST* p_inq = &btm_cb.btm_inq_vars;

  bool update = true;



  std::vector<uint8_t> tmp;
@@ -2832,15 +2829,16 @@ void btm_ble_process_adv_pkt_cont(uint16_t evt_type, tBLE_ADDR_TYPE addr_type, 
  if (p_i == NULL) {

    p_i = btm_inq_db_new(bda, true);

    if (p_i != NULL) {

      p_inq->inq_cmpl_info.num_resp++;

      btm_cb.btm_inq_vars.inq_cmpl_info.num_resp++;

      p_i->time_of_resp = bluetooth::common::time_get_os_boottime_ms();

    } else

      return;

  } else if (p_i->inq_count !=

             p_inq->inq_counter) /* first time seen in this inquiry */

             btm_cb.btm_inq_vars

                 .inq_counter) /* first time seen in this inquiry */

  {

    p_i->time_of_resp = bluetooth::common::time_get_os_boottime_ms();

    p_inq->inq_cmpl_info.num_resp++;

    btm_cb.btm_inq_vars.inq_cmpl_info.num_resp++;

  }



  /* update the LE device information in inquiry database */
@@ -2877,7 +2875,7 @@ void btm_ble_process_adv_pkt_cont(uint16_t evt_type, tBLE_ADDR_TYPE addr_type, 


  if (!update) result &= ~BTM_BLE_INQ_RESULT;



  tBTM_INQ_RESULTS_CB* p_inq_results_cb = p_inq->p_inq_results_cb;

  tBTM_INQ_RESULTS_CB* p_inq_results_cb = btm_cb.btm_inq_vars.p_inq_results_cb;

  if (p_inq_results_cb && (result & BTM_BLE_INQ_RESULT)) {

    (p_inq_results_cb)((tBTM_INQ_RESULTS*)&p_i->inq_info.results,

                       const_cast<uint8_t*>(adv_data.data()), adv_data.size());
@@ -2904,7 +2902,6 @@ void btm_ble_process_adv_pkt_cont_for_inquiry( 
    uint8_t primary_phy, uint8_t secondary_phy, uint8_t advertising_sid,

    int8_t tx_power, int8_t rssi, uint16_t periodic_adv_int,

    std::vector<uint8_t> advertising_data) {

  tBTM_INQUIRY_VAR_ST* p_inq = &btm_cb.btm_inq_vars;

  bool update = true;



  bool include_rsi = false;
@@ -2938,7 +2935,7 @@ void btm_ble_process_adv_pkt_cont_for_inquiry( 
  if (p_i == NULL) {

    p_i = btm_inq_db_new(bda, true);

    if (p_i != NULL) {

      p_inq->inq_cmpl_info.num_resp++;

      btm_cb.btm_inq_vars.inq_cmpl_info.num_resp++;

      p_i->time_of_resp = bluetooth::common::time_get_os_boottime_ms();

      btm_cb.neighbor.le_inquiry.results++;

      btm_cb.neighbor.le_legacy_scan.results++;
@@ -2947,10 +2944,11 @@ void btm_ble_process_adv_pkt_cont_for_inquiry( 
      return;

    }

  } else if (p_i->inq_count !=

             p_inq->inq_counter) /* first time seen in this inquiry */

             btm_cb.btm_inq_vars

                 .inq_counter) /* first time seen in this inquiry */

  {

    p_i->time_of_resp = bluetooth::common::time_get_os_boottime_ms();

    p_inq->inq_cmpl_info.num_resp++;

    btm_cb.btm_inq_vars.inq_cmpl_info.num_resp++;

  }



  /* update the LE device information in inquiry database */
@@ -2985,7 +2983,7 @@ void btm_ble_process_adv_pkt_cont_for_inquiry( 


  if (!update) result &= ~BTM_BLE_INQ_RESULT;



  tBTM_INQ_RESULTS_CB* p_inq_results_cb = p_inq->p_inq_results_cb;

  tBTM_INQ_RESULTS_CB* p_inq_results_cb = btm_cb.btm_inq_vars.p_inq_results_cb;

  if (p_inq_results_cb && (result & BTM_BLE_INQ_RESULT)) {

    (p_inq_results_cb)((tBTM_INQ_RESULTS*)&p_i->inq_info.results,

                       const_cast<uint8_t*>(advertising_data.data()),
@@ -3075,7 +3073,6 @@ static void btm_ble_stop_scan(void) { 
 *

 ******************************************************************************/

void btm_ble_stop_inquiry(void) {

  tBTM_INQUIRY_VAR_ST* p_inq = &btm_cb.btm_inq_vars;

  tBTM_BLE_CB* p_ble_cb = &btm_cb.ble_ctr_cb;



  alarm_cancel(p_ble_cb->inq_var.inquiry_timer);
@@ -3107,10 +3104,12 @@ void btm_ble_stop_inquiry(void) { 


  /* If we have a callback registered for inquiry complete, call it */

  LOG_VERBOSE("BTM Inq Compl Callback: status 0x%02x, num results %d",

              p_inq->inq_cmpl_info.status, p_inq->inq_cmpl_info.num_resp);

              btm_cb.btm_inq_vars.inq_cmpl_info.status,

              btm_cb.btm_inq_vars.inq_cmpl_info.num_resp);



  btm_process_inq_complete(

      HCI_SUCCESS, (uint8_t)(p_inq->inqparms.mode & BTM_BLE_INQUIRY_MASK));

      HCI_SUCCESS,

      (uint8_t)(btm_cb.btm_inq_vars.inqparms.mode & BTM_BLE_INQUIRY_MASK));

}



/*******************************************************************************