fixeria submitted this change.

View Change

Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified
BTS_Tests: tune back to CCCH/BCCH in TC_sacch_chan_act_ho_[a]sync

Sending L1CTL_DM_REL_REQ does not make the L1 tune back to CCCH/BCCH.
We need to send a L1CTL_FBSB_REQ after releasing a dedicated channel
if we want to establish another one. This is essential when running
tests using Calypso PHY, and will be required once we have proper
trxcon_fsm implementation [1] merged.

Related: [1] osmocom-bb.git Ifaf63ead9dd180181358e771367b2a686ba159ca
Change-Id: I65fb243a62fc7670b43f467d6b79268cdfb98f36
---
M bts/BTS_Tests.ttcn
1 file changed, 12 insertions(+), 0 deletions(-)

diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index 6772cc5..a322fde 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -1456,6 +1456,9 @@
f_rsl_chan_deact();
f_L1CTL_DM_REL_REQ(L1CTL, g_chan_nr);

+ /* Tune back to CCCH/BCCH */
+ f_l1_tune(L1CTL);
+

/* Step 2: Activate ASYNC HO channel with MS power IE */

@@ -1533,6 +1536,9 @@
f_rsl_chan_deact();
f_L1CTL_DM_REL_REQ(L1CTL, g_chan_nr);

+ /* Tune back to CCCH/BCCH */
+ f_l1_tune(L1CTL);
+

/* Step 2a: Activate SYNC HO channel with only MS power IE */

@@ -1558,6 +1564,9 @@
f_rsl_chan_deact();
f_L1CTL_DM_REL_REQ(L1CTL, g_chan_nr);

+ /* Tune back to CCCH/BCCH */
+ f_l1_tune(L1CTL);
+

/* Step 2b: Activate SYNC HO channel with TA IE */

@@ -1582,6 +1591,9 @@
f_rsl_chan_deact();
f_L1CTL_DM_REL_REQ(L1CTL, g_chan_nr);

+ /* Tune back to CCCH/BCCH */
+ f_l1_tune(L1CTL);
+

/* Step 3: Activate SYNC HO channel with MS power IE and TA IE */


2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I65fb243a62fc7670b43f467d6b79268cdfb98f36
Gerrit-Change-Number: 28833
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged