Review at  https://gerrit.osmocom.org/6871

measurement.c: Fix sdcch4_meas_rep_fn102

Since Change-Id: I23fba50f48415314da40cf5bf86fce2ed3e66af6 we were not
reporting measurements for SDCCH/4 channel types due to the wrong
encoding of the sdcch4_meas_rep_fn102 table.

Let's fix the table by encoding the needed information:

 "What is the modulo-102 remainder of the first burst of the last block
  before fn%102 reaches 37?"

The TS 45.002 Clause 7 tables have to be consulted carefully to
determine this information.

Change-Id: Icf02354872670126ab3297b787b216981ca6c309
Related: OS#2965
---
M src/common/measurement.c
1 file changed, 5 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/71/6871/1

diff --git a/src/common/measurement.c b/src/common/measurement.c
index 69fbd94..edb1e31 100644
--- a/src/common/measurement.c
+++ b/src/common/measurement.c
@@ -78,11 +78,12 @@
        [7] = 11 + 35
 };
 
+/* FN of the first burst whose block completes before reaching fn%102=37 */
 static const uint8_t sdcch4_meas_rep_fn102[] = {
-       [0] = 36 + 4,
-       [1] = 36 + 8,
-       [2] = 36 + 14,
-       [3] = 36 + 18
+       [0] = 88,       /* 37(SDCCH), 57(SACCH), 88(SDCCH) */
+       [1] = 92,       /* 41(SDCCH), 61(SACCH), 92(SDCCH) */
+       [2] = 6,        /* 6(SACCH), 47(SDCCH), 98(SDCCH) */
+       [3] = 10        /* 10(SACCH), 0(SDCCH), 51(SDCCH) */
 };
 
 /* Note: The reporting of the measurement results is done via the SACCH 
channel.

-- 
To view, visit https://gerrit.osmocom.org/6871
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icf02354872670126ab3297b787b216981ca6c309
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <lafo...@gnumonks.org>

Reply via email to