From ad253850d8435f3c7e25697fde316a7affb99f3b Mon Sep 17 00:00:00 2001 From: Pau Espin Pedrol Date: Wed, 22 Mar 2023 14:28:17 +0100 Subject: [PATCH] Use new GSM0408 defines for half-octet tags MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The old ones have been deprecated and shall not be used anymore· Depends: libosmocore.git Change-Id I799e35dc8d4d153fa63bf50563a5482cdf4de2d7 Change-Id: Id3be8e38ec87ae39c4e1b4fab163563b24fb2cee --- TODO-RELEASE | 3 ++- src/osmo-bsc/gsm_04_08_rr.c | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/TODO-RELEASE b/TODO-RELEASE index 98023fc3a..2658cdd1c 100644 --- a/TODO-RELEASE +++ b/TODO-RELEASE @@ -12,4 +12,5 @@ osmo-bsc CTRL,VTY osmo_fsm instance IDs now use new dynamic timeslot names 'DYNAMIC_OSMOCOM' and 'DYNAMIC_IPACCESS' libosmogsm >1.8.0 circuit switched data stuff (gsm0808_enc/dec_channel_type etc.) libosmo-abis >1.4.0 osmo_ortp.h: add RTP_PT_CSDATA -libosmo-sccp >1.7.0 osmo_sccp_{get,set}_priv() \ No newline at end of file +libosmo-sccp >1.7.0 osmo_sccp_{get,set}_priv() +libosmocore >1.8.0 GSM48_IE_CIP_MODE_SET_HO and GSM48_IE_SYNC_IND_HO \ No newline at end of file diff --git a/src/osmo-bsc/gsm_04_08_rr.c b/src/osmo-bsc/gsm_04_08_rr.c index ee772df6c..8395dcfa2 100644 --- a/src/osmo-bsc/gsm_04_08_rr.c +++ b/src/osmo-bsc/gsm_04_08_rr.c @@ -565,7 +565,7 @@ struct msgb *gsm48_make_ho_cmd(const struct gsm_lchan *new_lchan, * TODO: NCI (Normal cell indication), currently 0. */ const uint8_t sync_ind = async ? 0x00 : 0x01; /* T (4 bit) + V (4 bit), see 3GPP TS 44.018, 10.5.2.39 */ - msgb_v_put(msg, GSM48_IE_SYNC_IND | (sync_ind & 0x0f)); + msgb_v_put(msg, (GSM48_IE_SYNC_IND_HO << 4) | (sync_ind & 0x0f)); } if (new_lchan->ts->hopping.enabled) { @@ -595,7 +595,7 @@ struct msgb *gsm48_make_ho_cmd(const struct gsm_lchan *new_lchan, if (new_lchan->encr.alg_a5_n > 0) cms = (new_lchan->encr.alg_a5_n - 1) << 1 | 1; /* T (4 bit) + V (4 bit), see 3GPP TS 44.018, 10.5.2.9 */ - msgb_v_put(msg, GSM48_IE_CIP_MODE_SET | (cms & 0x0f)); + msgb_v_put(msg, (GSM48_IE_CIP_MODE_SET_HO << 4) | (cms & 0x0f)); } /* in case of multi rate we need to attach a config */