<p>Neels Hofmeyr <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/11667">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">lchan release: always Deact SACCH<br><br>If an lchan is being released and had a SACCH active, there is no reason to<br>omit the Deact SACCH message ever. All of the callers that passed<br>do_deact_sacch = false did so for no good reason.<br><br>Drop the do_deact_sacch flag everywhere and, when the lchan type matches and<br>SAPI[0] is still active, simply always send a Deact SACCH message.<br><br>The do_deact_sacch flag was carried over from legacy code, by me, mainly<br>because I never really understood why it was there. I do hope I'm correct now,<br>asserting that having this flag makes no sense.<br><br>Change-Id: Id3301df059582da2377ef82feae554e94fa42035<br>---<br>M include/osmocom/bsc/bsc_subscr_conn_fsm.h<br>M include/osmocom/bsc/gsm_data.h<br>M include/osmocom/bsc/lchan_fsm.h<br>M src/osmo-bsc/abis_rsl.c<br>M src/osmo-bsc/assignment_fsm.c<br>M src/osmo-bsc/bsc_subscr_conn_fsm.c<br>M src/osmo-bsc/bsc_vty.c<br>M src/osmo-bsc/gsm_04_08_rr.c<br>M src/osmo-bsc/gsm_08_08.c<br>M src/osmo-bsc/handover_fsm.c<br>M src/osmo-bsc/lchan_fsm.c<br>M tests/gsm0408/gsm0408_test.c<br>M tests/handover/handover_test.c<br>13 files changed, 32 insertions(+), 32 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/67/11667/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/11667">change 11667</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/11667"/><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: Id3301df059582da2377ef82feae554e94fa42035 </div>
<div style="display:none"> Gerrit-Change-Number: 11667 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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 (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>