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

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

Merge "Add check if controller supports write link supervision timeout" am:... 

Merge "Add check if controller supports write link supervision timeout" am: 4c738515 am: d0cd8cb6

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

Change-Id: I556aebb08deb862ec9001ffefb5217d512ce425b
parents f91fc641 d0cd8cb6

system/main/shim/controller.cc

+5 −0
Original line number Diff line number Diff line
@@ -423,3 +423,8 @@ const controller_t* bluetooth::shim::controller_get_interface() { 
void bluetooth::shim::controller_clear_event_mask() {

  bluetooth::shim::GetController()->SetEventMask(0);

}



bool bluetooth::shim::controller_is_write_link_supervision_timeout_supported() {

  return bluetooth::shim::GetController()->IsSupported(

      bluetooth::hci::OpCode::WRITE_LINK_SUPERVISION_TIMEOUT);

}

system/main/shim/controller.h

+1 −0
Original line number Diff line number Diff line
@@ -26,6 +26,7 @@ namespace shim { 
const controller_t* controller_get_interface();



void controller_clear_event_mask();

bool controller_is_write_link_supervision_timeout_supported();



}  // namespace shim

}  // namespace bluetooth

system/stack/acl/btm_acl.cc

+20 −2
Original line number Diff line number Diff line
@@ -33,6 +33,8 @@ 


#define LOG_TAG "btm_acl"



#include <base/logging.h>



#include <cstdint>



#include "bta/include/bta_dm_acl.h"
@@ -44,6 +46,7 @@ 
#include "include/l2cap_hci_link_interface.h"

#include "main/shim/acl_api.h"

#include "main/shim/btm_api.h"

#include "main/shim/controller.h"

#include "main/shim/dumpsys.h"

#include "main/shim/l2c_api.h"

#include "main/shim/shim.h"
@@ -69,8 +72,6 @@ 
#include "types/hci_role.h"

#include "types/raw_address.h"



#include <base/logging.h>



void BTM_update_version_info(const RawAddress& bd_addr,

                             const remote_version_info& remote_version_info);
@@ -207,6 +208,15 @@ void hci_btm_set_link_supervision_timeout(tACL_CONN& link, uint16_t timeout) { 
    return;

  }



  if (!bluetooth::shim::

          controller_is_write_link_supervision_timeout_supported()) {

    LOG_WARN(

        "UNSUPPORTED by controller write link supervision timeout:%.2fms "

        "bd_addr:%s",

        supervision_timeout_to_seconds(timeout),

        PRIVATE_ADDRESS(link.RemoteAddress()));

    return;

  }

  LOG_DEBUG("Setting link supervision timeout:%.2fs peer:%s",

            double(timeout) * 0.01, PRIVATE_ADDRESS(link.RemoteAddress()));

  link.link_super_tout = timeout;
@@ -1176,6 +1186,14 @@ tBTM_STATUS BTM_SetLinkSuperTout(const RawAddress& remote_bda, 


  /* Only send if current role is Central; 2.0 spec requires this */

  if (p_acl->link_role == HCI_ROLE_CENTRAL) {

    if (!bluetooth::shim::

            controller_is_write_link_supervision_timeout_supported()) {

      LOG_WARN(

          "UNSUPPORTED by controller write link supervision timeout:%.2fms "

          "bd_addr:%s",

          supervision_timeout_to_seconds(timeout), PRIVATE_ADDRESS(remote_bda));

      return BTM_MODE_UNSUPPORTED;

    }

    p_acl->link_super_tout = timeout;

    btsnd_hcic_write_link_super_tout(p_acl->hci_handle, timeout);

    LOG_DEBUG("Set supervision timeout:%.2fms bd_addr:%s",

system/test/mock/mock_main_shim_controller.cc

+5 −0
Original line number Diff line number Diff line
@@ -39,3 +39,8 @@ const controller_t* bluetooth::shim::controller_get_interface() { 
void bluetooth::shim::controller_clear_event_mask() {

  mock_function_count_map[__func__]++;

}



bool bluetooth::shim::controller_is_write_link_supervision_timeout_supported() {

  mock_function_count_map[__func__]++;

  return false;

}