diff options
author | Vasil Velichkov <vvvelichkov@gmail.com> | 2019-03-13 20:53:24 +0200 |
---|---|---|
committer | Vasil Velichkov <vvvelichkov@gmail.com> | 2019-03-19 17:36:28 +0200 |
commit | c5bb436cdad352880741ce1496e45a66cfc06f8c (patch) | |
tree | 7a7fea577847cfae7bca46644f478eaa264d628f /grc | |
parent | 81ba56f59dcefb2d39d023c68d3a077b4a321520 (diff) |
Fix the TCH/H decoder and demapper XML definitions
- The second argument of tch_h_decoder constructor is a string that
accepts the value of MultiRate configuration element and not an enum
like the tch_f_decoder decoder.
- Make the demapper's burst sink required.
- Make all parameters visible.
- Rename the TCH/H Channel parameter to Sub-channel number.
- Add the qa_tch_h_decoder test in the CMakeLists.txt
- Fix several typos
Change-Id: I92d4f49955c634df7d76f17cfb58d7106846c1bd
Diffstat (limited to 'grc')
-rw-r--r-- | grc/decoding/gsm_tch_h_decoder.xml | 48 | ||||
-rw-r--r-- | grc/demapping/gsm_tch_h_chans_demapper.xml | 15 |
2 files changed, 24 insertions, 39 deletions
diff --git a/grc/decoding/gsm_tch_h_decoder.xml b/grc/decoding/gsm_tch_h_decoder.xml index 895104e..3936336 100644 --- a/grc/decoding/gsm_tch_h_decoder.xml +++ b/grc/decoding/gsm_tch_h_decoder.xml @@ -3,54 +3,27 @@ <name>TCH/H decoder</name> <key>gsm_tch_h_decoder</key> <import>import grgsm</import> - <make>grgsm.tch_h_decoder($sub_channel, $mode, $boundary_check)</make> + <make>grgsm.tch_h_decoder($sub_channel, $multi_rate, $boundary_check)</make> <param> - <name>sub-channel number</name> + <name>Sub-channel number</name> <key>sub_channel</key> <value>0</value> <type>int</type> + <hide>none</hide> <option> - <name>sub-channel 0</name> + <name>0</name> <key>0</key> </option> <option> - <name>sub-channel 1</name> + <name>1</name> <key>1</key> </option> </param> <param> - <name>TCH/H coding mode</name> - <key>mode</key> - <type>enum</type> - <option> - <name>GSM-HS</name> - <key>grgsm.TCH_HS</key> - </option> - <option> - <name>GSM-AMR 7.95</name> - <key>grgsm.TCH_AHS7_95</key> - </option> - <option> - <name>GSM-AMR 7.4</name> - <key>grgsm.TCH_AHS7_4</key> - </option> - <option> - <name>GSM-AMR 6.7</name> - <key>grgsm.TCH_AHS6_7</key> - </option> - <option> - <name>GSM-AMR 5.9</name> - <key>grgsm.TCH_AHS5_9</key> - </option> - <option> - <name>GSM-AMR 5.15</name> - <key>grgsm.TCH_AHS5_15</key> - </option> - <option> - <name>GSM-AMR 4.75</name> - <key>grgsm.TCH_AHS4_75</key> - </option> + <name>MultiRate configuration</name> + <key>multi_rate</key> + <type>string</type> </param> <param> <name>Voice boundary detection</name> @@ -83,6 +56,11 @@ </source> <doc> +The MultiRate configuration string should contains the hex string from the +MultiRate configuration element from the Assignment Command message. +Example: 28111a40. +See 3GPP TS 44.018 - 10.5.2.21aa MultiRate configuratio + If "Voice boundary detection" is enabled, then only bursts are decoded as voice where - the framenumber is greater then the framenumber of a received "Connect" or "Connect Acknowlegde" message, and diff --git a/grc/demapping/gsm_tch_h_chans_demapper.xml b/grc/demapping/gsm_tch_h_chans_demapper.xml index f2bd67f..52148f5 100644 --- a/grc/demapping/gsm_tch_h_chans_demapper.xml +++ b/grc/demapping/gsm_tch_h_chans_demapper.xml @@ -10,22 +10,29 @@ <key>timeslot_nr</key> <value>2</value> <type>int</type> - <hide>part</hide> + <hide>none</hide> </param> <param> - <name>TCH/H Channel</name> + <name>Sub-channel number</name> <key>tch_h_channel</key> <value>0</value> <type>int</type> - <hide>#if $tch_type() == 0 then 'all' else 'none'#</hide> + <hide>none</hide> + <option> + <name>0</name> + <key>0</key> + </option> + <option> + <name>1</name> + <key>1</key> + </option> </param> <check>$tch_h_channel() > -1 and $tch_h_channel() < 2</check> <sink> <name>bursts</name> <type>message</type> - <optional>1</optional> </sink> <source> <name>tch_bursts</name> |