Change in osmo-bsc[master]: lchan_fsm: do not include 12.2k in active set on HR channels

dexter gerrit-no-reply at
Fri Mar 1 17:06:27 UTC 2019

dexter has uploaded this change for review. (

Change subject: lchan_fsm: do not include 12.2k in active set on HR channels

lchan_fsm: do not include 12.2k in active set on HR channels

When "Config-NB-Code = 1" is set via the S-bits, then the resulting
multirate configuration IE will contain 12.2K. Since the generator
function is not aware if the lchan is activated for HR or FR it sets the
flag for 12.2k always.

We have to add a check here in order to set the 12.2k flag in the IE
back to 0 so that the active set contains a set of useable rates.

Change-Id: I40e7f568f4822040a2d1e78f22dbba0e49d0167e
Related: SYS#4470
M src/osmo-bsc/lchan_fsm.c
1 file changed, 9 insertions(+), 0 deletions(-)

  git pull ssh:// refs/changes/95/13095/1

diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c
index 0c54efb..7cbba44 100644
--- a/src/osmo-bsc/lchan_fsm.c
+++ b/src/osmo-bsc/lchan_fsm.c
@@ -545,6 +545,15 @@
+		/* Do not include 12.2 kbps rate when S1 is set. */
+		if (lchan->type == GSM_LCHAN_TCH_H && (info->s15_s0 & GSM0808_SC_CFG_AMR_4_75_5_90_7_40_12_20)) {
+			/* See also 3GPP TS 28.062, chapter "In case this Configuration
+			 * "Config-NB-Code = 1" is signalled in the TFO Negotiation for the HR_AMR
+			 * Codec Type,then it shall be assumed that AMR mode 12.2 kbps is (of course)
+			 * not included. */
+			mr_conf.m12_2 = 0;
+		}
 		if (lchan_mr_config(lchan, &mr_conf) < 0) {
 			lchan_fail("Can not generate multirate configuration IE\n");

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I40e7f568f4822040a2d1e78f22dbba0e49d0167e
Gerrit-Change-Number: 13095
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier at>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list