Change in osmo-bts[master]: bts-trx: loops.c: Take into account RSL CHAN ACT ms power level limits

[laforge]

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

Change subject: bts-trx: loops.c: Take into account RSL CHAN ACT ms power level limits
......................................................................

bts-trx: loops.c: Take into account RSL CHAN ACT ms power level limits

TS 48.058 sec 4.8 MS power control:
"""
TRX then attempts to keep the power control parameters within the limits
set by the MS POWER CONTROL message (or by the CHANNEL ACTIVATION message)
by changing the MS Power Level field of the L1 header sent to MS in each
SACCH block.
"""

Fixes TTCN3 BTS_Tests.TC_rsl_ms_pwr_dyn_max

Change-Id: Ifda92155bd9c277ac150a327a7ab63c854087788
---
M src/osmo-bts-trx/loops.c
1 file changed, 4 insertions(+), 0 deletions(-)

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



diff --git a/src/osmo-bts-trx/loops.c b/src/osmo-bts-trx/loops.c
index 8a5dd2e..4e77bca 100644
--- a/src/osmo-bts-trx/loops.c
+++ b/src/osmo-bts-trx/loops.c
@@ -57,6 +57,10 @@
 	if (new_power < 0)
 		new_power = 0;
 
+	/* Don't ask for smaller ms power level than the one set by BSC upon RSL CHAN ACT */
+	if (new_power < lchan->ms_power)
+		new_power = lchan->ms_power;
+
 	/* saturate at the maximum possible power level for the given band */
 	// FIXME: to go above 1W, we need to know classmark of MS
 	if (arfcn >= 512 && arfcn <= 885) {

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ifda92155bd9c277ac150a327a7ab63c854087788
Gerrit-Change-Number: 15877
Gerrit-PatchSet: 3
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-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191028/8165f820/attachment.htm>