<p>Neels Hofmeyr <strong>uploaded patch set #6</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/9284">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">use libosmocore's gsm0808_permitted_speech(), gsm0808_chosen_channel()<br><br>The guts of bssap_speech_from_lchan() and lchan_to_chosen_channel() have been<br>moved to libosmocore; call those instead.<br><br>The return value of bssap_speech_from_lchan() used to be -1 on error, now the<br>error value is 0. The only caller did not handle -1 properly, but fed it<br>directly to a uint8_t.<br><br>On gsm0808_chosen_channel() error, log the error. Proper handling is missing.<br><br>Fixing the error handling in send_ass_compl() is a separate issue: currently it<br>is limited to logging, there is no way to return an error yet, nor any actions<br>to take on error.<br><br>Depends: Icca23940791f97fa64dbc3f2734270b99f9550c1 (libosmocore)<br>Change-Id: Ib5c940a9dae11c5e26d4b47fa9d95fef889ad2f6<br>---<br>M src/osmo-bsc/bsc_subscr_conn_fsm.c<br>1 file changed, 7 insertions(+), 84 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/84/9284/6</pre><p>To view, visit <a href="https://gerrit.osmocom.org/9284">change 9284</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/9284"/><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: Ib5c940a9dae11c5e26d4b47fa9d95fef889ad2f6 </div>
<div style="display:none"> Gerrit-Change-Number: 9284 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: 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 </div>