<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? afaik we don't have that in other osmo-bts models... I'd expect to have them sent over usual UL path and then for each specific logical channel do whatever based on it being IDLE or not (for instance lost TCH frames have len=0 but still are sent to increase the RTP clock).</p><p style="white-space: pre-wrap; word-wrap: break-word;">In any case I'd wait to merge this until there's someone using that new fn callback you are adding...</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 style="white-space: pre-wrap; word-wrap: break-word;">In here it says it will return on IDLE bursts, but are adding code below this point to handle them...</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-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 06 Nov 2019 13:33:22 +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>