Change in osmo-bts[master]: power_control.c: Log rx current and target signal levels

[laforge]

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

Change subject: power_control.c: Log rx current and target signal levels
......................................................................

power_control.c: Log rx current and target signal levels

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

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved



diff --git a/src/common/power_control.c b/src/common/power_control.c
index 106e2a8..574825e 100644
--- a/src/common/power_control.c
+++ b/src/common/power_control.c
@@ -99,15 +99,17 @@
 	}
 
 	if (lchan->ms_power_ctrl.current == new_power) {
-		LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "Keeping MS new_power at control level %d (%d dBm)\n",
-			new_power, ms_pwr_dbm(band, new_power));
+		LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "Keeping MS new_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);
 		return 0;
 	}
 
-	LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "%s MS new_power from control level %d (%d dBm) to %d (%d dBm)\n",
+	LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "%s MS new_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));
+		new_power, ms_pwr_dbm(band, new_power), ms_power, 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/+/16073
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I31ce7930a1daa20a2ff81f31a37df94298c877d6
Gerrit-Change-Number: 16073
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191118/0cdd15a2/attachment.htm>