Change in osmo-bsc[master]: SCCPlite Assignment Complete: include Speech Codec (Chosen)

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

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Thu Sep 6 14:35:48 UTC 2018


Neels Hofmeyr has uploaded this change for review. ( https://gerrit.osmocom.org/10810


Change subject: SCCPlite Assignment Complete: include Speech Codec (Chosen)
......................................................................

SCCPlite Assignment Complete: include Speech Codec (Chosen)

Compose the Speech Codec (Chosen) IE not only for AoIP, but also for SCCPlite:
place the code that assigns the codec to sc_ptr outside of the gscon_is_aoip()
if scope.

This way the MSC is told the chosen speech codec, which is mandatory for IP
based user plane, and was missing until now.

Related: OS#3528
Change-Id: Ibedade8d71a7994d25a63bc2faa2a24a10bfffa1
---
M src/osmo-bsc/assignment_fsm.c
1 file changed, 4 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/10/10810/1

diff --git a/src/osmo-bsc/assignment_fsm.c b/src/osmo-bsc/assignment_fsm.c
index e1da45b..63776f0 100644
--- a/src/osmo-bsc/assignment_fsm.c
+++ b/src/osmo-bsc/assignment_fsm.c
@@ -163,12 +163,12 @@
 				return;
 			}
 			addr_local_p = &addr_local;
-
-			/* Extrapolate speech codec from speech mode */
-			gsm0808_speech_codec_from_chan_type(&sc, perm_spch);
-			sc_ptr = ≻
 		}
+
+		/* Extrapolate speech codec from speech mode */
 		/* FIXME: AMR codec configuration must be derived from lchan1! */
+		gsm0808_speech_codec_from_chan_type(&sc, perm_spch);
+		sc_ptr = ≻
 	}
 
 	resp = gsm0808_create_ass_compl(lchan->abis_ip.ass_compl.rr_cause,

-- 
To view, visit https://gerrit.osmocom.org/10810
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibedade8d71a7994d25a63bc2faa2a24a10bfffa1
Gerrit-Change-Number: 10810
Gerrit-PatchSet: 1
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20180906/65cfdc1b/attachment.htm>


More information about the gerrit-log mailing list