Attention is currently required from: dexter.
pespin has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-bsc/+/31113
)
Change subject: pcuif_proto: add indication to communicate E1 parameters
......................................................................
Patch Set 1:
(4 comments)
Commit Message:
https://gerrit.osmocom.org/c/osmo-bsc/+/31113/comment/9b295746_0d1474b0
PS1, Line 10: setup. To avioud duplicate configuration the BSC has to communicate the
"avoid"
https://gerrit.osmocom.org/c/osmo-bsc/+/31113/comment/968f1de1_6913b767
PS1, Line 11: E1 parameters (which TS, SS etc.) to the PCU. Lets add a new primitive
What do you mean with "to avoid duplicate configuration" here? where would the
duplicate configuration be? what would the other way be?
File include/osmocom/bsc/pcuif_proto.h:
https://gerrit.osmocom.org/c/osmo-bsc/+/31113/comment/914f87e2_5196a3c6
PS1, Line 263: uint8_t pdch_trx_no;
this was not yet changed to trx_nr, why?
https://gerrit.osmocom.org/c/osmo-bsc/+/31113/comment/7de699c2_f0b4dd12
PS1, Line 264: uint8_t pdch_ts;
this was not yet changed to ts_nr, why?
--
To view, visit
https://gerrit.osmocom.org/c/osmo-bsc/+/31113
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ia7928489130c1205b06bb9b10de0fb1461843301
Gerrit-Change-Number: 31113
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 30 Jan 2023 17:42:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment