<p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15676">View Change</a></p><p>8 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15676/2/library/RLCMAC_Types.ttcn">File library/RLCMAC_Types.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/15676/2/library/RLCMAC_Types.ttcn@309">Patch Set #2, Line 309:</a> <code style="font-family:monospace,monospace">tr_RLCMAC_DUMMY_CTRL</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Cool! Very useful template.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15676/2/library/RLCMAC_Types.ttcn@315">Patch Set #2, Line 315:</a> <code style="font-family:monospace,monospace">usf := ?</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I know you may not need this in you test case scenarios, but please parametrize this field. It will definitely be needed soon.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15676/2/pcu/PCU_Tests_RAW.ttcn">File pcu/PCU_Tests_RAW.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/15676/2/pcu/PCU_Tests_RAW.ttcn@617">Patch Set #2, Line 617:</a> <code style="font-family:monospace,monospace">pcuif_</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">BTW: regarding this prefix, we're not dealing with the PCU interface directly anymore. Let's avoid it? Not a merge blocker though.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15676/2/pcu/PCU_Tests_RAW.ttcn@628">Patch Set #2, Line 628:</a> <code style="font-family:monospace,monospace">/* 16 UL blocks (0 .. 32 dB, step = 2 dB) */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Copy-pasted, please dd.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15676/2/pcu/PCU_Tests_RAW.ttcn@631">Patch Set #2, Line 631:</a> <code style="font-family:monospace,monospace">countdown := (num_blocks - i)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Keep in mind, this is a countdown. If you're sending only one block, it shall be set to 0 (no more blocks pending). With this code, OsmoPCU will always expect one more UL block.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15676/2/pcu/PCU_Tests_RAW.ttcn@658">Patch Set #2, Line 658:</a> <code style="font-family:monospace,monospace">f_rx_pcuif_rlcmac_dl_block_exp_dummy</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same comment as for 15675, do we really need to create functions for such basic things? IMHO, this decreases readability: one would have to jump between different helper definitions while reading the code.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15676/2/pcu/PCU_Tests_RAW.ttcn@1003">Patch Set #2, Line 1003:</a> <code style="font-family:monospace,monospace">1</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You may need to change this to 2 as soon as you fix the countdown calculation in f_tx_pcuif_rlcmac_ul_n_blocks().</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/15676/2/pcu/PCU_Tests_RAW.ttcn@1010">Patch Set #2, Line 1010:</a> <code style="font-family:monospace,monospace">f_tx_pcuif_rlcmac_ul_n_blocks</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">... and this call can also be removed then.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15676">change 15676</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-ttcn3-hacks/+/15676"/><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-Change-Id: I587413a7de7956daee3423057530e4052a55ba88 </div>
<div style="display:none"> Gerrit-Change-Number: 15676 </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: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 04 Oct 2019 19:55:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>