Attention is currently required from: dexter. pespin 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: Yes, my quick understanding is that the flag is used to signal "direct phy". We can make sure that's the case by checking when osmo-bts sets that flag (eg based on whether it is VTY-configured to use direct phy or not). IMHO we should add flags or configurations or whatever to describe per-bts which type it is so that osmo-pcu knows what to do. something like an enum with 1 field per l1if type: { L1IF_ALL_PCUIF, L1IF_DIRECT_SYSMO, L1IF_DIRECT_LC15, L1IF_DIRECT_O2CG, L1IF_DIRECT_RBS, } To start with, perhaps simply rename s/PCU_IF_FLAG_SYSMO/PCU_IF_FLAG_DIRECT_PHY/ if that makes sense. In any case, now that we have these "if (fl1h)" conditional blocks, I wonder whether it really makes sense to have the #ifef ENABLE_DIRECT_PHY around there. It may make sense to get rid of them in the common paths and simply make the check at runtime. -- 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: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 28 Sep 2023 17:47:43 +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