From c25d22728009d1ffd89422ecd2ccc0162020cb79 Mon Sep 17 00:00:00 2001 From: Jin Cheng Date: Mon, 6 Feb 2023 15:30:35 +0800 Subject: [PATCH] Fixed build errors when sniff subrating is enabled. --- components/bt/host/bluedroid/bta/dm/bta_dm_pm.c | 2 +- .../bluedroid/common/include/common/bt_target.h | 2 +- components/bt/host/bluedroid/stack/btm/btm_pm.c | 8 ++++---- .../bt/host/bluedroid/stack/btu/btu_hcif.c | 17 +++-------------- 4 files changed, 9 insertions(+), 20 deletions(-) diff --git a/components/bt/host/bluedroid/bta/dm/bta_dm_pm.c b/components/bt/host/bluedroid/bta/dm/bta_dm_pm.c index ff42a65a17..41e77a6e92 100644 --- a/components/bt/host/bluedroid/bta/dm/bta_dm_pm.c +++ b/components/bt/host/bluedroid/bta/dm/bta_dm_pm.c @@ -54,7 +54,7 @@ static void bta_dm_pm_stop_timer_by_index(tBTA_PM_TIMER *p_timer, #if (BTM_SSR_INCLUDED == TRUE) #if (defined BTA_HH_INCLUDED && BTA_HH_INCLUDED == TRUE) -#include "../hh/bta_hh_int.h" +#include "bta_hh_int.h" /* BTA_DM_PM_SSR1 will be dedicated for HH SSR setting entry, no other profile can use it */ #define BTA_DM_PM_SSR_HH BTA_DM_PM_SSR1 #endif /* (defined BTA_HH_INCLUDED && BTA_HH_INCLUDED == TRUE) */ diff --git a/components/bt/host/bluedroid/common/include/common/bt_target.h b/components/bt/host/bluedroid/common/include/common/bt_target.h index 1b2c210c91..1891981f39 100644 --- a/components/bt/host/bluedroid/common/include/common/bt_target.h +++ b/components/bt/host/bluedroid/common/include/common/bt_target.h @@ -880,7 +880,7 @@ /* TRUE to include Sniff Subrating */ #if (BTA_DM_PM_INCLUDED == TRUE) #ifndef BTM_SSR_INCLUDED -#define BTM_SSR_INCLUDED FALSE +#define BTM_SSR_INCLUDED TRUE #endif #endif /* BTA_DM_PM_INCLUDED */ diff --git a/components/bt/host/bluedroid/stack/btm/btm_pm.c b/components/bt/host/bluedroid/stack/btm/btm_pm.c index 3ea3dfbddd..db3679e182 100644 --- a/components/bt/host/bluedroid/stack/btm/btm_pm.c +++ b/components/bt/host/bluedroid/stack/btm/btm_pm.c @@ -838,7 +838,7 @@ void btm_pm_proc_ssr_evt (UINT8 *p, UINT16 evt_len) UINT8 status; UINT16 handle; UINT16 max_rx_lat; - int xx, yy; + int xx; tBTM_PM_MCB *p_cb; tACL_CONN *p_acl = NULL; UINT16 use_ssr = TRUE; @@ -863,10 +863,10 @@ void btm_pm_proc_ssr_evt (UINT8 *p, UINT16 evt_len) } /* notify registered parties */ - for (yy = 0; yy < BTM_MAX_PM_RECORDS; yy++) { - if (btm_cb.pm_reg_db[yy].mask & BTM_PM_REG_NOTIF) { + for (xx = 0; xx < BTM_MAX_PM_RECORDS; xx++) { + if (btm_cb.pm_reg_db[xx].mask & BTM_PM_REG_NOTIF) { if ( p_acl) { - (*btm_cb.pm_reg_db[yy].cback)( p_acl->remote_addr, BTM_PM_STS_SSR, use_ssr, status); + (*btm_cb.pm_reg_db[xx].cback)( p_acl->remote_addr, BTM_PM_STS_SSR, use_ssr, status); } } } diff --git a/components/bt/host/bluedroid/stack/btu/btu_hcif.c b/components/bt/host/bluedroid/stack/btu/btu_hcif.c index 30dfaa436b..da9ae8a93f 100644 --- a/components/bt/host/bluedroid/stack/btu/btu_hcif.c +++ b/components/bt/host/bluedroid/stack/btu/btu_hcif.c @@ -116,11 +116,7 @@ static void btu_hcif_link_supv_to_changed_evt (UINT8 *p); static void btu_hcif_enhanced_flush_complete_evt (void); #endif -#if (BTM_SSR_INCLUDED == TRUE) static void btu_hcif_ssr_evt (UINT8 *p, UINT16 evt_len); -#else -static void btu_hcif_ssr_evt_dump (UINT8 *p, UINT16 evt_len); -#endif /* BTM_SSR_INCLUDED == TRUE */ #if BLE_INCLUDED == TRUE static void btu_ble_ll_conn_complete_evt (UINT8 *p, UINT16 evt_len); @@ -293,11 +289,7 @@ void btu_hcif_process_event (UNUSED_ATTR UINT8 controller_id, BT_HDR *p_msg) btu_hcif_esco_connection_chg_evt (p); break; case HCI_SNIFF_SUB_RATE_EVT: -#if (BTM_SSR_INCLUDED == TRUE) btu_hcif_ssr_evt (p, hci_evt_len); -#else - btu_hcif_ssr_evt_dump (p, hci_evt_len); -#endif /* BTM_SSR_INCLUDED == TRUE */ break; case HCI_RMT_HOST_SUP_FEAT_NOTIFY_EVT: btu_hcif_host_support_evt (p); @@ -1445,14 +1437,12 @@ static void btu_hcif_mode_change_evt (UINT8 *p) ** Returns void ** *******************************************************************************/ -#if (BTM_SSR_INCLUDED == TRUE) static void btu_hcif_ssr_evt (UINT8 *p, UINT16 evt_len) { +#if (BTM_SSR_INCLUDED == TRUE) btm_pm_proc_ssr_evt(p, evt_len); -} -#else -static void btu_hcif_ssr_evt_dump (UINT8 *p, UINT16 evt_len) -{ +#endif + UINT8 status; UINT16 handle; UINT16 max_tx_lat; @@ -1470,7 +1460,6 @@ static void btu_hcif_ssr_evt_dump (UINT8 *p, UINT16 evt_len) HCI_TRACE_WARNING("hcif ssr evt: st 0x%x, hdl 0x%x, tx_lat %d rx_lat %d", status, handle, max_tx_lat, max_rx_lat); } -#endif /******************************************************************************* **