Attention is currently required from: jolly, laforge.

jolly uploaded patch set #3 to the change originally created by laforge.

View Change

The following approvals got outdated and were removed: Code-Review-1 by jolly, Verified+1 by Jenkins Builder

mobile: Fix PCS ARFCN handling: PCS can only be ARFCN 512..810

While it is correct to use the band indicator from SI1 rest octets,
it may only be applied for ARFCN values in the range 512..810.

The function gsm_refer_pcs() is used to determine, if the cell (which
'talks' about ARFCNs) refers to them PCS or DCS channels. It returns
true, if it refers to PCS, but this only means that ARFCNs in the range
512..810 are PCS channels, not all ARFCNs.

The new function gsm_arfcn_refer_pcs() is used to add the PCS flag to an
ARFCN, if the given cell refers to PCS and the given ARFCN is in the PCS
range 512..810.

Change-Id: Id99c8534bf853f4f24f99364790c1ac1df6cc007
Related: OS#6078
---
M src/host/layer23/include/osmocom/bb/common/sysinfo.h
M src/host/layer23/src/common/sysinfo.c
M src/host/layer23/src/mobile/gsm322.c
M src/host/layer23/src/mobile/gsm48_rr.c
4 files changed, 71 insertions(+), 41 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/32/33532/3

To view, visit change 33532. To unsubscribe, or for help writing mail filters, visit settings.