Attention is currently required from: fixeria, laforge, pespin. falconia has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/36669?usp=email )
Change subject: gsm0808: add knowledge of TW-TS-003 BSSMAP IE ...................................................................... Patch Set 2: (1 comment) File src/gsm/gsm0808.c: https://gerrit.osmocom.org/c/libosmocore/+/36669/comment/4c1ecbfa_4969098e PS1, Line 2316: [GSM0808_IE_THEMWI_RTP_EXTENSIONS] = { TLV_TYPE_TV }, > You may want to keep this as TLV to match the Abis IE. The A interface can't be changed as easily as Abis here: while Abis-IP is essentially a private interface fully inside Osmocom kingdom, the AoIP interface (3GPP specs plus TW-TS-003 extension) connects two different fiefdoms: Osmocom kingdom on BSS side and W. R. of Themyscira on CN side. TW-TS-003 is thus an external ICD, unlike Osmocom-internal definitions like the Abis IE, and TW-TS-003 section 5.2 defines this IE as having TV format. It is true that TW-TS-003 is still a very new spec and can be revised, but revising TW-TS is a more formal/involved process than Gerrit here. However, to make it easier to merge the most essential part (reserving IEI code point 0xF2 for Themyscira and identifying it by the same symbolic name as in TW=TS-003 spec), I'll resubmit this patch with just protocol/gsm_08_08.h part kept, and drop this TLV decoder addition for now. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/36669?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Idd6f1838bdc0cd71a2f6357e9b6d8b8fa4f7c46c Gerrit-Change-Number: 36669 Gerrit-PatchSet: 2 Gerrit-Owner: falconia <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: laforge <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Comment-Date: Thu, 09 May 2024 15:58:20 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <[email protected]> Gerrit-MessageType: comment
