Attention is currently required from: pespin.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/33457 )
Change subject: osmo-bts-sysmo: mute PHY until OML is ready ......................................................................
Patch Set 8:
(2 comments)
File src/osmo-bts-sysmo/l1_if.c:
https://gerrit.osmocom.org/c/osmo-bts/+/33457/comment/0d303118_06367600 PS7, Line 1271: oml_mo_rf_lock_chg(&trx->mo, fl1h->last_rf_mute, 0);
I meant call to the oml_mo_rf_lock_chg() function, which also happens in the success case. […]
I've verified that it sends a Change Administrative State Ack/Nack. Updated the patch to send it only after opstart_success, and verified that it behaves as expected.
File src/osmo-bts-sysmo/l1_if.c:
https://gerrit.osmocom.org/c/osmo-bts/+/33457/comment/846a6fe7_23ffd135 PS8, Line 1342: int trx_rf_lock(struct gsm_bts_trx *trx, int locked, l1if_compl_cb *cb); Done
-
osmith