aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeels Hofmeyr <nhofmeyr@sysmocom.de>2022-02-23 14:25:22 +0100
committerNeels Hofmeyr <nhofmeyr@sysmocom.de>2022-02-28 17:37:05 +0100
commitbde89d5662f6d78e9c7b9ebc026be40ee5474192 (patch)
tree2aae19ec7c9dd355949cb0ba2172db7064c6ed3a
parent3e1ff457c20b56c4e6e7a419e40b3756645674d7 (diff)
BSSAP: HO Request Ack: add missing Codec List (BSS Supported)2021q4
-rw-r--r--TODO-RELEASE1
-rw-r--r--include/osmocom/gsm/gsm0808.h6
-rw-r--r--src/gsm/gsm0808.c6
3 files changed, 12 insertions, 1 deletions
diff --git a/TODO-RELEASE b/TODO-RELEASE
index 38551459..89023719 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -8,3 +8,4 @@
# If any interfaces have been removed or changed since the last public release: c:r:0.
#library what description / commit summary line
libosmogsm ABI BREAKAGE CELL_IDENT_WHOLE_GLOBAL_PS changed enum number
+libosmogsm add struct member Add codec_list_bss_supported to gsm0808_handover_request_ack (more_items flag ensures ABI compat)
diff --git a/include/osmocom/gsm/gsm0808.h b/include/osmocom/gsm/gsm0808.h
index c79da6a4..b4c78032 100644
--- a/include/osmocom/gsm/gsm0808.h
+++ b/include/osmocom/gsm/gsm0808.h
@@ -246,8 +246,12 @@ struct gsm0808_handover_request_ack {
const struct sockaddr_storage *aoip_transport_layer;
+ bool more_items; /*!< set this to true iff any fields below are used */
+
+ struct gsm0808_speech_codec_list codec_list_bss_supported; /*< omit when .len == 0 */
+
/* more items are defined in the spec and may be added later */
- bool more_items; /*!< always set this to false */
+ bool more_items2; /*!< always set this to false */
};
struct msgb *gsm0808_create_handover_request_ack2(const struct gsm0808_handover_request_ack *params);
diff --git a/src/gsm/gsm0808.c b/src/gsm/gsm0808.c
index 4f9c6875..7013d8db 100644
--- a/src/gsm/gsm0808.c
+++ b/src/gsm/gsm0808.c
@@ -1045,6 +1045,12 @@ struct msgb *gsm0808_create_handover_request_ack2(const struct gsm0808_handover_
if (params->aoip_transport_layer)
gsm0808_enc_aoip_trasp_addr(msg, params->aoip_transport_layer);
+ /* AoIP: add Codec List (BSS Supported) 3.2.2.103.
+ * (codec_list_bss_supported was added to struct gsm0808_handover_request_ack later than speech_codec_chosen
+ * below, but it needs to come before it in the message coding). */
+ if (params->more_items && params->codec_list_bss_supported.len)
+ gsm0808_enc_speech_codec_list(msg, &params->codec_list_bss_supported);
+
/* AoIP: Speech Codec (Chosen) 3.2.2.104 */
if (params->speech_codec_chosen_present)
gsm0808_enc_speech_codec(msg, &params->speech_codec_chosen);