laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/39836?usp=email ) Change subject: cosmetic: Document missing CBQ3 optional field in SI4ro ...................................................................... cosmetic: Document missing CBQ3 optional field in SI4ro Change-Id: I52705a6a3422911698547c101329f7d01176e634 --- M src/gsm/gsm48_rest_octets.c 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/src/gsm/gsm48_rest_octets.c b/src/gsm/gsm48_rest_octets.c index 02469a7..ddcd258 100644 --- a/src/gsm/gsm48_rest_octets.c +++ b/src/gsm/gsm48_rest_octets.c @@ -748,6 +748,11 @@ } else { bitvec_set_bit(&bv, L); } + + /* TODO: + * {L | H < CBQ3 : bit (2) > "Cell Bar Qualify 3" in TS 44.018 Table 10.5.2.11.1 + * {0 | 1 < SI13alt Position : bit >}} ; + */ } else { /* L and break indicator */ bitvec_set_bit(&bv, L); -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/39836?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I52705a6a3422911698547c101329f7d01176e634 Gerrit-Change-Number: 39836 Gerrit-PatchSet: 1 Gerrit-Owner: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt;