falconia submitted this change.

View Change


Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved
TRAU<->RTP HRv1: fix SID frame definition

HRv1 SID frame consists of 33 bits of comfort noise parameters
followed by 79 bits of all-1s SID codeword. The RTP-encoded
representation of this construct in trau_rtp_conv.c was off
by one bit - fix it.

Change-Id: I4554a70f05bee428e0fb304868e25ee61ff375a1
---
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 c70736d..4a62cc0 100644
--- a/src/trau/trau_rtp_conv.c
+++ b/src/trau/trau_rtp_conv.c
@@ -145,7 +145,7 @@
FT_NO_DATA = 7,
};

-static const uint8_t rtp_hr_sid[14] = { 0x00, 0x00, 0x00, 0x00, 0x3f, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff };
+static const uint8_t rtp_hr_sid[14] = { 0x00, 0x00, 0x00, 0x00, 0x7f, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff };

/*! Generate the 14 bytes ETSI TS 101 318 RTP payload for HR from a decoded 16k TRAU frame.
* Note that thsi differs from the IETF RFC5993 format. However, as OsmoBTS implements

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

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I4554a70f05bee428e0fb304868e25ee61ff375a1
Gerrit-Change-Number: 37149
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