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

Commit 9638e17d authored by Md Shahriar Hossain Sajib's avatar Md Shahriar Hossain Sajib Committed by Automerger Merge Worker
Browse files

Stop btif queue to get stuck on the same node am: 25efd4a1 

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/21028704



Change-Id: Ic9d7b92ef8fad55eb1b7d72ecf916281308227bc
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 53523665 25efd4a1

system/bta/hf_client/bta_hf_client_api.cc

+4 −3
Original line number Diff line number Diff line
@@ -80,17 +80,17 @@ void BTA_HfClientDisable(void) { bta_hf_client_api_disable(); } 
 * Description      Opens up a RF connection to the remote device and

 *                  subsequently set it up for a HF SLC

 *

 * Returns          void

 * Returns          bt_status_t

 *

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

void BTA_HfClientOpen(const RawAddress& bd_addr, uint16_t* p_handle) {

bt_status_t BTA_HfClientOpen(const RawAddress& bd_addr, uint16_t* p_handle) {

  APPL_TRACE_DEBUG("%s", __func__);

  tBTA_HF_CLIENT_API_OPEN* p_buf =

      (tBTA_HF_CLIENT_API_OPEN*)osi_malloc(sizeof(tBTA_HF_CLIENT_API_OPEN));



  if (!bta_hf_client_allocate_handle(bd_addr, p_handle)) {

    APPL_TRACE_ERROR("%s: could not allocate handle", __func__);

    return;

    return BT_STATUS_FAIL;

  }



  p_buf->hdr.event = BTA_HF_CLIENT_API_OPEN_EVT;
@@ -98,6 +98,7 @@ void BTA_HfClientOpen(const RawAddress& bd_addr, uint16_t* p_handle) { 
  p_buf->bd_addr = bd_addr;



  bta_sys_sendmsg(p_buf);

  return BT_STATUS_SUCCESS;

}



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

system/bta/include/bta_hf_client_api.h

100755 → 100644
+2 −2
Original line number Diff line number Diff line
@@ -321,10 +321,10 @@ void BTA_HfClientDisable(void); 
 *                  calls to do any AT operations

 *

 *

 * Returns          void

 * Returns          bt_status_t

 *

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

void BTA_HfClientOpen(const RawAddress& bd_addr, uint16_t* p_handle);

bt_status_t BTA_HfClientOpen(const RawAddress& bd_addr, uint16_t* p_handle);



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

 *

system/btif/src/btif_hf_client.cc

+1 −3
Original line number Diff line number Diff line
@@ -313,9 +313,7 @@ static bt_status_t connect_int(RawAddress* bd_addr, uint16_t uuid) { 
   * The handle is valid until we have called BTA_HfClientClose or the LL

   * has notified us of channel close due to remote closing, error etc.

   */

  BTA_HfClientOpen(cb->peer_bda, &cb->handle);



  return BT_STATUS_SUCCESS;

  return BTA_HfClientOpen(cb->peer_bda, &cb->handle);

}



static bt_status_t connect(RawAddress* bd_addr) {

system/test/mock/mock_bta_hf_client_api.cc

+2 −1
Original line number Diff line number Diff line
@@ -51,8 +51,9 @@ void BTA_HfClientAudioOpen(uint16_t handle) { 
void BTA_HfClientClose(uint16_t handle) { mock_function_count_map[__func__]++; }

void BTA_HfClientDisable(void) { mock_function_count_map[__func__]++; }

void BTA_HfClientDumpStatistics(int fd) { mock_function_count_map[__func__]++; }

void BTA_HfClientOpen(const RawAddress& bd_addr, uint16_t* p_handle) {

bt_status_t BTA_HfClientOpen(const RawAddress& bd_addr, uint16_t* p_handle) {

  mock_function_count_map[__func__]++;

  return BT_STATUS_SUCCESS;

}

void BTA_HfClientSendAT(uint16_t handle, tBTA_HF_CLIENT_AT_CMD_TYPE at,

                        uint32_t val1, uint32_t val2, const char* str) {