Change in osmo-bts[2021q1]: sched_lchan_tch_x: use ul_cmr and ul_ft when generating RTP bad frame

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

osmith gerrit-no-reply at lists.osmocom.org
Fri Sep 24 10:06:02 UTC 2021


osmith has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bts/+/25572 )


Change subject: sched_lchan_tch_x: use ul_cmr and ul_ft when generating RTP bad frame
......................................................................

sched_lchan_tch_x: use ul_cmr and ul_ft when generating RTP bad frame

All generated RTP that originates from the BTS relates to uplink. When
generating AMR BAD frame RTP packets, we must use ul_cmr/ul_ft and not
dl_cmr/dl_ft.

Change-Id: Ifa009819791cf7df2dd8201f442b0dae06f622a4
Related: SYS#5549
---
M src/osmo-bts-trx/sched_lchan_tchf.c
M src/osmo-bts-trx/sched_lchan_tchh.c
2 files changed, 4 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/72/25572/1

diff --git a/src/osmo-bts-trx/sched_lchan_tchf.c b/src/osmo-bts-trx/sched_lchan_tchf.c
index 2202b48..698e4a6 100644
--- a/src/osmo-bts-trx/sched_lchan_tchf.c
+++ b/src/osmo-bts-trx/sched_lchan_tchf.c
@@ -282,8 +282,8 @@
 				break;
 			case GSM48_CMODE_SPEECH_AMR: /* AMR */
 				rc = osmo_amr_rtp_enc(tch_data,
-					chan_state->codec[chan_state->dl_cmr],
-					chan_state->codec[chan_state->dl_ft],
+					chan_state->codec[chan_state->ul_cmr],
+					chan_state->codec[chan_state->ul_ft],
 					AMR_BAD);
 				if (rc < 2) {
 					LOGL1S(DL1P, LOGL_ERROR, l1t, bi->tn, chan, bi->fn,
diff --git a/src/osmo-bts-trx/sched_lchan_tchh.c b/src/osmo-bts-trx/sched_lchan_tchh.c
index b0a9082..125d346 100644
--- a/src/osmo-bts-trx/sched_lchan_tchh.c
+++ b/src/osmo-bts-trx/sched_lchan_tchh.c
@@ -323,8 +323,8 @@
 				break;
 			case GSM48_CMODE_SPEECH_AMR: /* AMR */
 				rc = osmo_amr_rtp_enc(tch_data,
-					chan_state->codec[chan_state->dl_cmr],
-					chan_state->codec[chan_state->dl_ft],
+					chan_state->codec[chan_state->ul_cmr],
+					chan_state->codec[chan_state->ul_ft],
 					AMR_BAD);
 				if (rc < 2) {
 					LOGL1S(DL1P, LOGL_ERROR, l1t, bi->tn, chan, bi->fn,

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/25572
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: 2021q1
Gerrit-Change-Id: Ifa009819791cf7df2dd8201f442b0dae06f622a4
Gerrit-Change-Number: 25572
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210924/77b7b92a/attachment.htm>


More information about the gerrit-log mailing list