Attention is currently required from: laforge, dexter.

pespin 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. […]
a msg_id assigned by PCU is certainly enough, since it's the PCU taking care of 
internally keeping some lookup table between that msg_id and whatever 
information it may deem necessary, be it a tlli, an fn, a pointer or whatever.
The exact point is that the actual implementation matching the PCUIF message 
ends up in osmo-pcu and can be changed/improved at any point as needed.



--
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-CC: pespin <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Wed, 09 Aug 2023 16:28:36 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Gerrit-MessageType: comment

Reply via email to