<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25607">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  laforge: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Fix MEAS parsing, as Ericsson RBS reports TA shifted by 2 bits.<br><br>This fixes call setup issues when more than ~1km from the tower.<br><br>NOTE: We use the last reported TA from the UE in the CHANnel ACTIVation.<br>When the UE is more than 1km from the tower, (unshifted) TA in the<br>measurement report can be 8 or greater. Once we send TA of 8 in the<br>CHAN ACTIV message, the lchan is unrecoverable.<br><br>Change-Id: I1c9bd5bf2fd126e62bcbec419f3499d2e0465559<br>---<br>M src/osmo-bsc/abis_rsl.c<br>1 file changed, 3 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo-bsc/abis_rsl.c b/src/osmo-bsc/abis_rsl.c</span><br><span>index c7399eb..fb8e386 100644</span><br><span>--- a/src/osmo-bsc/abis_rsl.c</span><br><span>+++ b/src/osmo-bsc/abis_rsl.c</span><br><span>@@ -1319,9 +1319,10 @@</span><br><span>                if (val[0] & 0x04)</span><br><span>                       mr->flags |= MEAS_REP_F_FPC;</span><br><span>              mr->ms_l1.ta = val[1];</span><br><span style="color: hsl(0, 100%, 40%);">-               /* BS11 and Nokia reports TA shifted by 2 bits */</span><br><span style="color: hsl(120, 100%, 40%);">+             /* BS11, Nokia and RBS report TA shifted by 2 bits */</span><br><span>                if (msg->lchan->ts->trx->bts->type == GSM_BTS_TYPE_BS11</span><br><span style="color: hsl(0, 100%, 40%);">-           || msg->lchan->ts->trx->bts->type == GSM_BTS_TYPE_NOKIA_SITE)</span><br><span style="color: hsl(120, 100%, 40%);">+          || msg->lchan->ts->trx->bts->type == GSM_BTS_TYPE_NOKIA_SITE</span><br><span style="color: hsl(120, 100%, 40%);">+           || msg->lchan->ts->trx->bts->type == GSM_BTS_TYPE_RBS2000)</span><br><span>                   mr->ms_l1.ta >>= 2;</span><br><span>                 /* store TA for handover decision, and for intra-cell re-assignment */</span><br><span>               mr->lchan->last_ta = mr->ms_l1.ta;</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/25607">change 25607</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/c/osmo-bsc/+/25607"/><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-Change-Id: I1c9bd5bf2fd126e62bcbec419f3499d2e0465559 </div>
<div style="display:none"> Gerrit-Change-Number: 25607 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>