Attention is currently required from: osmith, neels.
fixeria has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bsc/+/29836 )
Change subject: gsm48_parse_meas_rep: fix parsing multi-band list
......................................................................
Patch Set 2:
(3 comments)
File src/osmo-bsc/gsm_04_08_rr.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29836/comment/73ce17d1_5486ea88
PS2, Line 770: TS 04.08
FYI: TS 44.018 is a more recent version of TS 04.08
https://gerrit.osmocom.org/c/osmo-bsc/+/29836/comment/a20a3696_a6a84c41
PS2, Line 775: struct bitvec *nbv
const
https://gerrit.osmocom.org/c/osmo-bsc/+/29836/comment/1d1cfdb7_468063cb
PS2, Line 780: arfcn = 0
"In each BCCH channel sub list the absolute RF channel numbers are placed in
increasing order of ARFCN, except that ARFCN 0, if included in the set, is put in the last
position in the sub list" -- do we handle ARFCN 0 as described?
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/29836
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ic5e4f0531e08685460948b102367825588d839ba
Gerrit-Change-Number: 29836
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 21 Oct 2022 18:40:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment