<p>Patch set 4:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/26049">View Change</a></p><p>2 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/+/26049/4/src/common/measurement.c">File src/common/measurement.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/+/26049/4/src/common/measurement.c@957">Patch Set #4, Line 957:</a> <code style="font-family:monospace,monospace"> if (mr && mr->meas_valid == 0) /* 0 = valid */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I've seen this clarification several times already: mr->meas_valid == 0) /* 0 = valid */<br>Please add a static inline boor meas_is_valid(mr);</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/26049/4/src/common/power_control.c">File src/common/power_control.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/+/26049/4/src/common/power_control.c@338">Patch Set #4, Line 338:</a> <code style="font-family:monospace,monospace">      if (ctrl_interval_skip_block(params, state))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">not entirely relaed to this patch, but I just noticed that if we don'tn receive proper meas result, then we don't decrease the period counter, which is wrong imho. We should ideally keep decreasing it, and if it reaches 0 when meas is invalid, then keep it at 1 to make it trigger next time we recieve a MR.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/26049">change 26049</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/+/26049"/><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: I937117cf26fb718d57920382f6972390ad498c51 </div>
<div style="display:none"> Gerrit-Change-Number: 26049 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 02 Nov 2021 10:45:40 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>