Attention is currently required from: osmith, pespin.
falconia has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/36896?usp=email )
Change subject: common: add support for TW-TS-001 ......................................................................
Patch Set 4:
(2 comments)
This change is ready for review.
File src/common/l1sap.c:
https://gerrit.osmocom.org/c/osmo-bts/+/36896/comment/67eaea98_9303058b PS1, Line 1998: uint8_t teh;
Trau-like Extension Header, see the text of TW-TS-001 spec.
Done
File src/common/rtp_input_preen.c:
https://gerrit.osmocom.org/c/osmo-bts/+/36896/comment/64b03db5_b6a712a3 PS1, Line 104: /* Strip TEH octet, leaving only standard EFR payload. */
Ack
Compilation builds are now passing thanks to libosmocore patches being merged, but Jenkins builder is still marking this patch as V-1 because of the linter tripping up on TEH. What do we do? Is there any kind of manual override? Or do we need to patch osmo-ci.git like @osmith@sysmocom.de originally suggested, and then maybe put that "teh||the" line back in after this osmo-bts patch is merged?
-
falconia