pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/23261 )

Change subject: Track TDMA clock with DATA.ind instead of TIME.ind
......................................................................


Patch Set 2:

I think as of now the behavior stays mostly the same, since the 
pollController->expireTimedout() is still being used to trigger time outs, and 
in real world at most there will be extra delays to trigger the timeout due to 
some packets not arriving correctly due to bad signal. So nothing really 
critical imho.
I would like to avoid increasing and hence mantaing different PCUIF versions if 
not really needed due to ABI incompatibility.


--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/23261
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ibc495173119465e74f726ddc36e312334e6dc0fd
Gerrit-Change-Number: 23261
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <[email protected]>
Gerrit-Comment-Date: Mon, 08 Mar 2021 11:22:33 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to