fixeria submitted this change.

View Change


Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
osmo_ortp: register a PayloadType for RTP_PT_CSDATA

Add a PayloadType definition for payload type 120 as per RFC 4040.

Change-Id: I48e91d2c4e9124ee1b7eb88e26af24603aa69f25
Related: OS#1572
---
M src/trau/osmo_ortp.c
1 file changed, 24 insertions(+), 0 deletions(-)

diff --git a/src/trau/osmo_ortp.c b/src/trau/osmo_ortp.c
index b681888..015ac34 100644
--- a/src/trau/osmo_ortp.c
+++ b/src/trau/osmo_ortp.c
@@ -39,6 +39,7 @@

#include "config.h"

+static PayloadType *payload_type_clearmode;
static PayloadType *payload_type_efr;
static PayloadType *payload_type_hr;
static RtpProfile *osmo_pt_profile;
@@ -266,6 +267,16 @@
{
PayloadType *pt;

+ /* CLEARMODE as per RFC 4040 (chapter 3) */
+ pt = payload_type_new();
+ pt->type = PAYLOAD_OTHER;
+ pt->clock_rate = 8000;
+ pt->bits_per_sample = 8;
+ pt->mime_type = "CLEARMODE";
+ pt->normal_bitrate = 64000;
+ pt->channels = 1;
+ payload_type_clearmode = pt;
+
/* EFR */
pt = payload_type_new();
pt->type = PAYLOAD_AUDIO_PACKETIZED;
@@ -290,6 +301,7 @@
/* add the GSM specific payload types. They are all dynamically
* assigned, but in the Osmocom GSM system we have allocated
* them as follows: */
+ rtp_profile_set_payload(osmo_pt_profile, RTP_PT_CSDATA, payload_type_clearmode);
rtp_profile_set_payload(osmo_pt_profile, RTP_PT_GSM_EFR, payload_type_efr);
rtp_profile_set_payload(osmo_pt_profile, RTP_PT_GSM_HALF, payload_type_hr);
rtp_profile_set_payload(osmo_pt_profile, RTP_PT_AMR, &payload_type_amr);

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

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I48e91d2c4e9124ee1b7eb88e26af24603aa69f25
Gerrit-Change-Number: 34223
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged