Attention is currently required from: fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/29881 ) Change subject: bts: bts_snd_dl_ass(): use GSM_TDMA_FN_SUM() macro ...................................................................... Patch Set 1: Code-Review+2 (1 comment) File src/bts.cpp: https://gerrit.osmocom.org/c/osmo-pcu/+/29881/comment/5639ca50_c77f6d9a PS1, Line 1096: GSM_TDMA_FN_SUM(tbf->pdch[ts_no]->last_rts_fn, 21216), you could even move this to a variable in a previous line to untangle a bit the call this function (write_immediate_assignment). -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/29881 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: If363e3320372c798b9b16274327fa8041358e350 Gerrit-Change-Number: 29881 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Wed, 26 Oct 2022 13:42:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment