Attention is currently required from: dexter.

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    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.

To view, visit change 34575. To unsubscribe, or for help writing mail filters, visit settings.