diff options
| author | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2013-10-26 18:01:35 +0200 |
|---|---|---|
| committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2013-10-30 21:24:11 +0100 |
| commit | 86300bbeea0601e2a1be34021f438433b5f01a27 (patch) | |
| tree | 86d5c58c9c36f964b711df21e40206bf88786037 /src/bts.cpp | |
| parent | af8094d7995daccd5255b6f7450c3faef6409ef1 (diff) | |
tbf: Move the timer routine into the class
The timer is used for various timeouts and there is still external
client code that is calling it. In a perfect world the client code
would indicate that an event has happened and the internal timer
will be stopped. The best compromise is the "stop_t3191" method. It
allows to add semantic verification that the timer has been running.
Diffstat (limited to 'src/bts.cpp')
| -rw-r--r-- | src/bts.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/bts.cpp b/src/bts.cpp index 3c7ce407..089ef288 100644 --- a/src/bts.cpp +++ b/src/bts.cpp @@ -637,7 +637,7 @@ int gprs_rlcmac_pdch::rcv_control_block(struct gprs_rlcmac_bts *bts, } tbf_new_state(tbf, GPRS_RLCMAC_FLOW); /* stop pending assignment timer */ - tbf_timer_stop(tbf); + tbf->stop_timer(); if ((tbf->state_flags & (1 << GPRS_RLCMAC_FLAG_TO_DL_ASS))) { tbf->state_flags &= @@ -698,8 +698,7 @@ int gprs_rlcmac_pdch::rcv_control_block(struct gprs_rlcmac_bts *bts, } /* reset N3105 */ tbf->n3105 = 0; - /* stop timer T3191 */ - tbf_timer_stop(tbf); + tbf->stop_t3191(); tlli = tbf->tlli; LOGP(DRLCMAC, LOGL_DEBUG, "RX: [PCU <- BTS] TFI: %u TLLI: 0x%08x Packet Downlink Ack/Nack\n", tbf->tfi, tbf->tlli); tbf->poll_state = GPRS_RLCMAC_POLL_NONE; |