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:

I am not sure if I understand the question correctly. […]

Ah I see it now, in common code: #ifdef ENABLE_DIRECT_PHY bts->trx[trx_nr].fl1h = l1if_open_trx( ... #endif

This is called though based on this condition: if ((info_ind->flags & PCU_IF_FLAG_SYSMO) && info_ind->trx[trx_nr].hlayer1) { Are you passing PCUIF_FLAG_SYSMO when using an RBS from within osmo-bsc? this looks wrong imho, that flag is SYSMO specific and doesn't relate to RBS? Or maybe we don't need anything SYSMO specific and we can rename it?

I envision at some point the PCU needs to know the type of BTS, so it knows how to hook to it on the l1if. And knowing if it's SYSMO or not is not enough imho.