<p><a href="https://gerrit.osmocom.org/11083">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11083/5/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/5/bts/BTS_Tests.ttcn@1285">Patch Set #5, Line 1285:</a> <code style="font-family:monospace,monospace">          actual_ta := f_tolerance(l1p.ms_actual_ta, 0, 63, mp_tolerance_timing_offset_256syms/256)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">According to "3GPP TS 48.058", "9.3.10 L1 Information":<br>"""<br>Octets 2-3 contain the L1 header information of SACCH blocks. The information fields and codings are as defined in <br>3GPP TS 44.004. <br>"""</p><p style="white-space: pre-wrap; word-wrap: break-word;">Then. According to "3GPP TS 44.004", "6.1  Physical layer protocol fields and procedures:"</p><p style="white-space: pre-wrap; word-wrap: break-word;">"""<br>For all the bands except GSM 400, the values 0 to 63 are valid TA values.  The bit pattern "1111111" indicates that the <br>field does not contain a timing advance value. All other bit combinations (64 to 126 decimal) are reserved. For all bands <br>except GSM 400 bit 8 is set to spare.<br>"""</p><p style="white-space: pre-wrap; word-wrap: break-word;">So this one is good.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11083/5/bts/BTS_Tests.ttcn@1288">Patch Set #5, Line 1288:</a> <code style="font-family:monospace,monospace">   var template uint8_t t_toffs := f_tolerance(offs, -63, 192, mp_tolerance_timing_offset_256syms/256);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I think I swapped these values with those of line 1285, I need to have a look at these ranges again.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">According to "3GPP TS 48.058", "9.3.37 MS Timing Offset":<br>"""<br>The meaning of the MS Timing Offset is as defined in 3GPP TS 45.010. The value of MS Timing Offset is the binary <br>value of the 8-bit Timing Offset field (octet 2) - 63. The range of MS Timing Offset is therefore -63 to 192. <br>"""</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">And looking for "Timing Offset" in TS 45.010 I could only find this:<br>"""<br>MS timing offset:<br> delay of the received signal relative to the expect<br>ed signal from an MS at zero distance under static <br>channel conditions with zero timing advance. This is accurate to ± 1 symbol, and reported once per SACCH or after a <br>RACH as. required (i.e. at the same rate as timing advan<br>ce). For example, for an MS with a round trip propagation <br>delay of P symbols, but with a timing advance of T symbols, the reported timing offset will be P-T quantized to the <br>nearest symbol. For GPRS the MS timing offset is not reported. <br>"""</pre><p style="white-space: pre-wrap; word-wrap: break-word;">So, as a result:<br>line 1264 should have ranges (-63*256, 192*256).<br>line 1285 should have ranges (0,63) (correct in current version).<br>line 1288 should have ranges (0, 255).</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: 5 </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-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 28 Sep 2018 18:16:52 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>