<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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><p style="white-space: pre-wrap; word-wrap: break-word;">For sure we are. We are always requesting a given MS Power level to the MS. MS doesn't necessarily need to support that power level afair, it should then use the one immediately below (above in dBm) which is supported IIRC.</p><p style="white-space: pre-wrap; word-wrap: break-word;">See my previous comment, I believe the fix should be using:<br>> state->current, current_dbm</p><p style="white-space: pre-wrap; word-wrap: break-word;">Before the "ignore" flag was added, this was not needed because well, the value used in current code and the changes I propose were basically the same.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Please give it a try and submit the fix if it improves the situation.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Another topic to improve would be the "curr %d" part of the message, which prints the currently used Power level by the MS on that period (Announced MS Power Level). "curr" is a bit vague here, specially since it conflicts with our naming "current" we use in variable name for the "currently requested MS Power level". Maybe "announced %d" would be clearer there too, feel free to submit an extra patch with that fix.</p><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: Wed, 06 Oct 2021 14:36:42 +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>