falconia has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/37148?usp=email )

Change subject: rtp2trau_hr16: fix UFE bit setting ......................................................................

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(-)

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

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 */