<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +2</span></p><p><a href="https://gerrit.osmocom.org/10774">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/10774/1/src/osmo-bsc/assignment_fsm.c">File src/osmo-bsc/assignment_fsm.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/10774/1/src/osmo-bsc/assignment_fsm.c@171">Patch Set #1, Line 171:</a> <code style="font-family:monospace,monospace">          /* FIXME: AMR codec configuration must be derived from lchan1! */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks for spotting this!</p><p style="white-space: pre-wrap; word-wrap: break-word;">The below call was actually without effect, because it did not assign sc_ptr.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But we also have chosen a speech codec for SCCPlite connections. I'm pretty sure above gsm0808_speech_codec_from_chan_type() should be here, outside the "if AoIP". It means that so far we fail to send the chosen speech codec back to the MSC for SCCPlite connections. It is mandatory for IP based user plane, which we use, so definitely needs to be fixed. See 3GPP TS 48.008 3.2.1.2 ASSIGNMENT COMPLETE. https://osmocom.org/issues/3528</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10774">change 10774</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/10774"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I34d9281944b36cd89ad8e1c5774f0ea80fdfadc8 </div>
<div style="display:none"> Gerrit-Change-Number: 10774 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 06 Sep 2018 11:34:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>