Merge branch 'bugfix/config_tpoll_fail_v4.3' into 'release/v4.3'

Bugfix/config tpoll fail v4.3

See merge request espressif/esp-idf!26454
This commit is contained in:
Wang Meng Yang 2023-10-16 10:59:55 +08:00
commit 947085b7c4
5 changed files with 34 additions and 29 deletions

@ -1 +1 @@
Subproject commit 19fa60b819db1e3bb8a07790f819b69657e4ae3f Subproject commit 943b0f24eb6ed390213599e4a9556c66844ef0db

View File

@ -420,6 +420,10 @@ esp_err_t esp_bt_gap_set_qos(esp_bd_addr_t remote_bda, uint32_t t_poll)
return ESP_ERR_INVALID_STATE; return ESP_ERR_INVALID_STATE;
} }
if (t_poll < ESP_BT_GAP_TPOLL_MIN || t_poll > ESP_BT_GAP_TPOLL_MAX) {
return ESP_ERR_INVALID_ARG;
}
msg.sig = BTC_SIG_API_CALL; msg.sig = BTC_SIG_API_CALL;
msg.pid = BTC_PID_GAP_BT; msg.pid = BTC_PID_GAP_BT;
msg.act = BTC_GAP_BT_ACT_SET_QOS; msg.act = BTC_GAP_BT_ACT_SET_QOS;

View File

@ -238,6 +238,11 @@ typedef enum {
#define ESP_BT_GAP_MIN_INQ_LEN (0x01) /*!< Minimum inquiry duration, unit is 1.28s */ #define ESP_BT_GAP_MIN_INQ_LEN (0x01) /*!< Minimum inquiry duration, unit is 1.28s */
#define ESP_BT_GAP_MAX_INQ_LEN (0x30) /*!< Maximum inquiry duration, unit is 1.28s */ #define ESP_BT_GAP_MAX_INQ_LEN (0x30) /*!< Maximum inquiry duration, unit is 1.28s */
/** Minimum, Default and Maximum poll interval **/
#define ESP_BT_GAP_TPOLL_MIN (0x0006) /*!< Minimum poll interval, unit is 625 microseconds */
#define ESP_BT_GAP_TPOLL_DFT (0x0028) /*!< Default poll interval, unit is 625 microseconds */
#define ESP_BT_GAP_TPOLL_MAX (0x1000) /*!< Maximum poll interval, unit is 625 microseconds */
/// GAP state callback parameters /// GAP state callback parameters
typedef union { typedef union {
/** /**

View File

@ -34,13 +34,13 @@
void bta_dm_set_qos(tBTA_DM_MSG *p_data) void bta_dm_set_qos(tBTA_DM_MSG *p_data)
{ {
FLOW_SPEC p_flow = { FLOW_SPEC p_flow = {
.qos_flags = 0, /* TBD */ .qos_flags = 0, /* TBD */
.service_type = GUARANTEED, /* see below */ .service_type = NO_TRAFFIC, /* service_type */
.token_rate = 0, /* bytes/second */ .token_rate = 0, /* bytes/second */
.token_bucket_size = 0, /* bytes */ .token_bucket_size = 0, /* bytes */
.peak_bandwidth = 0, /* bytes/second */ .peak_bandwidth = 0, /* bytes/second */
.latency = 625 * p_data->qos_set.t_poll, /* microseconds */ .latency = 625 * p_data->qos_set.t_poll, /* microseconds */
.delay_variation = 0xFFFFFFFF /* microseconds */ .delay_variation = 0xFFFFFFFF /* microseconds */
}; };
tBTM_STATUS status = BTM_SetQoS (p_data->qos_set.bd_addr, &p_flow, p_data->qos_set.p_cb); tBTM_STATUS status = BTM_SetQoS (p_data->qos_set.bd_addr, &p_flow, p_data->qos_set.p_cb);

View File

@ -2659,37 +2659,33 @@ void btm_acl_connected(BD_ADDR bda, UINT16 handle, UINT8 link_type, UINT8 enc_mo
*******************************************************************************/ *******************************************************************************/
void btm_acl_disconnected(UINT16 handle, UINT8 reason) void btm_acl_disconnected(UINT16 handle, UINT8 reason)
{ {
BOOLEAN need_report = TRUE;
/* Report BR/EDR ACL disconnection result to upper layer */
tACL_CONN *conn = btm_handle_to_acl(handle);
if (conn) {
#if BLE_INCLUDED == TRUE
if (conn->transport == BT_TRANSPORT_BR_EDR)
#endif
{
tBTM_ACL_LINK_STAT_EVENT_DATA evt_data = {
.event = BTM_ACL_DISCONN_CMPL_EVT,
.link_act.disconn_cmpl.reason = reason,
.link_act.disconn_cmpl.handle = handle,
};
bdcpy(evt_data.link_act.disconn_cmpl.bd_addr, conn->remote_addr);
btm_acl_link_stat_report(&evt_data);
}
}
#if BTM_SCO_INCLUDED == TRUE #if BTM_SCO_INCLUDED == TRUE
/* If L2CAP doesn't know about it, send it to SCO */ /* If L2CAP doesn't know about it, send it to SCO */
if (!l2c_link_hci_disc_comp (handle, reason)) { if (!l2c_link_hci_disc_comp (handle, reason)) {
btm_sco_removed (handle, reason); btm_sco_removed (handle, reason);
need_report = FALSE;
} }
#else #else
l2c_link_hci_disc_comp(handle, reason); l2c_link_hci_disc_comp(handle, reason);
#endif /* BTM_SCO_INCLUDED */ #endif /* BTM_SCO_INCLUDED */
if (need_report) {
/* Report BR/EDR ACL disconnection result to upper layer */
tACL_CONN *conn = btm_handle_to_acl(handle);
if (conn) {
#if BLE_INCLUDED == TRUE
if (conn->transport == BT_TRANSPORT_BR_EDR)
#endif
{
tBTM_ACL_LINK_STAT_EVENT_DATA evt_data = {
.event = BTM_ACL_DISCONN_CMPL_EVT,
.link_act.disconn_cmpl.reason = reason,
.link_act.disconn_cmpl.handle = handle,
};
bdcpy(evt_data.link_act.disconn_cmpl.bd_addr, conn->remote_addr);
btm_acl_link_stat_report(&evt_data);
}
}
}
#if (SMP_INCLUDED == TRUE) #if (SMP_INCLUDED == TRUE)
/* Notify security manager */ /* Notify security manager */
btm_sec_disconnected(handle, reason); btm_sec_disconnected(handle, reason);