<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Are you sure it makes sense to have a specific function for IDLE/NOPE burst indications?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Well, we don't really need this feature for the most logical channels. It's only required for those which change their state. For example, both TCH/F and TCH/H handlers need to shift the burst buffer due to block-diagonal interleaving. We may also want to avoid involving the Viterbi decoding when at least one TCH burst is lost, while for xCCH we can still try (because its redundancy is much higher).</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">afaik we don't have that in other osmo-bts models...</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Of course. Other BTS models deal with bursts internally, so you only get L2 frames from them.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/15989">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-bts/+/15989/1/src/common/scheduler.c">File src/common/scheduler.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/osmo-bts/+/15989/1/src/common/scheduler.c@1363">Patch Set #1, Line 1363:</a> <code style="font-family:monospace,monospace">   /* omit bursts which have no handler, like IDLE bursts */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">In here it says it will return on IDLE bursts, but are adding code below this point to handle them.. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">'... like IDLE bursts' is just an example meaning that TRXC_IDLE has no Uplink burst handler. Both SCH and FCCH also have no Uplink handlers, neither they exist on the Uplink part of the multiframe layouts. This comment is a bit misleading.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/15989">change 15989</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/+/15989"/><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: I71c552f44c25e56e9779d8b8ef5d4de9f8475637 </div>
<div style="display:none"> Gerrit-Change-Number: 15989 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <axilirator@gmail.com> </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-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 06 Nov 2019 14:19:33 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>