<p><a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24425">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-bsc/+/24425/5/src/osmo-bsc/meas_rep.c">File src/osmo-bsc/meas_rep.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-bsc/+/24425/5/src/osmo-bsc/meas_rep.c@37">Patch Set #5, Line 37:</a> <code style="font-family:monospace,monospace">         return rep->dl.full.rx_lev + (rep->bs_power * 2);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Shouldn't this be doe by the caller of get_field and not this getter itself? looks weird asking for  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">i specifically picked this place to apply the BS Power effect, because all of our code expects RXLEV to be the actual practical DL power; fixing the RXLEV value here makes sure that all of the handover and congestion resolution code everywhere uses the value adjusted for practical relevance. We could apply the BS Power to all of the callers separately, but IMHO that would be a bad choice.</p><p style="white-space: pre-wrap; word-wrap: break-word;">We could rename the enum values to MEAS_REP_DL_RXLEV_AND_BS_POWER_FULL and *_SUB?<br>or introduce a separate MEAS_REP_DL_RXLEV_RAW (which then no code would use, ever).</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bsc/+/24425">change 24425</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-bsc/+/24425"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ifcf59964b5e2d550d79e4ba14d90962808f79dae </div>
<div style="display:none"> Gerrit-Change-Number: 24425 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@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-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 27 May 2021 14:35:32 +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>