<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 6:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I hope I have found a better way now to transfer the sysinfo to the PCU. Actually there is a SAPI 4 in the PCUIF protocol through which we already send SI13. This can be extended to transfer SI1 and SI3 as well.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Nice! Would be funny it we discovered this after merging your initial approach ;) If you remove the first (redundant?) octet in DATA.ind messages, there would be no need to bump PCUIF version at all.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369/6/pcu/PCU_Tests.ttcn">File pcu/PCU_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/osmo-ttcn3-hacks/+/22369/6/pcu/PCU_Tests.ttcn@3589">Patch Set #6, Line 3589:</a> <code style="font-family:monospace,monospace">'01'O</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">What is the purpose of this octet? I guess to indicate the type of SI?<br>We don't need it because it's contained in the first octet of the SI itself:</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">  '55'O - SI1,<br>  '49'O - SI3,<br>  '01'O - SI13.</pre></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/22369">change 22369</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/+/22369"/><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: I9054ab0e969c0fbfdc671c92d44cc61360959adc </div>
<div style="display:none"> Gerrit-Change-Number: 22369 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sat, 30 Jan 2021 01:09:43 +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>