<p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/11083">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11083/2/bts/BTS_Tests.ttcn">File bts/BTS_Tests.ttcn:</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/11083/2/bts/BTS_Tests.ttcn@58">Patch Set #2, Line 58:</a> <code style="font-family:monospace,monospace">mp_tolerance_ms_power_level</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">tolerance doesn't make sense here. the MS power level is instructed by the BTS to the MS, and the MS must transmit at the ordered power level back in uplink.  There is no measurement involved.  It is a digital value that must match.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11083/2/bts/BTS_Tests.ttcn@59">Patch Set #2, Line 59:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mp_tolerance_ms_actual_ta := 0;<br> integer mp_tolerance_timing_offset_256syms<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">those two can be merged.  As both the TA as well as the TOA256 are derived from the same timing measurement, there is only one source of determining timing distortion/error in the path.  The 256syms value is of course 256 times the tolerance on the TA scale, which counts (if i remember correctly) in full symbols.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11083/2/bts/BTS_Tests.ttcn@62">Patch Set #2, Line 62:</a> <code style="font-family:monospace,monospace">mp_ms_power_level_exp</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">the expected power level must always match the instructed power level, otherwise the BTS or MS are broken in some way. Making this value configurable doesn't appear to make much sense, IMHO.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11083">change 11083</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/11083"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ttcn3-hacks </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Icf1d2216d29c1ebf68c672e6ca06c54a7457304b </div>
<div style="display:none"> Gerrit-Change-Number: 11083 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@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-Comment-Date: Wed, 26 Sep 2018 17:42:13 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>