pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/32272 )

Change subject: bts-sysmo: Delay marking phy_link as connected until L1 reset + got info ......................................................................

bts-sysmo: Delay marking phy_link as connected until L1 reset + got info

Better wait until the DSP is reset and all the information is retrieved before telling upper layers that the phy is ready. Keep it in CONNECTING state meanwhile.

Change-Id: Ifdc791336fb8efd42f4428893f687093085af129 --- M src/osmo-bts-sysmo/l1_if.c 1 file changed, 15 insertions(+), 2 deletions(-)

Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve fixeria: Looks good to me, approved

diff --git a/src/osmo-bts-sysmo/l1_if.c b/src/osmo-bts-sysmo/l1_if.c index 59a168f..9a0b19f 100644 --- a/src/osmo-bts-sysmo/l1_if.c +++ b/src/osmo-bts-sysmo/l1_if.c @@ -1515,6 +1515,8 @@

msgb_free(resp);

+ phy_link_state_set(trx->pinst->phy_link, PHY_LINK_CONNECTED); + /* FIXME: clock related */ return 0; } @@ -1917,7 +1919,5 @@ hdl = pinst->u.sysmobts.hdl; osmo_strlcpy(bts->sub_model, sysmobts_model(hdl->hw_info.model_nr, hdl->hw_info.trx_nr), sizeof(bts->sub_model));

- phy_link_state_set(plink, PHY_LINK_CONNECTED); - return 0; }