<p>dexter <strong>uploaded patch set #9</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/11060">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">codec_pref: handle S0-S15 in ASSIGNMENT REQUEST<br><br>Opposed to all other codecs that are common in GSM, AMR requires a codec<br>configuration that is expressed by a bitmask (S0 to S15) in the speech<br>codec list in the ASSIGNMENT REQUEST. Also the BSC acknowledges those<br>configuration in the ASSIGNMENT COMPLETE message.<br><br>At the moment osmo-bsc ignores all incoming configuration bits. The bits<br>in the ASSIGNMENT COMPLETE speech codec (choosen) field are hardcoded.<br><br>- Store the configuration bits while parsing the ASSIGNMENT COMPLETE<br>- Create an intersection with the configuration that is actually<br>  supported by the BSS<br>- Return the resulting (chosen) configuration bits with the assignment<br>  complete message.<br>- Use the (highest of the) agreed codec rates in RSL channel activation.<br><br>Change-Id: I2d8ded51b3eb4c003fe2da6f2d6f48d001b73737<br>Related: OS#3529<br>---<br>M include/osmocom/bsc/codec_pref.h<br>M include/osmocom/bsc/gsm_data.h<br>M include/osmocom/bsc/lchan_fsm.h<br>M src/osmo-bsc/assignment_fsm.c<br>M src/osmo-bsc/codec_pref.c<br>M src/osmo-bsc/handover_fsm.c<br>M src/osmo-bsc/lchan_fsm.c<br>M src/osmo-bsc/osmo_bsc_bssap.c<br>M tests/codec_pref/codec_pref_test.c<br>M tests/codec_pref/codec_pref_test.ok<br>10 files changed, 190 insertions(+), 101 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/60/11060/9</pre><p>To view, visit <a href="https://gerrit.osmocom.org/11060">change 11060</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/11060"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I2d8ded51b3eb4c003fe2da6f2d6f48d001b73737 </div>
<div style="display:none"> Gerrit-Change-Number: 11060 </div>
<div style="display:none"> Gerrit-PatchSet: 9 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Assignee: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>