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

Commit a8187fc9 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "stack::avct: Various cleanup" into main am: 26ea51ee 

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



Change-Id: Ie04f5a6da6349e5575ee7ee51c6b5c7a23f2f1e6
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents f6541bcd 26ea51ee

system/bta/av/bta_av_act.cc

+25 −22
Original line number Diff line number Diff line
@@ -321,7 +321,7 @@ static void bta_av_rc_msg_cback(uint8_t handle, uint8_t label, uint8_t opcode, t 
 * Returns          the created rc handle

 *

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

uint8_t bta_av_rc_create(tBTA_AV_CB* p_cb, uint8_t role, uint8_t shdl, uint8_t lidx) {

uint8_t bta_av_rc_create(tBTA_AV_CB* p_cb, tAVCT_ROLE role, uint8_t shdl, uint8_t lidx) {

  if ((!btif_av_src_sink_coexist_enabled() ||

       (btif_av_src_sink_coexist_enabled() && !btif_av_is_sink_enabled() &&

        btif_av_is_source_enabled())) &&
@@ -330,14 +330,13 @@ uint8_t bta_av_rc_create(tBTA_AV_CB* p_cb, uint8_t role, uint8_t shdl, uint8_t l 
    return BTA_AV_RC_HANDLE_NONE;

  }



  tAVRC_CONN_CB ccb;

  RawAddress bda = RawAddress::kAny;

  uint8_t status = BTA_AV_RC_ROLE_ACP;

  int i;

  uint8_t rc_handle;

  tBTA_AV_RCB* p_rcb;

  tBTA_AV_RCB* p_rcb{nullptr};



  if (role == AVCT_INT) {

  if (role == AVCT_ROLE_INITIATOR) {

    // Can't grab a stream control block that doesn't have a valid handle

    if (!shdl) {

      log::error("Can't grab stream control block for shdl = {} -> index = {}", shdl, shdl - 1);
@@ -357,14 +356,16 @@ uint8_t bta_av_rc_create(tBTA_AV_CB* p_cb, uint8_t role, uint8_t shdl, uint8_t l 
    }

  }



  ccb.ctrl_cback = base::Bind(bta_av_rc_ctrl_cback);

  ccb.msg_cback = base::Bind(bta_av_rc_msg_cback);

  ccb.company_id = p_bta_av_cfg->company_id;

  ccb.conn = role;

  /* note: BTA_AV_FEAT_RCTG = AVRC_CT_TARGET, BTA_AV_FEAT_RCCT = AVRC_CT_CONTROL

   */

  ccb.control = p_cb->features &

                (BTA_AV_FEAT_RCTG | BTA_AV_FEAT_RCCT | BTA_AV_FEAT_METADATA | AVRC_CT_PASSIVE);

  tAVRC_CONN_CB ccb = {

          .ctrl_cback = base::Bind(bta_av_rc_ctrl_cback),

          .msg_cback = base::Bind(bta_av_rc_msg_cback),

          .company_id = p_bta_av_cfg->company_id,

          .conn = role,

          // note: BTA_AV_FEAT_RCTG = AVRC_CT_TARGET, BTA_AV_FEAT_RCCT = AVRC_CT_CONTROL

          .control =

                  static_cast<uint8_t>(p_cb->features & (BTA_AV_FEAT_RCTG | BTA_AV_FEAT_RCCT |

                                                         BTA_AV_FEAT_METADATA | AVRC_CT_PASSIVE)),

  };



  if (AVRC_Open(&rc_handle, &ccb, bda) != AVRC_SUCCESS) {

    DEVICE_IOT_CONFIG_ADDR_INT_ADD_ONE(bda, IOT_CONF_KEY_AVRCP_CONN_FAIL_COUNT);
@@ -392,8 +393,8 @@ uint8_t bta_av_rc_create(tBTA_AV_CB* p_cb, uint8_t role, uint8_t shdl, uint8_t l 
    p_cb->rc_acp_idx = (i + 1);

    log::verbose("rc_acp_handle:{} idx:{}", p_cb->rc_acp_handle, p_cb->rc_acp_idx);

  }

  log::verbose("create {}, role: {}, shdl:{}, rc_handle:{}, lidx:{}, status:0x{:x}", i, role, shdl,

               p_rcb->handle, lidx, p_rcb->status);

  log::verbose("create {}, role: {}, shdl:{}, rc_handle:{}, lidx:{}, status:0x{:x}", i,

               avct_role_text(role), shdl, p_rcb->handle, lidx, p_rcb->status);



  return rc_handle;

}
@@ -539,7 +540,7 @@ void bta_av_rc_opened(tBTA_AV_CB* p_cb, tBTA_AV_DATA* p_data) { 
  /* listen to browsing channel when the connection is open,

   * if peer initiated AVRCP connection and local device supports browsing

   * channel */

  AVRC_OpenBrowse(p_data->rc_conn_chg.handle, AVCT_ACP);

  AVRC_OpenBrowse(p_data->rc_conn_chg.handle, AVCT_ROLE_ACCEPTOR);



  if (p_cb->rcb[i].lidx == (BTA_AV_NUM_LINKS + 1) && shdl != 0) {

    /* rc is opened on the RC only ACP channel, but is for a specific
@@ -618,7 +619,7 @@ void bta_av_rc_opened(tBTA_AV_CB* p_cb, tBTA_AV_DATA* p_data) { 
         (rc_open.peer_tg_features & BTA_AV_FEAT_BROWSE))) {

      if ((p_cb->rcb[i].status & BTA_AV_RC_ROLE_MASK) == BTA_AV_RC_ROLE_INT) {

        log::verbose("opening AVRC Browse channel");

        AVRC_OpenBrowse(p_data->rc_conn_chg.handle, AVCT_INT);

        AVRC_OpenBrowse(p_data->rc_conn_chg.handle, AVCT_ROLE_INITIATOR);

      }

    }

    return;
@@ -649,7 +650,7 @@ void bta_av_rc_opened(tBTA_AV_CB* p_cb, tBTA_AV_DATA* p_data) { 
  if ((p_cb->features & BTA_AV_FEAT_BROWSE) && (rc_open.peer_features & BTA_AV_FEAT_BROWSE) &&

      ((p_cb->rcb[i].status & BTA_AV_RC_ROLE_MASK) == BTA_AV_RC_ROLE_INT)) {

    log::verbose("opening AVRC Browse channel");

    AVRC_OpenBrowse(p_data->rc_conn_chg.handle, AVCT_INT);

    AVRC_OpenBrowse(p_data->rc_conn_chg.handle, AVCT_ROLE_INITIATOR);

  }

}
@@ -1335,7 +1336,7 @@ void bta_av_conn_chg(tBTA_AV_DATA* p_data) { 


    /* if the AVRCP is no longer listening, create the listening channel */

    if (bta_av_cb.rc_acp_handle == BTA_AV_RC_HANDLE_NONE && bta_av_cb.features & BTA_AV_FEAT_RCTG) {

      bta_av_rc_create(&bta_av_cb, AVCT_ACP, 0, BTA_AV_NUM_LINKS + 1);

      bta_av_rc_create(&bta_av_cb, AVCT_ROLE_ACCEPTOR, 0, BTA_AV_NUM_LINKS + 1);

    }

  }
@@ -1554,7 +1555,7 @@ void bta_av_sig_chg(tBTA_AV_DATA* p_data) { 
      p_lcb->conn_msk = 0; /* clear the connect mask */

      /* start listening when the signal channel is open */

      if (p_cb->features & BTA_AV_FEAT_RCTG) {

        bta_av_rc_create(p_cb, AVCT_ACP, 0, p_lcb->lidx);

        bta_av_rc_create(p_cb, AVCT_ROLE_ACCEPTOR, 0, p_lcb->lidx);

      }

      /* this entry is not used yet. */

      p_cb->conn_lcb |= mask; /* mark it as used */
@@ -2118,7 +2119,8 @@ static void bta_av_rc_disc_done_all(tBTA_AV_DATA* /* p_data */) { 
          ((p_cb->features & BTA_AV_FEAT_RCTG) && (peer_ct_features & BTA_AV_FEAT_RCCT))) {

        p_lcb = bta_av_find_lcb(p_scb->PeerAddress(), BTA_AV_LCB_FIND);

        if (p_lcb) {

          rc_handle = bta_av_rc_create(p_cb, AVCT_INT, (uint8_t)(p_scb->hdi + 1), p_lcb->lidx);

          rc_handle = bta_av_rc_create(p_cb, AVCT_ROLE_INITIATOR, (uint8_t)(p_scb->hdi + 1),

                                       p_lcb->lidx);

          if (rc_handle != BTA_AV_RC_HANDLE_NONE) {

            p_cb->rcb[rc_handle].peer_ct_features = peer_ct_features;

            p_cb->rcb[rc_handle].peer_tg_features = peer_tg_features;
@@ -2302,7 +2304,8 @@ void bta_av_rc_disc_done(tBTA_AV_DATA* p_data) { 
          ((p_cb->features & BTA_AV_FEAT_RCTG) && (peer_features & BTA_AV_FEAT_RCCT))) {

        p_lcb = bta_av_find_lcb(p_scb->PeerAddress(), BTA_AV_LCB_FIND);

        if (p_lcb) {

          rc_handle = bta_av_rc_create(p_cb, AVCT_INT, (uint8_t)(p_scb->hdi + 1), p_lcb->lidx);

          rc_handle = bta_av_rc_create(p_cb, AVCT_ROLE_INITIATOR, (uint8_t)(p_scb->hdi + 1),

                                       p_lcb->lidx);

          if (rc_handle < BTA_AV_NUM_RCB) {

            p_cb->rcb[rc_handle].peer_features = peer_features;

            p_cb->rcb[rc_handle].cover_art_psm = cover_art_psm;
@@ -2484,7 +2487,7 @@ void bta_av_rc_closed(tBTA_AV_DATA* p_data) { 
  bta_av_data.rc_close = rc_close;

  (*p_cb->p_cback)(BTA_AV_RC_CLOSE_EVT, &bta_av_data);

  if (bta_av_cb.rc_acp_handle == BTA_AV_RC_HANDLE_NONE && bta_av_cb.features & BTA_AV_FEAT_RCTG) {

    bta_av_rc_create(&bta_av_cb, AVCT_ACP, 0, BTA_AV_NUM_LINKS + 1);

    bta_av_rc_create(&bta_av_cb, AVCT_ROLE_ACCEPTOR, 0, BTA_AV_NUM_LINKS + 1);

  }

}

system/bta/av/bta_av_int.h

+3 −2
Original line number Diff line number Diff line
@@ -37,6 +37,7 @@ 
#include "macros.h"

#include "osi/include/list.h"

#include "stack/include/a2dp_constants.h"

#include "stack/include/avct_api.h"

#include "stack/include/avdt_api.h"

#include "stack/include/bt_hdr.h"

#include "stack/include/hci_error_code.h"
@@ -526,7 +527,7 @@ public: 
  bool use_rc;                    /* true if AVRCP is allowed */

  bool started;                   /* true if stream started */

  bool use_rtp_header_marker_bit; /* true if the encoded data packets have RTP

                                   * headers, and the Marker bit in the header

                                   * headers, with the Marker bit in the header

                                   * is set according to RFC 6416 */

  uint8_t co_started;             /* non-zero, if stream started from call-out perspective */

  bool recfg_sup;                 /* true if the first attempt to reconfigure the stream was
@@ -708,7 +709,7 @@ bool bta_av_chk_start(tBTA_AV_SCB* p_scb); 
void bta_av_restore_switch(void);

void bta_av_conn_cback(uint8_t handle, const RawAddress& bd_addr, uint8_t event, tAVDT_CTRL* p_data,

                       uint8_t scb_index);

uint8_t bta_av_rc_create(tBTA_AV_CB* p_cb, uint8_t role, uint8_t shdl, uint8_t lidx);

uint8_t bta_av_rc_create(tBTA_AV_CB* p_cb, tAVCT_ROLE role, uint8_t shdl, uint8_t lidx);

void bta_av_stream_chg(tBTA_AV_SCB* p_scb, bool started);

bool bta_av_is_scb_opening(tBTA_AV_SCB* p_scb);

bool bta_av_is_scb_incoming(tBTA_AV_SCB* p_scb);

system/bta/av/bta_av_main.cc

+2 −2
Original line number Diff line number Diff line
@@ -649,7 +649,7 @@ static void bta_av_api_register(tBTA_AV_DATA* p_data) { 
      }

      /* start listening when A2DP is registered */

      if (bta_av_cb.features & BTA_AV_FEAT_RCTG) {

        bta_av_rc_create(&bta_av_cb, AVCT_ACP, 0, BTA_AV_NUM_LINKS + 1);

        bta_av_rc_create(&bta_av_cb, AVCT_ROLE_ACCEPTOR, 0, BTA_AV_NUM_LINKS + 1);

      }



      /* if the AV and AVK are both supported, it cannot support the CT role
@@ -658,7 +658,7 @@ static void bta_av_api_register(tBTA_AV_DATA* p_data) { 
        /* if TG is not supported, we need to register to AVCT now */

        if ((bta_av_cb.features & (BTA_AV_FEAT_RCTG)) == 0) {

          bta_ar_reg_avct();

          bta_av_rc_create(&bta_av_cb, AVCT_ACP, 0, BTA_AV_NUM_LINKS + 1);

          bta_av_rc_create(&bta_av_cb, AVCT_ROLE_ACCEPTOR, 0, BTA_AV_NUM_LINKS + 1);

        }

        if (com::android::bluetooth::flags::avrcp_sdp_records()) {

          // Add control record for sink profile.

system/bta/jv/bta_jv_act.cc

+2 −4
Original line number Diff line number Diff line
@@ -38,8 +38,6 @@ 
#include "osi/include/allocator.h"

#include "osi/include/properties.h"

#include "stack/btm/btm_sec.h"

#include "stack/include/avct_api.h"  // AVCT_PSM

#include "stack/include/avdt_api.h"  // AVDT_PSM

#include "stack/include/bt_hdr.h"

#include "stack/include/bt_psm_types.h"

#include "stack/include/bt_types.h"
@@ -621,8 +619,8 @@ bool bta_jv_check_psm(uint16_t psm) { 
          }

          break;



        case AVCT_PSM: /* 0x17 */

        case AVDT_PSM: /* 0x19 */

        case BT_PSM_AVCTP: /* 0x17 */

        case BT_PSM_AVDTP: /* 0x19 */

          if (!bta_sys_is_register(BTA_ID_AV)) {

            ret = true;

          }

system/btif/avrcp/avrcp_service.cc

+2 −1
Original line number Diff line number Diff line
@@ -32,6 +32,7 @@ 
#include "btif/include/btif_av.h"

#include "btif/include/btif_common.h"

#include "osi/include/osi.h"

#include "profile/avrcp/avrcp_config.h"

#include "profile/avrcp/device.h"

#include "stack/include/a2dp_api.h"

#include "stack/include/bt_hdr.h"
@@ -100,7 +101,7 @@ public: 
    return AVRC_Open(p_handle, p_ccb, bd_addr);

  }



  uint16_t OpenBrowse(uint8_t handle, uint8_t conn_role) override {

  uint16_t OpenBrowse(uint8_t handle, tAVCT_ROLE conn_role) override {

    return AVRC_OpenBrowse(handle, conn_role);

  }