fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30319 )

Change subject: trxcon: trxcon_fsm: permit direct DCH/PDCH transitions ......................................................................

trxcon: trxcon_fsm: permit direct DCH/PDCH transitions

It may happen that the MS needs to go directly from a PDCH to a TCH or SDCCH, e.g. in case of a Mobile Terminated call. I don't know if the opposite transition is needed on practice, but let's simply allow direct transitions for both TRXCON_ST_{DEDICATED,PACKET_DATA}.

Change-Id: I1a854e4683f102c40f1c174a291b6dc638f49b5c Related: OS#5599 --- M src/host/trxcon/src/trxcon_fsm.c 1 file changed, 3 insertions(+), 1 deletion(-)

Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, approved pespin: Looks good to me, but someone else must approve

diff --git a/src/host/trxcon/src/trxcon_fsm.c b/src/host/trxcon/src/trxcon_fsm.c index 264ba20..47da0bd 100644 --- a/src/host/trxcon/src/trxcon_fsm.c +++ b/src/host/trxcon/src/trxcon_fsm.c @@ -611,7 +611,8 @@ .name = "DEDICATED", .out_state_mask = S(TRXCON_ST_RESET) | S(TRXCON_ST_FBSB_SEARCH) - | S(TRXCON_ST_DEDICATED), + | S(TRXCON_ST_DEDICATED) + | S(TRXCON_ST_PACKET_DATA), .in_event_mask = S(TRXCON_EV_DCH_REL_REQ) | S(TRXCON_EV_DCH_EST_REQ) | S(TRXCON_EV_TX_ACCESS_BURST_REQ) @@ -627,6 +628,7 @@ .name = "PACKET_DATA", .out_state_mask = S(TRXCON_ST_RESET) | S(TRXCON_ST_FBSB_SEARCH) + | S(TRXCON_ST_DEDICATED) | S(TRXCON_ST_PACKET_DATA), .in_event_mask = S(TRXCON_EV_DCH_REL_REQ) | S(TRXCON_EV_DCH_EST_REQ)