fixeria has uploaded this change for review.

View Change

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(-)

git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/54/30954/1
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 change 30954. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I829948de325461ab5d7e950493497a7537ba06ac
Gerrit-Change-Number: 30954
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-MessageType: newchange