Change in osmo-bsc[master]: power_control: enable Uplink DPC by default if format is known

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

fixeria gerrit-no-reply at lists.osmocom.org
Sun Feb 7 03:26:50 UTC 2021


fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/22768 )


Change subject: power_control: enable Uplink DPC by default if format is known
......................................................................

power_control: enable Uplink DPC by default if format is known

So far osmo-bsc would enable Uplink DPC (Dynamic Power Control) only
for osmo-bts, and the 'static' mode for all other BTS models.  This
decision dates back to the time when ip.access specific encoding for
dynamic power control parameters was not implemented, and the MS
Power Parameters IE was sent empty in the RSL messages.

Let's make a step forward by enabling Uplink DPC by default for
all BTS models which declare the API for vendor-specific encoding
of the power control parameters.  Currently this includes osmo-bts
and nanoBTS, but supporting ip.access specific format.

Change-Id: If86d27d4332af3d82f862737340d061e42e34eba
Related: SYS#4918
---
M doc/manuals/chapters/power_control.adoc
M src/osmo-bsc/bts.c
2 files changed, 8 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/68/22768/1

diff --git a/doc/manuals/chapters/power_control.adoc b/doc/manuals/chapters/power_control.adoc
index 5de2b9c..dbe2d13 100644
--- a/doc/manuals/chapters/power_control.adoc
+++ b/doc/manuals/chapters/power_control.adoc
@@ -117,8 +117,10 @@
 <2> Send both RSL MS/BS Power IE and vendor-specific MS/BS Power Parameters IE.
 <3> Do not send any power control IEs in RSL CHANnel ACTIVation messages.
 
-By default, `static` mode is used for BS power control, while `dyn-bts` is used
-for MS power control.  Changing the mode at run-time would not affect already
+By default, `static` mode is used for BS power control, while `dyn-bts` mode is
+automatically enabled for MS power control if vendor-specific format of the power
+control parameters (see above) is implemented for particular BTS model.  Otherwise
+`static` mode is used too.  Changing the mode at run-time would not affect already
 established connections, only the new ones (check flag `l`).
 
 For BS power control, there is an additional parameter:
diff --git a/src/osmo-bsc/bts.c b/src/osmo-bsc/bts.c
index ac0e2af..628d301 100644
--- a/src/osmo-bsc/bts.c
+++ b/src/osmo-bsc/bts.c
@@ -522,9 +522,6 @@
 
 	switch (bts->type) {
 	case GSM_BTS_TYPE_OSMOBTS:
-		/* Enable dynamic Uplink power control by default */
-		bts->ms_power_ctrl.mode = GSM_PWR_CTRL_MODE_DYN_BTS;
-		/* fall-through */
 	case GSM_BTS_TYPE_NANOBTS:
 		/* Set the default OML Stream ID to 0xff */
 		bts->oml_tei = 0xff;
@@ -543,6 +540,10 @@
 		break;
 	}
 
+	/* Enable dynamic Uplink power control by default (if supported) */
+	if (model->power_ctrl_enc_rsl_params != NULL)
+		bts->ms_power_ctrl.mode = GSM_PWR_CTRL_MODE_DYN_BTS;
+
 	return 0;
 }
 

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: If86d27d4332af3d82f862737340d061e42e34eba
Gerrit-Change-Number: 22768
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210207/5121ac5f/attachment.htm>


More information about the gerrit-log mailing list