laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/33339 )

Change subject: pcu_l1_if: use correct SAPI in PCUIF message PCU_IF_MSG_DATA_CNF_DT ......................................................................

pcu_l1_if: use correct SAPI in PCUIF message PCU_IF_MSG_DATA_CNF_DT

When we receive PCU_IF_MSG_DATA_CNF_DT, we check on PCU_IF_SAPI_PCH. This is formally not correct, we should check on PCU_IF_SAPI_PCH_DT instead.

(This patch will only affect osmo-bsc but not osmo-bts. The reason for this is that osmo-bts still uses the older PCUIF v.10)

Depends: osmo-bsc.git Id5c799e625c56e57f7b51cd4fb57f5bea9c973d2 Change-Id: I0883b51fc232ec0267f1511c3a37c0bcd0967a08 --- M src/pcu_l1_if.cpp 1 file changed, 18 insertions(+), 1 deletion(-)

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

diff --git a/src/pcu_l1_if.cpp b/src/pcu_l1_if.cpp index 7437e16..686853c 100644 --- a/src/pcu_l1_if.cpp +++ b/src/pcu_l1_if.cpp @@ -539,7 +539,7 @@ data_cnf_dt->sapi, data_cnf_dt->fn, current_fn);

switch (data_cnf_dt->sapi) { - case PCU_IF_SAPI_PCH: + case PCU_IF_SAPI_PCH_DT: bts_rcv_imm_ass_cnf(bts, NULL, data_cnf_dt->tlli, data_cnf_dt->fn); break; default: