Attention is currently required from: dexter.

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/34105 )

Change subject: pcuif_proto: remove unnecessary members from 
gsm_pcu_if_data_cnf_dt
......................................................................


Patch Set 1:

(1 comment)

Patchset:

PS1:
I'm not sure if the 'msg_id' is really sufficient.  AFAIR, Ericsson RBS 
actually use the TLLI in their confirmation.  And if there are situations where 
the TLLI itself is not unique, the (fn, tlli) tuple is certainly unique?



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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I779605858648e2a1c202c37e197a6e32e6ea3786
Gerrit-Change-Number: 34105
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Wed, 09 Aug 2023 14:52:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to