Harald Welte has submitted this change and it was merged.

Change subject: a_iface: s/Abis/L3/ for speech version/preference fields
......................................................................


a_iface: s/Abis/L3/ for speech version/preference fields

There is no encoding of speech version / preference on Abis, only
on L3.  L3 is carried on Um, Abis and A.  Hence, referrin to Abis
in function names and comments is irritating.

Change-Id: Id226cd1414ca2a92356801bc71f43102d03ba37e
---
M src/libmsc/a_iface.c
1 file changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/libmsc/a_iface.c b/src/libmsc/a_iface.c
index 3983ede..ed6dcdb 100644
--- a/src/libmsc/a_iface.c
+++ b/src/libmsc/a_iface.c
@@ -234,10 +234,10 @@
 }
 
 /* Convert speech version field */
-static uint8_t convert_Abis_sv_to_A_sv(int speech_ver)
+static uint8_t convert_l3_sv_to_A_sv(int speech_ver)
 {
        /* The speech versions that are transmitted in the Bearer capability
-        * information element, that is transmitted on the Abis interfece
+        * information element, that is transmitted on the Layer 3 (CC)
         * use a different encoding than the permitted speech version
         * identifier, that is signalled in the channel type element on the A
         * interface. (See also 3GPP TS 48.008, 3.2.2.1 and 3GPP TS 24.008,
@@ -270,11 +270,11 @@
 }
 
 /* Convert speech preference field */
-static uint8_t convert_Abis_prev_to_A_pref(int radio)
+static uint8_t convert_l3_prev_to_A_pref(int radio)
 {
        /* The Radio channel requirement field that is transmitted in the
         * Bearer capability information element, that is transmitted on the
-        * Abis interfece uses a different encoding than the Channel rate and
+        * Layer 3 (CC) uses a different encoding than the Channel rate and
         * type field that is signalled in the channel type element on the A
         * interface. (See also 3GPP TS 48.008, 3.2.2.1 and 3GPP TS 24.008,
         * 10.5.102 */
@@ -309,7 +309,7 @@
        for (i = 0; i < ARRAY_SIZE(bc->speech_ver); i++) {
                if (bc->speech_ver[i] == -1)
                        break;
-               sv = convert_Abis_sv_to_A_sv(bc->speech_ver[i]);
+               sv = convert_l3_sv_to_A_sv(bc->speech_ver[i]);
                if (sv != 0xFF) {
                        /* Detect if something else than
                         * GSM HR V1 is supported */
@@ -331,7 +331,7 @@
                 * preference and assume a preference for full rate. */
                ct->ch_rate_type = GSM0808_SPEECH_FULL_BM;
        else
-               ct->ch_rate_type = convert_Abis_prev_to_A_pref(bc->radio);
+               ct->ch_rate_type = convert_l3_prev_to_A_pref(bc->radio);
 
        if (count)
                return 0;

-- 
To view, visit https://gerrit.osmocom.org/6378
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id226cd1414ca2a92356801bc71f43102d03ba37e
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder

Reply via email to