Attention is currently required from: dexter.

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

Change subject: PCUIF: Add support for PCU_IF_SAPI_AGCH_DT
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

Patchset:

PS1:
I think it doesn't make sense to start adding stuff for the AGCH API until we 
really solve in osmo-pcu the way to submit a seq_nr and store messages in a 
queue to be able to match them back, all that without using a TLLI because in 
AGCH the Assignment may not have a TLLI.

Better focus first on merging the tlli->msg_id and _dt->_2 patches to have a 
cleaned PCUIF API, then we can think about adding the osmo-pcu internals (I can 
help with that) and finally having a new AGCH SAPI.
We can even think about adding the new AGCH API later on after v11 (and have a 
v12) if time is tight (we are releasing soon).
The AGCH API is not urgent since functionally we already forward to AGCH in 
osmo-bts/bsc if PCH tlli=0xffffffff.



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

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I40e05a2e68cca77d3c2f41df9af8d35762488abf
Gerrit-Change-Number: 34066
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Thu, 10 Aug 2023 13:47:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Reply via email to