diff options
author | Max <msuraev@sysmocom.de> | 2016-10-18 16:19:33 +0200 |
---|---|---|
committer | Max <msuraev@sysmocom.de> | 2016-10-18 16:19:33 +0200 |
commit | 582e4f627674f46310a90d9061e82fb342051b42 (patch) | |
tree | 9db731ddb27534e5785441609e0b698bfc36847f | |
parent | fbd96f55fd904d412c7a84c7764810f16e2ec714 (diff) |
Replace magic number with define
Value 4 used as magic number by both OpenBSC and OsmoBTS so it make
sense to add it to shared header. See
ebb483b69a5319e522ba5f713e9cb6f68a814a6a in osmo-bts for details.
Change-Id: I9c6ad68f4c6aa72d39ec7e5a6968b36ec20e79f4
-rw-r--r-- | openbsc/include/openbsc/gsm_data_shared.h | 4 | ||||
-rw-r--r-- | openbsc/src/libcommon/gsm_data_shared.c | 2 |
2 files changed, 5 insertions, 1 deletions
diff --git a/openbsc/include/openbsc/gsm_data_shared.h b/openbsc/include/openbsc/gsm_data_shared.h index d88995970..487a3f397 100644 --- a/openbsc/include/openbsc/gsm_data_shared.h +++ b/openbsc/include/openbsc/gsm_data_shared.h @@ -46,6 +46,10 @@ enum gsm_chreq_reason_t { GSM_CHREQ_REASON_OTHER, }; +/* lchans 0..3 are SDCCH in combined channel configuration, + use 4 as magic number for BCCH hack - see osmo-bts-../oml.c:opstart_compl() */ +#define CCCH_LCHAN 4 + #define TRX_NR_TS 8 #define TS_MAX_LCHAN 8 diff --git a/openbsc/src/libcommon/gsm_data_shared.c b/openbsc/src/libcommon/gsm_data_shared.c index b8853ad69..df204feab 100644 --- a/openbsc/src/libcommon/gsm_data_shared.c +++ b/openbsc/src/libcommon/gsm_data_shared.c @@ -579,7 +579,7 @@ uint8_t gsm_pchan2chan_nr(enum gsm_phys_chan_config pchan, * here. This should never be sent in an RSL message. * See osmo-bts-xxx/oml.c:opstart_compl(). */ - if (lchan_nr == 4) + if (lchan_nr == CCCH_LCHAN) chan_nr = 0; else OSMO_ASSERT(lchan_nr < 4); |