<p style="white-space: pre-wrap; word-wrap: break-word;">The first *first_fn pointer points to chan_state->ul_first_fn. I was assuming that this is per channel and that the fn of the first received burst is recoreded and so it sounded logical to to me that this is the FN of the complete block that is then handed upwards to the higher layers, even when the block is spread over non consecutive bursts. We still have the fn where the block has begun.</p><p style="white-space: pre-wrap; word-wrap: break-word;">What I find confusing is that at the moment we use different fn for the measurement indications and for the TCH/DATA indications. To me this sounds incorrect. From my understanding, the measurement indication should have the same fn as the TCH/DATA indication.</p><p style="white-space: pre-wrap; word-wrap: break-word;">At the moment I am trying to merge TCH/DATA and measurement inidcation so that they are a single object that is handed upwards. It would be important to know that if we we would end up with two different types of FN in that unified indication or not. Assuming everything is calculated correctly, would end up with a single FN?</p><p style="white-space: pre-wrap; word-wrap: break-word;">We need to solve this somehow, otherwise it will not be possible to merge TCH/DATA indication with the measurement indication cleanly.</p><p><a href="https://gerrit.osmocom.org/12779">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/12779/1/src/osmo-bts-trx/scheduler_trx.c">File src/osmo-bts-trx/scheduler_trx.c:</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/12779/1/src/osmo-bts-trx/scheduler_trx.c@1306">Patch Set #1, Line 1306:</a> <code style="font-family:monospace,monospace">printf</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Left from debugging? ;)</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Done</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12779">change 12779</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/12779"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ia27254bbf6e36426f7890ece6154dcd395673f63 </div>
<div style="display:none"> Gerrit-Change-Number: 12779 </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 (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 04 Feb 2019 10:04:47 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>