laforge has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/24618 )
Change subject: pcuif_proto.h: Introduce anr_{req,cnf} messages
......................................................................
Patch Set 3:
I would suggest to somehow introduce an entirely new "transparent" channel from
the BSC. The way how you're doing it herre (flat next to the BTS originated
messages) looks like the BTS will still need to be updated every time some new
message between PCU and BSC is added.
I would simply introduce something like a BSC_TO_PCU_IND and PCU_TO_BSC_IND at
the msg_type level. At that point the BTS doesn't care anymore, it just passes
those messages back and forth between PCU and BSC. The payload is opaqua and
needs to be understood only by those two.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/24618
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I8deef11f9ec191475c99133629cd5cd9048c4bcb
Gerrit-Change-Number: 24618
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-CC: laforge <[email protected]>
Gerrit-Comment-Date: Tue, 15 Jun 2021 16:59:28 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
