Attention is currently required from: laforge, fixeria.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmocom-bb/+/32575 )
Change subject: trxcon: get rid of the timer driven clock module
......................................................................
Patch Set 2:
(3 comments)
File src/host/trxcon/src/trx_if.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/32575/comment/2e884004_1a375bc8
PS2, Line 707: .fn = bi.fn,
the trx_fn_advance may still make sense to be possible to apply it here?
https://gerrit.osmocom.org/c/osmocom-bb/+/32575/comment/eeb00c21_8e774498
PS2, Line 711: trxcon_phyif_handle_rts_ind(trx->priv, &rts);
does that mean that now if we don't receive DL data, we'll never to UL RTS.ind? Is
that supposed to work because the BTS is expected to always send data?
File src/host/trxcon/src/trxcon_main.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/32575/comment/b5451a18_8e6582d1
PS2, Line 244: XXX: no-op
TODO: print a no-op warning
Ack
--
To view, visit
https://gerrit.osmocom.org/c/osmocom-bb/+/32575
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ic8a5b6277c6b16392026e0557376257d71c9d230
Gerrit-Change-Number: 32575
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 01 Jun 2023 16:43:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment