Attention is currently required from: jolly, fixeria, pespin.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647 )
Change subject: GSM_RR_Types: Add support for VBS/VGCS related L3 RR messages ......................................................................
Patch Set 2:
(7 comments)
File library/GSM_RR_Types.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/7559ee54_ec9ac... PS2, Line 590: HEX1
Below in `UplinkAccessIndTV` you're using `uint4_t`. […]
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/a74d8c8c_cae8d... PS2, Line 597: } with { variant "TAG(cksn, tag = 'B'H)" };
... since I see no "tag" nor "cksn" fields in here? […]
The way I did it, the IEI is tied to the message in whihc the IE occurs, and not to the IE itself. In GSM 04.08 and some other protocols there are instances where the same IE has different IEIs depending on where they appear. In this case, the IEI must be "glued" to the containing message, and not to the IE itself.
In this specific instance, I don't know if the CipheringKeySeqNrTV occurs in other places (with or without different IEI). But I don't see anything wrong with it, given that also in the SPEC, the IEIs are listed in the message, and not in the definition of the IE.
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/d35f1736_7b6d3... PS2, Line 650: } with { variant (len) "LENGTHTO(es,spare,uai_rach,priority)" };
Please add `variant "PRESENCE(iei = '31'O)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/c894a58f_6b937... PS2, Line 656: } with { variant "" };
Missing `variant "PRESENCE(iei = '32'O)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/4c6da24f_25404... PS2, Line 663: } with { variant (len) "LENGTHTO(talker_id)" };
Missing `variant "PRESENCE(iei = '33'O)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/5baa25bc_29225... PS2, Line 670: } with { variant "" };
Missing `variant "PRESENCE(iei = 8)"`.
Done
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/32647/comment/a2aa0f79_e3292... PS2, Line 690:
ws
Done