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:

> Patch Set 2:
>
> shouldn't we have some kind of feature check before blindly assuming that we 
> get DATA.ind at every clock?  PCUIF version change? some feature flag? ...

See my previous comment above. If you still think it's a better idea to bump 
PCUIF then I'll do it.


--
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-CC: laforge <[email protected]>
Gerrit-Comment-Date: Mon, 08 Mar 2021 11:38:54 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to