<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: Code-Review+1</p><p style="white-space: pre-wrap; word-wrap: break-word;">(1 comment)</p><p style="white-space: pre-wrap; word-wrap: break-word;">It seems to me that this case was implemented looking at the current behaviour of OsmoPCU, which may be incorrect (remember the situation with PTCCH/U, when everybody wrongly expected data blocks on it). Could you please add the spec. references where the process of Downlink TBF establishment is described? At least where they state that Immediate Assignment (assigning a Downlink TBF) shall be sent on PCH (and not on AGCH). The code itself looks good to me.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It was indeed done looking at the current behaviour of OsmoPCU. Whether it is correct behavior or not according to specs can be checked later, but at least we have some code to test current behavior of osmo-pcu in here which is useful for now, and I need to move onto other stuff now. </p><p><a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15681">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/15681">change 15681</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/+/15681"/><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: I3be5123ff5294e5851652ec14d54589442082b28 </div>
<div style="display:none"> Gerrit-Change-Number: 15681 </div>
<div style="display:none"> Gerrit-PatchSet: 6 </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: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 09 Oct 2019 15:32:51 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>