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