nimble/ll: Refactor transitions #1974
Conversation
mkasenberg
force-pushed
the
transition
branch
from
ef8591c to
af12eb5
Compare
mkasenberg
force-pushed
the
transition
branch
from
af12eb5 to
679f8b4
Compare
mkasenberg
force-pushed
the
transition
branch
from
679f8b4 to
b4af225
Compare
| void ble_phy_tifs_set(uint16_t tifs); | ||
| #endif | ||
| /* Set T_ifs for the next transition */ | ||
| void ble_phy_tifs_set(uint16_t usecs, uint8_t anchor); |
There was a problem hiding this comment.
this is no longer needed (?)
nimble/drivers/nrf5x/src/ble_phy.c
Outdated
| void | ||
| ble_phy_set_rxend_cb(ble_phy_rx_end_func rxend_cb, void *arg) | ||
| { | ||
| /* Set transmit end callback and arg */ |
There was a problem hiding this comment.
'receive'
what is the purpose of this callback? it doesn't seem to be used anywhere
| * non-zero for end of PDU */ | ||
| void ble_phy_tifs_txtx_set(uint16_t usecs, uint8_t anchor); | ||
| /* Set direction of the next transition */ | ||
| void ble_phy_transition_set(uint8_t trans, uint8_t anchor, uint16_t usecs); |
There was a problem hiding this comment.
I wonder if we should perhaps merge trans and anchor to a single parameter
standard tifs transitions would be called as they are now, but add 2 transitions that anchor at the beginning of pdu, smth like BLE_PHY_TRANSITION_TO_TX_SUB -> sub is short of 'subevent' as this is where it will be used
@sjanc ^^
There was a problem hiding this comment.
yeah, but I'd call it subevent for clarity
nimble/drivers/nrf5x/src/ble_phy.c
Outdated
| } | ||
|
|
||
| void | ||
| ble_phy_fast_transition_enable(void) |
There was a problem hiding this comment.
what is fast transition?
mkasenberg
force-pushed
the
transition
branch
2 times, most recently
from
740973e to
d5999ac
Compare
New Bluetooth features require fast RX-to-RX and TX-to-TX transitions and use a different TIFS than 150us. Co-authored-by: Andrzej Kaczmarek <[email protected]>
mkasenberg
force-pushed
the
transition
branch
from
d5999ac to
7b860c6
Compare
|
#AutoPTS run mynewt nrf52 GATT/CL/GAW/BV-01-C GATT/CL/GAD/BV-01-C GATT/CL/GAR/BV-01-C GATT/CL/GAN/BV-01-C GATT/SR/GAN/BV-01-C |
|
Scheduled PR #1974 (comment), board: nrf52, estimated start time: 08:02:23, test case count: 5, estimated duration: 0:12:34 Test cases to be runGATT/CL/GAD/BV-01-CGATT/CL/GAR/BV-01-C GATT/CL/GAW/BV-01-C GATT/CL/GAN/BV-01-C GATT/SR/GAN/BV-01-C |
|
AutoPTS Bot results: Failed tests (1)GATT GATT/SR/GAN/BV-01-C INDCSVSuccessful tests (4)GATT GATT/CL/GAD/BV-01-C PASSGATT GATT/CL/GAN/BV-01-C PASS GATT GATT/CL/GAR/BV-01-C PASS GATT GATT/CL/GAW/BV-01-C PASS |
New Bluetooth features require fast RX-to-RX and TX-to-TX transitions and use a different TIFS than 150us.