<p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/15918">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/15918/1/src/common/l1sap.c">File src/common/l1sap.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/+/15918/1/src/common/l1sap.c@74">Patch Set #1, Line 74:</a> <code style="font-family:monospace,monospace">static bool tch_data_meas_present = true;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">AFAIU You don't need to do any switching, simply have an API bool bts_model_tch_data_meas_present()  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">we do already have the BTS_FEAT_* constants and the gsm_bts_set_feature() API.  That would be the most logical way for me to deal with this.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Adding a single call to gsm_bts_set_feature() in all existing backends is also certainly not requiring a full re-test with all hardware.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/15918/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/+/15918/1/src/common/scheduler.c@760">Patch Set #1, Line 760:</a> <code style="font-family:monospace,monospace">l1sap->u.tch.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I am also a bit confused about the is_sub struct member. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I believe I explained this elsewhere here in gerrit or in redmine.  is_sub must at the very least be set for all SACCH  frames, and is set on other frames depending on the codec that's being used.  It controls which frames count into RXLEV/QUAL "SUB".</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/15918/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/osmo-bts/+/15918/1/src/osmo-bts-trx/scheduler_trx.c@a205">Patch Set #1, Line 205:</a> <code style="font-family:monospace,monospace">/* FIXME: use actual values for BER etc */</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The problem here is that we can not use actual values for BER. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">the best would be if those values simply wouldn't count.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Let's say you normally expect 23 values every measurement interval.  Then you compute the average by dividing by 23.  But if only 20 valid measurements were received and 3 were in bad frames, then the average should simply be computed by dividing by 20, and not by 23.  At this point you don't need to "invent" any values.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/15918">change 15918</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/+/15918"/><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: I710d0b7cf193afa8515807836ee69b8b7db84a84 </div>
<div style="display:none"> Gerrit-Change-Number: 15918 </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: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 18 Dec 2019 12:12:21 +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: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>