espressif/esp-nimble
1
0
Fork 0
mirror of https://github.com/espressif/esp-nimble.git synced 2026-09-03 15:50:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

nimble/ll: Fix setting sent_to_host flag

Browse Source 
There's no point in setting flag only to clear it in a moment - just
make sure we don't set it when not needed.
This commit is contained in:
Andrzej Kaczmarek
2019-11-05 13:49:09 +01:00
parent ffc76ac0e9
commit d2853c9a20
1 changed files with 3 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nimble/controller/src/ble_ll_scan.c
+3 -10
View File
@@ -2735,16 +2735,9 @@ ble_ll_hci_send_ext_adv_report(uint8_t ptype, uint8_t *adva, uint8_t adva_type,
ble_ll_hci_event_send(hci_ev);
if (aux_data) {
BLE_LL_AUX_SET_FLAG(aux_data, BLE_LL_SENT_EVENT_TO_HOST);
/* In case it is scannable AUX and we are waiting for scan response,
* let us clear BLE_LL_SENT_EVENT_TO_HOST flag as we consider scan response
* as separate report even aux_data is reused.
* This is needed to proper detect of not completed advertising
* reports.
*/
if ((aux_data->evt_type & BLE_HCI_ADV_SCAN_MASK) &&
!(aux_data->evt_type & BLE_HCI_ADV_SCAN_RSP_MASK)) {
BLE_LL_AUX_CLEAR_FLAG(aux_data, BLE_LL_SENT_EVENT_TO_HOST);
if (!(aux_data->evt_type & BLE_HCI_ADV_SCAN_MASK) ||
(aux_data->evt_type & BLE_HCI_ADV_SCAN_RSP_MASK)) {
BLE_LL_AUX_SET_FLAG(aux_data, BLE_LL_SENT_EVENT_TO_HOST);
}
}