pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
cosmetic: Improve comment

Change-Id: I8c963ba7421faf2b3dcab006ea629c4628d17b44
---
M src/tbf_ul_ass_fsm.c
1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/src/tbf_ul_ass_fsm.c b/src/tbf_ul_ass_fsm.c
index 26ed8ee..6f88f11 100644
--- a/src/tbf_ul_ass_fsm.c
+++ b/src/tbf_ul_ass_fsm.c
@@ -195,10 +195,10 @@
unsigned int sec, micro;
struct GprsMs *ms = tbf_ms(ctx->tbf);

- /* Here it's time where received PKT RES REQ or DL ACK/NACK to request a new UL TBF,
+ /* Here it's point in time where we received PKT RES REQ or DL ACK/NACK to request a new UL TBF,
* so MS will be gone after T3168 (* 4 retrans, 8.1.1.1.2) if we are unable to seize it.
- * Hence, attempt re-scheduling PKT UL ASS (states SEND_ASS<->WAIT_ACK ping-pong)
- * until T3168 we announced to the MS expires:
+ * Hence, attempt re-scheduling PKT UL ASS (states SEND_ASS<->WAIT_ACK ping-pong) until T3168 we
+ * announced (SI13) to the MS expires:
*/
if (prev_state == TBF_UL_ASS_NONE) {
/* tbf_free() called upon trigger */

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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I8c963ba7421faf2b3dcab006ea629c4628d17b44
Gerrit-Change-Number: 33303
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged