[mynewt-nimble] branch master updated: nimble/ll: Indicate proper transition from AUX_CONNECT_REQ

[andk]

This is an automated email from the ASF dual-hosted git repository.

andk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git


The following commit(s) were added to refs/heads/master by this push:
     new 496f458  nimble/ll: Indicate proper transition from AUX_CONNECT_REQ
496f458 is described below

commit 496f4583738fe68fa319be9bc5edadb9e01dc9a3
Author: Andrzej Kaczmarek <andrzej.kaczma...@codecoup.pl>
AuthorDate: Mon Aug 5 08:28:06 2019 +0200

    nimble/ll: Indicate proper transition from AUX_CONNECT_REQ
    
    nrf5x PHY works without this patch because it ignores this return value
    and configures transition speculatively on RX end, before processing
    received packet.
    
    With this fix such workaround is not necessary so we can fix nrf5x PHY.
---
 nimble/controller/src/ble_ll_adv.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/nimble/controller/src/ble_ll_adv.c 
b/nimble/controller/src/ble_ll_adv.c
index 86033a6..66e226d 100644
--- a/nimble/controller/src/ble_ll_adv.c
+++ b/nimble/controller/src/ble_ll_adv.c
@@ -4190,7 +4190,8 @@ ble_ll_adv_rx_isr_start(uint8_t pdu_type)
         /* Only accept connect requests if connectable advertising event */
         if (pdu_type == BLE_ADV_PDU_TYPE_CONNECT_IND) {
             if (advsm->props & BLE_HCI_LE_SET_EXT_ADV_PROP_CONNECTABLE) {
-                rc = 0;
+                /* Need transition to TX if extended adv */
+                rc = !(advsm->props & BLE_HCI_LE_SET_EXT_ADV_PROP_LEGACY);
             }
         }
     }