diff options
author | Max <msuraev@sysmocom.de> | 2018-01-05 15:04:50 +0100 |
---|---|---|
committer | Harald Welte <laforge@gnumonks.org> | 2018-01-12 14:24:00 +0000 |
commit | 58818585bc99dc4aaaf94fec4ffeedf211a7b209 (patch) | |
tree | 42c0dc7ac99238016c12e6af72888094d78fa3e0 /src/bts.h | |
parent | 8dce1de6d2669023b715945cc58813380ac7f322 (diff) |
Clarify RACH-related interfaces
* make is_11bit parameter into bool
* remove is_single_block() from public interface and mark it as static
* move logging outside of if ladder
* move side-effects from is_single_block() into separate static
functions
* simplify UL-TBF allocation in case of 11-bit RACH
This immediately makes it obvious that priority is never actually used
despite being computed - seems like a leftover from merge of incomplete
patch series.
Change-Id: If189b7166a29a87ffb17a7a9bc560f674851fd53
Related: OS#1548
Diffstat (limited to 'src/bts.h')
-rw-r--r-- | src/bts.h | 4 |
1 files changed, 1 insertions, 3 deletions
@@ -368,11 +368,9 @@ public: int tfi_find_free(enum gprs_rlcmac_tbf_direction dir, uint8_t *_trx, int8_t use_trx); int rcv_imm_ass_cnf(const uint8_t *data, uint32_t fn); - uint8_t is_single_block(uint16_t ra, enum ph_burst_type burst_type, - uint8_t is_11bit, uint16_t *ms_class, uint16_t *priority); uint32_t rfn_to_fn(int32_t rfn); - int rcv_rach(uint16_t ra, uint32_t Fn, int16_t qta, uint8_t is_11bit, + int rcv_rach(uint16_t ra, uint32_t Fn, int16_t qta, bool is_11bit, enum ph_burst_type burst_type); void snd_dl_ass(gprs_rlcmac_tbf *tbf, uint8_t poll, const char *imsi); |