<p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/20724">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/+/20724/2/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/+/20724/2/src/common/power_control.c@146">Patch Set #2, Line 146:</a> <code style="font-family:monospace,monospace">      LOGPLCHAN(lchan, DLOOP, LOGL_DEBUG,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I don't see why is this leally needed given that both values are logged immediatelly afterwards in t […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">As far as I can see, what's logged below is the old Uplink power level and the new Uplink power level (if it changes), but not the difference between the expected vs actual signal strength. The signal levels are logged separately, but not the diff. Am I missing something?</p><p style="white-space: pre-wrap; word-wrap: break-word;">In general, I agree that this logging statement might be redundant given that we add another one in the next patch. However, I still find it quite useful for debugging.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/20724">change 20724</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/+/20724"/><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: If9188c2b7dd7ad47fb22ef471e8a9a1ba74629f5 </div>
<div style="display:none"> Gerrit-Change-Number: 20724 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 19 Oct 2020 09:43:40 +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>