<p>pespin <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/25386">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">TA loop: Take into account UL SACCH 'Actual Timing advance' field<br><br>First step improving and reworking TA loop:<br>Move trigger of the loop to similar place done by BS/MS Power Control<br>Loop, that is, upon receivial of UL SACCH block, which contains<br>information about the TA used to transmit the block encode in L1SACCH<br>Header. Hence, from computed received TOA and TA used when transmitting<br>from the MS, we can infer the desired TA to be used by the MS, which<br>will send back to it later during DL SACCH block.<br><br>The values taken are actually the ones calculated for the previous SACCH<br>block and stored in lchan->meas.ms_toa256. That's because L1SACCH<br>contains the TA used for the previous reporting period, similarly to<br>what's specified for MS Power Control loop (TS 45.008 sec 4.2):<br>"""<br>The MS shall confirm the power control level that it is currently employing<br>in the SACCH L1 header on each uplink  channel. The indicated value shall<br>be the power control level actually used by the mobile for the last burst<br>of the  previous SACCH period.<br>"""<br>(The reader may observe that currently this is not properly done for MS<br>Power Control loop when calling lchan_ms_pwr_ctrl(): this is a bug.)<br><br>This new method also permits changing TA quicker, since we have more<br>confidence that the TA we request is aligned with the one used to<br>transmit, and we don't simply increment/decrement based on the value we<br>request to transmit.<br><br>Related: SYS#5371<br>Change-Id: I2d0f128c8dcac93ee382283a1c91fca76623b8fc<br>---<br>M include/osmo-bts/ta_control.h<br>M src/common/l1sap.c<br>M src/common/measurement.c<br>M src/common/ta_control.c<br>M tests/ta_control/ta_control_test.c<br>5 files changed, 74 insertions(+), 28 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/86/25386/2</pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/25386">change 25386</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-bts/+/25386"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I2d0f128c8dcac93ee382283a1c91fca76623b8fc </div>
<div style="display:none"> Gerrit-Change-Number: 25386 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>