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


Change subject: SI13: Set DRX_TIMER_MAX value actually transmitted
......................................................................

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 https://gerrit.osmocom.org/c/osmo-bsc/+/33754
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ibf01a50b258e197ba5e3173492513349ddffdb38
Gerrit-Change-Number: 33754
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-MessageType: newchange

Reply via email to