Attention is currently required from: fixeria, osmith, pespin.
laforge has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/35134?usp=email )
Change subject: rlp: Add support for 576bit RLP frames
......................................................................
Patch Set 3:
(2 comments)
File src/gsm/rlp.c:
https://gerrit.osmocom.org/c/libosmocore/+/35134/comment/aafd05b9_812a024d
PS3, Line 74: /* we only support 240 bit so far */
this is not true anymore?
Done
https://gerrit.osmocom.org/c/libosmocore/+/35134/comment/cb81046a_f26ced39
PS3, Line 88: out->fcs = (data[data_len-1] << 16) | (data[data_len-2]) <<
8 | (data[data_len-3] << 0);
all this could really go into the previous commit.
yes, with the difference that the 240bit is actually tested while the 576 is not.
I'd like to merge the 240bit code now and the 576bit later when I found devices and a
working setup to actually verify it.
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/35134?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: Idfdcabb19fe8733fb9c5ee76a39b0bf4cdf60c2c
Gerrit-Change-Number: 35134
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 28 Nov 2023 21:56:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment