pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/16439 )

Change subject: power_control.c: Log maximum allowed MS Power Level
......................................................................

power_control.c: Log maximum allowed MS Power Level

Change-Id: I983ff824ef6f54f1e800819d622158d5e2a51f04
---
M src/common/power_control.c
1 file changed, 10 insertions(+), 6 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/common/power_control.c b/src/common/power_control.c
index d2a6c1d..f6138d6 100644
--- a/src/common/power_control.c
+++ b/src/common/power_control.c
@@ -104,16 +104,20 @@

        if (lchan->ms_power_ctrl.current == new_power) {
                LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "Keeping MS power at control 
level %d, %d dBm "
-                         "(rx-ms-pwr-lvl %" PRIu8 ", rx-current %d dBm, 
rx-target %d dBm)\n",
-                       new_power, ms_pwr_dbm(band, new_power), ms_power, 
rxLevel, bts->ul_power_target);
+                         "(rx-ms-pwr-lvl %" PRIu8 ", max-ms-pwr-lvl %" PRIu8 
", rx-current %d dBm, rx-target %d dBm)\n",
+                         new_power, ms_pwr_dbm(band, new_power),
+                         ms_power, lchan->ms_power_ctrl.max,
+                         rxLevel, bts->ul_power_target);
                return 0;
        }

        LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "%s MS power from control level %d 
(%d dBm) to %d, %d dBm "
-                 "(rx-ms-pwr-lvl %" PRIu8 ", rx-current %d dBm, rx-target %d 
dBm)\n",
-               (diff > 0) ? "Raising" : "Lowering",
-               lchan->ms_power_ctrl.current, ms_pwr_dbm(band, 
lchan->ms_power_ctrl.current),
-               new_power, ms_pwr_dbm(band, new_power), ms_power, rxLevel, 
bts->ul_power_target);
+                 "(rx-ms-pwr-lvl %" PRIu8 ", max-ms-pwr-lvl %" PRIu8 ", 
rx-current %d dBm, rx-target %d dBm)\n",
+                 (diff > 0) ? "Raising" : "Lowering",
+                 lchan->ms_power_ctrl.current, ms_pwr_dbm(band, 
lchan->ms_power_ctrl.current),
+                 new_power, ms_pwr_dbm(band, new_power),
+                 ms_power, lchan->ms_power_ctrl.max,
+                 rxLevel, bts->ul_power_target);

        /* store the resulting new MS power level in the lchan */
        lchan->ms_power_ctrl.current = new_power;

--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/16439
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I983ff824ef6f54f1e800819d622158d5e2a51f04
Gerrit-Change-Number: 16439
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged

Reply via email to