falconia submitted this change.

View Change


Approvals: 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 Jenkins Builder: Verified
rtp2trau_hr16: fix UFE bit setting

Per TS 48.061 section 5.1.4.1.1, UFE=1 means UL was good and UFE=0
means UL was bad. In the absence of logic that reports the actual
state of UL, we need to tell the BTS that UL was good (like we do
for FR & EFR) - hence set UFE=1.

Change-Id: Ibeb8db332f2e20806bebb077844a509064b2d89d
---
M src/trau/trau_rtp_conv.c
1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/trau/trau_rtp_conv.c b/src/trau/trau_rtp_conv.c
index 5c87c46..c70736d 100644
--- a/src/trau/trau_rtp_conv.c
+++ b/src/trau/trau_rtp_conv.c
@@ -407,7 +407,7 @@
tf->c_bits[15] = 0; /* C16: SP */
tf->c_bits[16] = 0; /* C17: DTXd shall not be applied */
} else {
- tf->c_bits[11] = 0; /* C12: UFE */
+ tf->c_bits[11] = 1; /* C12: UFE */
tf->c_bits[12] = 1; /* C13: spare */
tf->c_bits[13] = 1; /* C14: spare */
tf->c_bits[14] = 1; /* C15: spare */

To view, visit change 37148. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: Ibeb8db332f2e20806bebb077844a509064b2d89d
Gerrit-Change-Number: 37148
Gerrit-PatchSet: 1
Gerrit-Owner: falconia <falcon@freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: falconia <falcon@freecalypso.org>
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged