<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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">tolerance doesn't make sense here. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">If I recall correctly, what I read is that the value should match only if the MS supports it, and if the MS doesn't support it, then it should take a lower one the closest to the announced max value. I'll cross-check that.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think I also saw at least once that the motorola phone announced a different power level (0?). I'll do some more tests and see how it behaves more in detail.</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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">those two can be merged. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">From what I understand from your comment, then it doesn't matter which of the 2 values/variables we use since they have the same precision (despite one being in jumps of 256?). Or did I understand it incorrectly and the 256syms one has more precision?</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><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">the expected power level must always match the instructed power level, otherwise the BTS or MS are b […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">As I said, I'll test it a bit more and come back with some descirption or change.</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-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 26 Sep 2018 18:01:39 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>