dexter has uploaded this change for review. ( 
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(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/39/33339/1

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:

--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/33339
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I0883b51fc232ec0267f1511c3a37c0bcd0967a08
Gerrit-Change-Number: 33339
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>
Gerrit-MessageType: newchange

Reply via email to