<p>Pau Espin Pedrol has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/11083">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bts: Don't expect specific ms_power_lvl when running without trxcon<br><br>When we don't use trxcon (ie we run real HW) we cannot set some<br>parameters like ms power level, so we need to relax template matching<br>when we receive UL measurements in that case.<br><br>This commit makes tests TC_meas_res_sign_tchf and<br>TC_meas_res_sign_tchh_toa256 pass with motorola CXX and osmo-trx+osmo-bts-trx.<br><br>Change-Id: Icf1d2216d29c1ebf68c672e6ca06c54a7457304b<br>---<br>M bts/BTS_Tests.ttcn<br>1 file changed, 17 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/83/11083/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn</span><br><span>index aa402d1..a5992f7 100644</span><br><span>--- a/bts/BTS_Tests.ttcn</span><br><span>+++ b/bts/BTS_Tests.ttcn</span><br><span>@@ -1239,6 +1239,15 @@</span><br><span>  return ret;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+/* return a template expecting specific value only when using trxc */</span><br><span style="color: hsl(120, 100%, 40%);">+private function f_expect_with_trxc(integer val) return template integer {</span><br><span style="color: hsl(120, 100%, 40%);">+       var template integer ret := ?;</span><br><span style="color: hsl(120, 100%, 40%);">+        if (mp_bb_trxc_port != -1) {</span><br><span style="color: hsl(120, 100%, 40%);">+          ret := val;</span><br><span style="color: hsl(120, 100%, 40%);">+   }</span><br><span style="color: hsl(120, 100%, 40%);">+     return ret;</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> </span><br><span> /* build a template for matching measurement results against */</span><br><span> private function f_build_meas_res_tmpl() runs on ConnHdlr return template RSL_Message {</span><br><span>@@ -1258,7 +1267,7 @@</span><br><span>   if (l1p.toa256_enabled) {</span><br><span>            ul_meas.len := (3+8);</span><br><span>                ul_meas.supp_meas_info := {</span><br><span style="color: hsl(0, 100%, 40%);">-                     toa256_mean := l1p.timing_offset_256syms,</span><br><span style="color: hsl(120, 100%, 40%);">+                     toa256_mean := f_expect_with_trxc(l1p.timing_offset_256syms),</span><br><span>                        toa256_min := ?,</span><br><span>                     toa256_max := ?,</span><br><span>                     toa256_std_dev := ?</span><br><span>@@ -1276,13 +1285,16 @@</span><br><span>                power_level := l1p.bs_power_level</span><br><span>    };</span><br><span>   var template RSL_IE_L1Info l1_info := {</span><br><span style="color: hsl(0, 100%, 40%);">-         ms_power_lvl := l1p.ms_power_level,</span><br><span style="color: hsl(120, 100%, 40%);">+           ms_power_lvl := f_expect_with_trxc(l1p.ms_power_level),</span><br><span>              fpc := false,</span><br><span>                reserved := 0,</span><br><span style="color: hsl(0, 100%, 40%);">-          actual_ta := l1p.ms_actual_ta</span><br><span style="color: hsl(120, 100%, 40%);">+         actual_ta := f_expect_with_trxc(l1p.ms_actual_ta)</span><br><span>    };</span><br><span style="color: hsl(0, 100%, 40%);">-      var uint8_t offs := toffs256s_to_rsl(l1p.timing_offset_256syms);</span><br><span style="color: hsl(0, 100%, 40%);">-        var template uint8_t t_toffs := (offs-1 .. offs+1);      /* some tolerance */</span><br><span style="color: hsl(120, 100%, 40%);">+ var template uint8_t t_toffs := ?;</span><br><span style="color: hsl(120, 100%, 40%);">+    if (mp_bb_trxc_port != -1) {</span><br><span style="color: hsl(120, 100%, 40%);">+          var uint8_t offs := toffs256s_to_rsl(l1p.timing_offset_256syms);</span><br><span style="color: hsl(120, 100%, 40%);">+              t_toffs := (offs-1 .. offs+1);   /* some tolerance */</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span>    return tr_RSL_MEAS_RES_OSMO(g_chan_nr, g_next_meas_res_nr, ul_meas, bs_power, l1_info,</span><br><span>                                   ?, t_toffs);</span><br><span> }</span><br><span></span><br></pre><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: newchange </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: 1 </div>
<div style="display:none"> Gerrit-Owner: Pau Espin Pedrol <pespin@sysmocom.de> </div>