pespin has uploaded this change for review.

View Change

layer23: modem: Fix wrong 'radio not started' vty message

Change-Id: I8871c9d0e820beb75d7820b5a2d8d3c77dac379d
---
M src/host/layer23/src/common/vty.c
M src/host/layer23/src/modem/app_modem.c
2 files changed, 13 insertions(+), 1 deletion(-)

git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/20/32420/1

diff --git a/src/host/layer23/src/common/vty.c b/src/host/layer23/src/common/vty.c
index c88d88d..dc6e580 100644
--- a/src/host/layer23/src/common/vty.c
+++ b/src/host/layer23/src/common/vty.c
@@ -137,7 +137,9 @@
 
 	if (!ms->started)
 		service = ", radio is not started";
-	else if (ms->mmlayer.state == GSM48_MM_ST_MM_IDLE) {
+	else if (ms->mmlayer.state == GSM48_MM_ST_NULL) {
+		service = ", MM connection not yet set up";
+	} else if (ms->mmlayer.state == GSM48_MM_ST_MM_IDLE) {
 		/* current MM idle state */
 		switch (ms->mmlayer.substate) {
 		case GSM48_MM_SST_NORMAL_SERVICE:
diff --git a/src/host/layer23/src/modem/app_modem.c b/src/host/layer23/src/modem/app_modem.c
index 644c630..fec759c 100644
--- a/src/host/layer23/src/modem/app_modem.c
+++ b/src/host/layer23/src/modem/app_modem.c
@@ -126,6 +126,7 @@
 	case S_L1CTL_RESET:
 		ms = signal_data;
 		layer3_app_reset();
+		ms->started = true;
 		return l1ctl_tx_fbsb_req(ms, ms->test_arfcn,
 					 L1CTL_FBSB_F_FB01SB, 100, 0,
 					 CCCH_MODE_NONE, dbm2rxlev(-85));

To view, visit change 32420. To unsubscribe, or for help writing mail filters, visit settings.