You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by an...@apache.org on 2021/09/30 11:06:06 UTC
[mynewt-nimble] branch master updated: nimble/ll: Fix TX time
restrictions during phy transition
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 08e6972 nimble/ll: Fix TX time restrictions during phy transition
08e6972 is described below
commit 08e69722d9b427c05400f4cb0483a02a0e63ce20
Author: Andrzej Kaczmarek <an...@codecoup.pl>
AuthorDate: Wed Sep 29 23:44:37 2021 +0200
nimble/ll: Fix TX time restrictions during phy transition
We do not need to restrict TX time after sending LL_PHY_REQ as a master,
this only applies to slave.
This makes LL/CON/CEN/BV-53-C pass instead of being inconclusive due to
data fragmentation done by IUT between LL_PHY_REQ and LL_PHU_UPDATE_IND.
Ref: Core 5.x, Vol. 6, Part B, section 5.1.10.1
---
nimble/controller/src/ble_ll_ctrl.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/nimble/controller/src/ble_ll_ctrl.c b/nimble/controller/src/ble_ll_ctrl.c
index c4ac650..384a487 100644
--- a/nimble/controller/src/ble_ll_ctrl.c
+++ b/nimble/controller/src/ble_ll_ctrl.c
@@ -2858,8 +2858,11 @@ ble_ll_ctrl_tx_done(struct os_mbuf *txpdu, struct ble_ll_conn_sm *connsm)
#endif
#if (BLE_LL_BT5_PHY_SUPPORTED == 1)
case BLE_LL_CTRL_PHY_REQ:
- connsm->phy_tx_transition =
- ble_ll_ctrl_phy_tx_transition_get(connsm->phy_data.req_pref_tx_phys_mask);
+ if (connsm->conn_role == BLE_LL_CONN_ROLE_SLAVE) {
+ connsm->phy_tx_transition =
+ ble_ll_ctrl_phy_tx_transition_get(
+ connsm->phy_data.req_pref_tx_phys_mask);
+ }
break;
case BLE_LL_CTRL_PHY_UPDATE_IND:
connsm->phy_tx_transition =