pespin 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:
> 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.
Fine with it. That was actually my first approach, but I decided to go for the
straightforward way of passing messages directly to have something working in a
faster way.
--
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 17:01:21 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
