[M] Change in osmo-bts[master]: tests/csd: add NT variants for TCH/F4.8 and TCH/F9.6 - gerrit-log

27 Sep 2024

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/38302?usp=email )
Change subject: tests/csd: add NT variants for TCH/F4.8 and TCH/F9.6
......................................................................
tests/csd: add NT variants for TCH/F4.8 and TCH/F9.6
The existing tests are all for T (transparent mode).
Add NT (non-transparent mode) variants.
Change-Id: Ie335bc0623dd7e887a0b3b1c40d61153b84924b2
Related: OS#1572
---
M tests/csd/csd_test.c
M tests/csd/csd_test.err
2 files changed, 62 insertions(+), 0 deletions(-)
Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved

diff --git a/tests/csd/csd_test.c b/tests/csd/csd_test.c
index 13a6419..0ad611f 100644
--- a/tests/csd/csd_test.c
+++ b/tests/csd/csd_test.c
@@ -54,12 +54,24 @@
    	.csd_mode = LCHAN_CSD_M_T_9600,
    },
    {
+		.name = "TCH/F9.6 NT",
+		.lchan_type = GSM_LCHAN_TCH_F,
+		.tch_mode = GSM48_CMODE_DATA_12k0,
+		.csd_mode = LCHAN_CSD_M_NT,
+	},
+	{
    	.name = "TCH/F4.8",
    	.lchan_type = GSM_LCHAN_TCH_F,
    	.tch_mode = GSM48_CMODE_DATA_6k0,
    	.csd_mode = LCHAN_CSD_M_T_4800,
    },
    {
+		.name = "TCH/F4.8 NT",
+		.lchan_type = GSM_LCHAN_TCH_F,
+		.tch_mode = GSM48_CMODE_DATA_6k0,
+		.csd_mode = LCHAN_CSD_M_NT,
+	},
+	{
    	.name = "TCH/H4.8",
    	.lchan_type = GSM_LCHAN_TCH_H,
    	.tch_mode = GSM48_CMODE_DATA_6k0,
diff --git a/tests/csd/csd_test.err b/tests/csd/csd_test.err
index b862324..cf19aff 100644
--- a/tests/csd/csd_test.err
+++ b/tests/csd/csd_test.err
@@ -24,6 +24,31 @@
     ff ff ff ff ff ff ff ff 3f 3f 3f 3f ff ff ff ff 
     ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
     ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+[i] Testing 'TCH/F9.6 NT' (bitnum=240)
+[i] csd_v110_rtp_encode() returns 160
+    3f 3f 3f 3f bf bf bf bf ff 7f 7f 7f bf bf bf bf 
+    ff 7f 7f 7f bf 3f 7f 7f bf bf bf bf ff 7f 7f 7f 
+    bf bf bf bf ff 7f 7f 7f 3f 3f 3f 3f bf bf bf bf 
+    ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f bf 7f 7f 7f 
+    bf bf bf bf ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f 
+    3f 3f 3f 3f bf bf bf bf ff 7f 7f 7f bf bf bf bf 
+    ff 7f 7f 7f bf bf 7f 7f bf bf bf bf ff 7f 7f 7f 
+    bf bf bf bf ff 7f 7f 7f 3f 3f 3f 3f bf bf bf bf 
+    ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f bf ff 7f 7f 
+    bf bf bf bf ff 7f 7f 7f bf bf bf bf ff 7f 7f 7f 
+[i] csd_v110_rtp_decode() returns 240
+[i] Testing 'TCH/F9.6 NT' (IDLE)
+[i] csd_v110_rtp_encode() returns 160
+    3f 3f 3f 3f ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff 3f 3f 3f 3f ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    3f 3f 3f 3f ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff 3f 3f 3f 3f ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
 [i] Testing 'TCH/F4.8' (bitnum=120)
 [i] csd_v110_rtp_encode() returns 160
     7f 7f 7f 7f 7f 7f 7f 7f ff 7f ff 7f ff 7f ff 7f 
@@ -49,6 +74,31 @@
     ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
     ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
     ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+[i] Testing 'TCH/F4.8 NT' (bitnum=120)
+[i] csd_v110_rtp_encode() returns 160
+    7f 7f 7f 7f 7f 7f 7f 7f ff 7f ff 7f ff 7f ff 7f 
+    ff ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 7f 
+    ff ff 7f ff 7f ff 7f ff ff 7f 7f 7f 7f ff 7f ff 
+    ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 
+    ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 
+    7f 7f 7f 7f 7f 7f 7f 7f ff 7f ff 7f ff 7f ff 7f 
+    ff ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 7f 
+    ff ff 7f ff 7f ff 7f ff ff 7f 7f ff 7f ff 7f ff 
+    ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 
+    ff 7f ff 7f ff 7f ff 7f ff ff 7f ff 7f ff 7f ff 
+[i] csd_v110_rtp_decode() returns 120
+[i] Testing 'TCH/F4.8 NT' (IDLE)
+[i] csd_v110_rtp_encode() returns 160
+    7f 7f 7f 7f 7f 7f 7f 7f ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    7f 7f 7f 7f 7f 7f 7f 7f ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
+    ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff 
 [i] Testing 'TCH/H4.8' (bitnum=240)
 [i] csd_v110_rtp_encode() returns 160
     3f 3f 3f 3f bf bf bf bf ff 7f 7f 7f bf bf bf bf
-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/38302?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ie335bc0623dd7e887a0b3b1c40d61153b84924b2
Gerrit-Change-Number: 38302
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria vyanitskiy@sysmocom.de
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria vyanitskiy@sysmocom.de
Gerrit-Reviewer: laforge laforge@osmocom.org
Gerrit-Reviewer: pespin pespin@sysmocom.de



    