[S] Change in libosmo-abis[master]: TRAU<->RTP HRv1: fix SID frame definition

falconia has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/37149?usp=email ) Change subject: TRAU<->RTP HRv1: fix SID frame definition ...................................................................... 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(-) 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 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 https://gerrit.osmocom.org/c/libosmo-abis/+/37149?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Change-Id: I4554a70f05bee428e0fb304868e25ee61ff375a1 Gerrit-Change-Number: 37149 Gerrit-PatchSet: 1 Gerrit-Owner: falconia &lt;falcon(a)freecalypso.org&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: falconia &lt;falcon(a)freecalypso.org&gt; Gerrit-Reviewer: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged