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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>

Reply via email to