Harald Welte has uploaded this change for review. ( 
https://gerrit.osmocom.org/14205


Change subject: Fix use of osmocom-extended RSL Channel Numbers
......................................................................

Fix use of osmocom-extended RSL Channel Numbers

Bring our TTCN-3 view of how RSL channel numbers are defined in sync
with that of our other implementations (BTS, libosmocore, trxcon, ...)

Change-Id: I48908058ac2501a3b5ae7c74e4e8527cbaee1b01
Related: OS#4027
---
M bts/BTS_Tests_SMSCB.ttcn
M library/GSM_Types.ttcn
2 files changed, 20 insertions(+), 7 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/05/14205/1

diff --git a/bts/BTS_Tests_SMSCB.ttcn b/bts/BTS_Tests_SMSCB.ttcn
index c02522b..4d357ba 100644
--- a/bts/BTS_Tests_SMSCB.ttcn
+++ b/bts/BTS_Tests_SMSCB.ttcn
@@ -321,8 +321,17 @@
        }
 }

+private function t_cbch_chan_nr(CbchTestPars pars, template uint8_t tn) return 
template RslChannelNr {
+       if (pars.use_sdcch4) {
+               return t_RslChanNr_CBCH4(tn);
+       } else {
+               return t_RslChanNr_CBCH8(tn);
+       }
+}
+
 /* shared function doing the heavy lifting for most CBCH tests */
 private function f_TC_smscb(CbchTestPars pars) runs on test_CT {
+       var template RslChannelNr t_chan_nr := t_cbch_chan_nr(pars, 0); /* 
FIXME: TS number */
        var L1ctlDlMessage dl;
        var integer msg_count;
        timer T;
@@ -344,8 +353,7 @@
        T.start(5.0 + 3.0 * int2float(msg_count));
        /* Expect this to show up exactly once on the basic CBCH (four blocks) 
*/
        alt {
-                                          /* FIXME: Channel Nr for SDCCH/8 */
-       [] L1CTL.receive(tr_L1CTL_DATA_IND(t_RslChanNr_CBCH(0))) -> value dl {
+       [] L1CTL.receive(tr_L1CTL_DATA_IND(t_chan_nr)) -> value dl {
                var integer tb := f_cbch_fn2tb(dl.dl_info.frame_nr);
                var CBCH_Block cb := 
dec_CBCH_Block(dl.payload.data_ind.payload);
                log("Tb=", tb, ", CBCH: ", dl, ", block: ", cb);
@@ -378,6 +386,7 @@
 }

 private function f_TC_smscb_default_only(CbchTestPars pars) runs on test_CT {
+       var template RslChannelNr t_chan_nr := t_cbch_chan_nr(pars, 0); /* 
FIXME: TS number */
        var L1ctlDlMessage dl;
        timer T := 5.0;

@@ -389,7 +398,7 @@

        T.start;
        alt {
-       [] L1CTL.receive(tr_L1CTL_DATA_IND(t_RslChanNr_CBCH(0))) -> value dl {
+       [] L1CTL.receive(tr_L1CTL_DATA_IND(t_chan_nr)) -> value dl {
                var integer tb := f_cbch_fn2tb(dl.dl_info.frame_nr);
                log("CBCH: ", dl);
                var CBCH_Block cb := 
dec_CBCH_Block(dl.payload.data_ind.payload);
@@ -658,6 +667,7 @@
                basic := valueof(t_CbchPC(msgs_1m_3b_default)),
                extended := omit
        };
+       var template RslChannelNr t_chan_nr := t_RslChanNr_CBCH4(0);
        var L1ctlDlMessage dl;
        timer T := 5.0;

@@ -672,7 +682,7 @@

        T.start;
        alt {
-       [] L1CTL.receive(tr_L1CTL_DATA_IND(t_RslChanNr_CBCH(0))) -> value dl {
+       [] L1CTL.receive(tr_L1CTL_DATA_IND(t_chan_nr)) -> value dl {
                log("CBCH: ", dl);
                var CBCH_Block cb := 
dec_CBCH_Block(dl.payload.data_ind.payload);
                /* detect the proper CBCH messages; check frame number */
diff --git a/library/GSM_Types.ttcn b/library/GSM_Types.ttcn
index 3e24f81..ec74ce5 100644
--- a/library/GSM_Types.ttcn
+++ b/library/GSM_Types.ttcn
@@ -68,7 +68,8 @@
                RSL_CHAN_NR_RACH        ('10001'B),
                RSL_CHAN_NR_PCH_AGCH    ('10010'B),
                RSL_CHAN_NR_OSMO_PDCH   ('11000'B),
-               RSL_CHAN_NR_OSMO_CBCH   ('11001'B)
+               RSL_CHAN_NR_OSMO_CBCH4  ('11001'B),
+               RSL_CHAN_NR_OSMO_CBCH8  ('11010'B)
        } with { variant "FIELDLENGTH(5)" variant "FIELDORDER(msb)" };

        type record RslChanNr2 {
@@ -115,7 +116,8 @@
        template RslChannelNr t_RslChanNr_PCH_AGCH(template uint3_t tn) := 
t_RslChanNr0(tn, RSL_CHAN_NR_PCH_AGCH);
        template RslChannelNr t_RslChanNr_Bm(template uint3_t tn) := 
t_RslChanNr0(tn, RSL_CHAN_NR_Bm_ACCH);
        template RslChannelNr t_RslChanNr_PDCH(template uint3_t tn) := 
t_RslChanNr0(tn, RSL_CHAN_NR_OSMO_PDCH);
-       template RslChannelNr t_RslChanNr_CBCH(template uint3_t tn) := 
t_RslChanNr0(tn, RSL_CHAN_NR_OSMO_CBCH);
+       template RslChannelNr t_RslChanNr_CBCH4(template uint3_t tn) := 
t_RslChanNr0(tn, RSL_CHAN_NR_OSMO_CBCH4);
+       template RslChannelNr t_RslChanNr_CBCH8(template uint3_t tn) := 
t_RslChanNr0(tn, RSL_CHAN_NR_OSMO_CBCH8);
        template RslChannelNr t_RslChanNr_Lm(template uint3_t tn, template 
uint1_t sub_slot) := {
                u := { lm := { tag := '0001'B, sub_chan := sub_slot } },
                tn := tn
@@ -138,7 +140,8 @@
        template (value) RslChannelNr ts_RslChanNr_PCH_AGCH(uint3_t tn) := 
ts_RslChanNr0(tn, RSL_CHAN_NR_PCH_AGCH);
        template (value) RslChannelNr ts_RslChanNr_Bm(uint3_t tn) := 
ts_RslChanNr0(tn, RSL_CHAN_NR_Bm_ACCH);
        template (value) RslChannelNr ts_RslChanNr_PDCH(uint3_t tn) := 
ts_RslChanNr0(tn, RSL_CHAN_NR_OSMO_PDCH);
-       template (value) RslChannelNr ts_RslChanNr_CBCH(uint3_t tn) := 
ts_RslChanNr0(tn, RSL_CHAN_NR_OSMO_CBCH);
+       template (value) RslChannelNr ts_RslChanNr_CBCH4(uint3_t tn) := 
ts_RslChanNr0(tn, RSL_CHAN_NR_OSMO_CBCH4);
+       template (value) RslChannelNr ts_RslChanNr_CBCH8(uint3_t tn) := 
ts_RslChanNr0(tn, RSL_CHAN_NR_OSMO_CBCH8);
        template (value) RslChannelNr ts_RslChanNr_Lm(uint3_t tn, uint1_t 
sub_slot) := {
                u := { lm := { tag := '0001'B, sub_chan := sub_slot } },
                tn := tn

--
To view, visit https://gerrit.osmocom.org/14205
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I48908058ac2501a3b5ae7c74e4e8527cbaee1b01
Gerrit-Change-Number: 14205
Gerrit-PatchSet: 1
Gerrit-Owner: Harald Welte <[email protected]>

Reply via email to