[PATCH] osmo-bts[master]: DTX DL: split ONSET state handling

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

Max gerrit-no-reply at lists.osmocom.org
Fri Oct 21 16:43:50 UTC 2016


Review at  https://gerrit.osmocom.org/1145

DTX DL: split ONSET state handling

Handling ONSET cause by Voice and FACCH separately. In case of Voice we
have RTP payload which we have to cache and send in next response to L1
while FACCH do not have any payload which have to be cached.

Change-Id: Idba14dcd0cb12cd7aee86391fcc152c49fcd7052
Related: OS#1801
---
M src/osmo-bts-litecell15/tch.c
1 file changed, 7 insertions(+), 1 deletion(-)


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

diff --git a/src/osmo-bts-litecell15/tch.c b/src/osmo-bts-litecell15/tch.c
index b251388..c23ef0d 100644
--- a/src/osmo-bts-litecell15/tch.c
+++ b/src/osmo-bts-litecell15/tch.c
@@ -284,9 +284,15 @@
 
 		/* DTX DL-specific logic below: */
 		switch (lchan->tch.dtx.dl_amr_fsm->state) {
-		case ST_ONSET_V:
 		case ST_ONSET_F:
 			*payload_type = GsmL1_TchPlType_Amr_Onset;
+			/* Next SID after FACCH ONSET is always FIRST */
+			lchan->tch.dtx.is_update = false;
+			*len = 1;
+			/* There is nothing to cache for FACCH ONSET */
+			return 0;
+		case ST_ONSET_V:
+			*payload_type = GsmL1_TchPlType_Amr_Onset;
 			dtx_cache_payload(lchan, rtp_pl, rtp_pl_len, fn, 0);
 			*len = 1;
 			return 1;

-- 
To view, visit https://gerrit.osmocom.org/1145
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idba14dcd0cb12cd7aee86391fcc152c49fcd7052
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>



More information about the gerrit-log mailing list