[GitHub] [mynewt-nimble] andrzej-kaczmarek commented on a change in pull request #1077: nimble/ll: Some fixes for ll_scan_aux

[GitBox <gi...@apache.org>]

andrzej-kaczmarek commented on a change in pull request #1077:
URL: https://github.com/apache/mynewt-nimble/pull/1077#discussion_r744581738



##########
File path: nimble/controller/src/ble_ll_scan.c
##########
@@ -1928,6 +1928,22 @@ ble_ll_scan_rx_pkt_in(uint8_t ptype, struct os_mbuf *om, struct ble_mbuf_hdr *hd
     struct ble_ll_scan_sm *scansm;
     struct ble_ll_scan_addr_data addrd;
     uint8_t *targeta;
+    uint8_t max_pdu_type;
+
+    scansm = &g_ble_ll_scan_sm;
+    rxinfo = &hdr->rxinfo;
+
+    /* Ignore PDUs we do not expect here */
+    max_pdu_type = BLE_ADV_PDU_TYPE_ADV_SCAN_IND;

Review comment:
       AUX_CONNECT_REQ and AUX_SCAN_REQ are the same PDUs as CONNECT_IND and SCAN_REQ and thus they have the same PDU type. The only difference is the channel on which they are sent - ble_ll_scan only scans on primary advertising channels.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org