<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;">LGTM.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I've noticed that osmo-bts-trx also has code to not change more than MS_{LOWER,RAISE}_MAX_DBM at once. Why was that omitted from the port?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Because the algo works slightly different. The one from loops.c was not receiving the currently used MS Power Level announced during the block, probably because the algo works on bursts instead of the entire block, so the content of the block is not available because it works at a lower layer. On the other hand, since the power_control.c one works on blocks received from lower layer, then the MS Power control being used by the MS matching the rx signal strength is available, so the algo can use that to know if the MS already reached the last announced power level.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So, instead of making sure it changes the power level at a rate acceptable for the MS, it simply changes to the calculated desired one and waits for the MS to reach the power level until applying corrections again based on received signal strength.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/16070">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/+/16070">change 16070</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/+/16070"/><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: I579967cc8bb69dc76a315c6c9d3a351f5961d92f </div>
<div style="display:none"> Gerrit-Change-Number: 16070 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 15 Nov 2019 14:48:47 +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>