<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">That's not correct. We want to print the power level we request, not the one the MS was using. You already have the information you want in ms_power_lvl.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">OK, I don't see how it makes sense to print the power level we "request", when we are not actually "requesting" anything.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I guess you see a difference there because it falls into the ignore=true case there. I wanted to print "Ignoring" instead of "Keeping" in that case, but fixeria voted against it because he said it was confusing. If "Ignoring" is printed, then the other values make sense imho.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;"><br>"Keeping" is a problem, you just can't say "Keeping 3 == 5"</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>If you still don't want to print "ignoring" in that case, then, the 2 params you are changing should be set to instead:<br>"""<br>state->current, current_dbm<br>"""</p></blockquote><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/25694">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/25694">change 25694</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/+/25694"/><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: I002b708cfda60371f57e32617e7ee33cb41c42f7 </div>
<div style="display:none"> Gerrit-Change-Number: 25694 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: keith <keith@rhizomatica.org> </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-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 05 Oct 2021 20:20:23 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>