Change in osmocom-bb[master]: trxcon: trx_if_open(): allocate trx->fi as a child of trxcon->fi

fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/28675 ) Change subject: trxcon: trx_if_open(): allocate trx->fi as a child of trxcon->fi ...................................................................... trxcon: trx_if_open(): allocate trx->fi as a child of trxcon->fi Change-Id: I19579d2a76c38ca972a4ca132a8e5e3bba876c67 --- M src/host/trxcon/src/trx_if.c 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/src/host/trxcon/src/trx_if.c b/src/host/trxcon/src/trx_if.c index 40247ef..dc0c0dc 100644 --- a/src/host/trxcon/src/trx_if.c +++ b/src/host/trxcon/src/trx_if.c @@ -710,9 +710,7 @@ } /* Allocate a new dedicated state machine */ - /* TODO: allocate it as a child of trxcon->fi */ - trx->fi = osmo_fsm_inst_alloc(&trx_fsm, trx, - NULL, LOGL_DEBUG, "trx_interface"); + trx->fi = osmo_fsm_inst_alloc_child(&trx_fsm, trxcon->fi, TRX_EVENT_OFFLINE); if (trx->fi == NULL) { LOGP(DTRX, LOGL_ERROR, "Failed to allocate an instance " "of FSM '%s'\n", trx_fsm.name); 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/28675 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I19579d2a76c38ca972a4ca132a8e5e3bba876c67 Gerrit-Change-Number: 28675 Gerrit-PatchSet: 3 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged