fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/32516 )
Change subject: trxcon: ignore TRXCON_EV_TX_DATA_CNF in TRXCON_ST_PACKET_DATA ......................................................................
trxcon: ignore TRXCON_EV_TX_DATA_CNF in TRXCON_ST_PACKET_DATA
Change-Id: Ie3c53ffbac1f1906cc97ef66eb74e24aee2cb2c4 Related: OS#5500 --- M src/host/trxcon/src/trxcon_fsm.c M src/host/trxcon/src/trxcon_shim.c 2 files changed, 14 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/16/32516/1
diff --git a/src/host/trxcon/src/trxcon_fsm.c b/src/host/trxcon/src/trxcon_fsm.c index 219f0c0..598f2bd 100644 --- a/src/host/trxcon/src/trxcon_fsm.c +++ b/src/host/trxcon/src/trxcon_fsm.c @@ -614,6 +614,8 @@ l1sched_reset(trxcon->sched, false); /* TODO: switch to (not implemented) TRXCON_ST_DCH_TUNING? */ break; + case TRXCON_EV_TX_DATA_CNF: + break; default: OSMO_ASSERT(0); } @@ -711,7 +713,8 @@ | S(TRXCON_EV_GPRS_UL_TBF_CFG_REQ) | S(TRXCON_EV_GPRS_DL_TBF_CFG_REQ) | S(TRXCON_EV_GPRS_UL_BLOCK_REQ) - | S(TRXCON_EV_RX_DATA_IND), + | S(TRXCON_EV_RX_DATA_IND) + | S(TRXCON_EV_TX_DATA_CNF), .onenter = &trxcon_st_packet_data_onenter, .onleave = &trxcon_st_packet_data_onleave, .action = &trxcon_st_packet_data_action, diff --git a/src/host/trxcon/src/trxcon_shim.c b/src/host/trxcon/src/trxcon_shim.c index 4f56d85..8b42efe 100644 --- a/src/host/trxcon/src/trxcon_shim.c +++ b/src/host/trxcon/src/trxcon_shim.c @@ -110,7 +110,6 @@ 0, 0, true); }
- /* XXX: do not send for L1SCHED_DT_PACKET_DATA */ return osmo_fsm_inst_dispatch(trxcon->fi, TRXCON_EV_TX_DATA_CNF, &cnf); }