Attention is currently required from: pespin.
dexter has posted comments on this change. (
https://gerrit.osmocom.org/c/osmo-pcu/+/34575?usp=email )
Change subject: gprs_rlcmac_sched: check if we really use direct phy
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS1:
Ah I see it now, in common code: […]
Yes, I am
setting PCU_IF_FLAG_SYSMO in osmo-bsc. I do not fully understand why it even exists. The
API looks and feels the same for all BTS models and the common PCU code does not need
extra knowledge about what specific hardware it is operating on.
At least at the moment, where we can include only one specific hardware support during
compile time there is also no hardware support we could chose from.
My best guess is that PCU_IF_FLAG_SYSMO set means "use direct phy",
PCU_IF_FLAG_SYSMO not set means "do not use use direct phy, use PCUIF for
everything"
--
To view, visit
https://gerrit.osmocom.org/c/osmo-pcu/+/34575?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I0808950b1154bbb9a789c3f706ad9fb6618764ec
Gerrit-Change-Number: 34575
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 28 Sep 2023 17:16:02 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: dexter <pmaier(a)sysmocom.de>
Gerrit-MessageType: comment