<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 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">This approach looks wrong to me, see my comments to https://gerrit.osmocom.org/c/osmo-bts/+/20840. Even if it makes the test cases pass, let's better discuss everything first.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">In this case I think the approach is correct. For #20840 I am also not sure either. For this my Idea is that the FACCH replaces one TCH frame, with the data indication also the measurement is passed up. The BFI that is passed up at the same time should not contain a copy of the current measurement value. It should just not count. The problem is that the excess amount of measurement values generates incorrect measurement results, especially for SUB frames. (TC_meas_res_speech_tchf currently passes but that I think is because the measurements contain all the same results because the fake RSSI is constant throught the test, but in the log you will see that it looks ugly, measurement.c complains.)</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20841">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20841">change 20841</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-bts/+/20841"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie3f2e620ba2a2ab2fecdbae627ef01c6128fce0b </div>
<div style="display:none"> Gerrit-Change-Number: 20841 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </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: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 23 Oct 2020 14:31:53 +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>