pespin has uploaded this change for review.

View Change

SI13: Set DRX_TIMER_MAX value actually transmitted

The set of values in seconds which can be expressed in the 3-bit field
DRX_TIMER_MAX (0s, 1s, 2s, 4s,...64s) don't include the "3s" that where
being specified. Instead, libosmocore's encode_drx_timer() was
taking both upper boundary "4s" and encoding it "0b011".
Hence, better write the value actually being transmitted to MS, to avoid
users/readers confusion.

Change-Id: Ibf01a50b258e197ba5e3173492513349ddffdb38
---
M src/osmo-bsc/system_information.c
1 file changed, 17 insertions(+), 1 deletion(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/54/33754/1
diff --git a/src/osmo-bsc/system_information.c b/src/osmo-bsc/system_information.c
index 35fce68..9ed77f9 100644
--- a/src/osmo-bsc/system_information.c
+++ b/src/osmo-bsc/system_information.c
@@ -1244,7 +1244,7 @@
.nmo = GPRS_NMO_II,
.t3168 = 2000,
.t3192 = 1500,
- .drx_timer_max = 3,
+ .drx_timer_max = 4,
.bs_cv_max = 15,
.ctrl_ack_type_use_block = bts->gprs.ctrl_ack_type_use_block,
.ext_info_present = true,

To view, visit change 33754. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ibf01a50b258e197ba5e3173492513349ddffdb38
Gerrit-Change-Number: 33754
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-MessageType: newchange