Attention is currently required from: neels, fixeria. osmith 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 3:
(3 comments)
File src/osmo-bsc/gsm_04_08_rr.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29836/comment/e86c028d_cf583026 PS2, Line 770: TS 04.08
FYI: TS 44.018 is a more recent version of TS 04. […]
thanks, done
https://gerrit.osmocom.org/c/osmo-bsc/+/29836/comment/4245c22b_fd26016f PS2, Line 775: struct bitvec *nbv
const
Done
https://gerrit.osmocom.org/c/osmo-bsc/+/29836/comment/839f7700_fae834b2 PS2, Line 780: arfcn = 0
"In each BCCH channel sub list the absolute RF channel numbers are placed in increasing order of ARF […]
good catch! this wasn't implemented correctly, fixed in patchset 3.
-
osmith