fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30954 ) Change subject: modem: fix modem_tx_chan_req(): check if SI13 was received ...................................................................... modem: fix modem_tx_chan_req(): check if SI13 was received There was no need for this check because the channel establishment logic was at the end of handle_si13(), so I overlooked this. Change-Id: I829948de325461ab5d7e950493497a7537ba06ac Fixes: 8a506dcd4e2736aac2bf30d1784f0e1f0c3a9d00 Related: OS#5500 --- M src/host/layer23/src/modem/app_modem.c 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved diff --git a/src/host/layer23/src/modem/app_modem.c b/src/host/layer23/src/modem/app_modem.c index 8af3243..ad05603 100644 --- a/src/host/layer23/src/modem/app_modem.c +++ b/src/host/layer23/src/modem/app_modem.c @@ -73,8 +73,8 @@ OSMO_ASSERT(rr->state == GSM48_RR_ST_IDLE); - if (!app_data.si.si1) - return -EBUSY; + if (!app_data.si.si1 || !app_data.si.si13) + return -EAGAIN; if (!app_data.si.gprs.supported) return -ENOTSUP; -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30954 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I829948de325461ab5d7e950493497a7537ba06ac Gerrit-Change-Number: 30954 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged