commit dd63aba0df7f2a3432cb6d02e0bd833ed54bf341
author Chris Manton <cmanton@google.com> Sun Jan 10 10:45:58 2021 -0800
committer Chris Manton <cmanton@google.com> Tue Jan 19 23:12:57 2021 -0800
tree c992fc524e4ec2dea2cf885c64570e85aed95d90
parent 86c60f789cb96d48ece3ef4820a507e0939a1316

Remove unnecessary stack/acl/btm_acl::btm_pm_is_le_link

Towards meaningful code

Bug: 163134718
Tag: #refactor
Test: gd/cert/run

Change-Id: Ic5569a10bad4de55ccba1821e0433ef053b033ad

system/stack/acl/acl.h

diff --git a/system/stack/acl/acl.h b/system/stack/acl/acl.h
index d174cbf..10c18c5 100644
--- a/system/stack/acl/acl.h
+++ b/system/stack/acl/acl.h
@@ -368,7 +368,6 @@
  ****************************************************/
 struct sACL_CB {
  private:
-  friend bool btm_pm_is_le_link(const RawAddress& remote_bda);
   friend int btm_pm_find_acl_ind(const RawAddress& remote_bda);
   friend tBTM_PM_MCB* acl_power_mode_from_handle(uint16_t hci_handle);
   friend tBTM_STATUS BTM_SetPowerMode(uint8_t pm_id,

system/stack/acl/btm_acl.cc

diff --git a/system/stack/acl/btm_acl.cc b/system/stack/acl/btm_acl.cc
index 3911265..7439296 100644
--- a/system/stack/acl/btm_acl.cc
+++ b/system/stack/acl/btm_acl.cc
@@ -2493,17 +2493,6 @@
   return xx;
 }
 
-bool btm_pm_is_le_link(const RawAddress& remote_bda) {
-  const tACL_CONN* p_acl = &btm_cb.acl_cb_.acl_db[0];
-  for (uint8_t xx = 0; xx < MAX_L2CAP_LINKS; xx++, p_acl++) {
-    if (p_acl->in_use && p_acl->remote_addr == remote_bda &&
-        p_acl->is_transport_ble()) {
-      return true;
-    }
-  }
-  return false;
-}
-
 /*******************************************************************************
  *
  * Function         btm_ble_refresh_local_resolvable_private_addr

system/stack/acl/btm_pm.cc

diff --git a/system/stack/acl/btm_pm.cc b/system/stack/acl/btm_pm.cc
index 6da7ee1..ac92a8a 100644
--- a/system/stack/acl/btm_pm.cc
+++ b/system/stack/acl/btm_pm.cc
@@ -205,10 +205,6 @@
 
   int acl_ind = btm_pm_find_acl_ind(remote_bda);
   if (acl_ind == MAX_L2CAP_LINKS) {
-    if (btm_pm_is_le_link(remote_bda)) {
-      LOG_ERROR("Setting power mode on le link is unsupported");
-      return BTM_MODE_UNSUPPORTED;
-    }
     LOG_ERROR("br_edr acl addr:%s is unknown", PRIVATE_ADDRESS(remote_bda));
     return BTM_UNKNOWN_ADDR;
   }

system/stack/include/acl_api.h

diff --git a/system/stack/include/acl_api.h b/system/stack/include/acl_api.h
index df7fbe4..dac414d 100644
--- a/system/stack/include/acl_api.h
+++ b/system/stack/include/acl_api.h
@@ -273,7 +273,6 @@
 const RawAddress acl_address_from_handle(uint16_t hci_handle);
 tBTM_PM_MCB* acl_power_mode_from_handle(uint16_t hci_handle);
 int btm_pm_find_acl_ind(const RawAddress& remote_bda);
-bool btm_pm_is_le_link(const RawAddress& remote_bda);
 
 void btm_ble_refresh_local_resolvable_private_addr(
     const RawAddress& pseudo_addr, const RawAddress& local_rpa);