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

Commit 938cf985 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 am: bde87465 am: 9df555d3

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

Change-Id: I2fadb190269be30261092397b21c34d0e23c7e16
parents 2750ef8f 9df555d3

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;

}