From 899e89d4c672c94cea794d5e3128e9cefce15382 Mon Sep 17 00:00:00 2001 From: Damien George Date: Sun, 5 Apr 2020 15:31:53 +1000 Subject: [PATCH] extmod/btstack: Pass through SCAN_RSP events. The latest version of BTstack has a bug fixed so that it correctly configures scan parameters if they are set right after activating the stack. This means that BLE.gap_scan() will correctly set the scanning to passive and so SCAN_RSP events are not passed through, so we don't need to explicitly filter them in our bindings. --- extmod/btstack/modbluetooth_btstack.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/extmod/btstack/modbluetooth_btstack.c b/extmod/btstack/modbluetooth_btstack.c index 84e1a85f34..c9cc0a8864 100644 --- a/extmod/btstack/modbluetooth_btstack.c +++ b/extmod/btstack/modbluetooth_btstack.c @@ -115,10 +115,7 @@ STATIC void btstack_packet_handler(uint8_t packet_type, uint16_t channel, uint8_ int8_t rssi = gap_event_advertising_report_get_rssi(packet); uint8_t length = gap_event_advertising_report_get_data_length(packet); const uint8_t *data = gap_event_advertising_report_get_data(packet); - // Emit an event for all advertising types except SCAN_RSP. - if (adv_event_type < 4) { - mp_bluetooth_gap_on_scan_result(address_type, address, adv_event_type, rssi, data, length); - } + mp_bluetooth_gap_on_scan_result(address_type, address, adv_event_type, rssi, data, length); } else if (event_type == HCI_EVENT_DISCONNECTION_COMPLETE) { DEBUG_EVENT_printf(" --> hci disconnect complete\n"); uint16_t conn_handle = hci_event_disconnection_complete_get_connection_handle(packet);