From 1ffd693f54ce71ec357b1f574d8340d66fd7bcd9 Mon Sep 17 00:00:00 2001 From: Aleksander Morgado Date: Fri, 17 Dec 2021 11:09:14 +0100 Subject: iface-modem-simple: fix connection attempt start place The 'ENABLE + 1' step is 'WAIT_FOR_ENABLED' really, and that should be only used when the current state is not a final one. If we're already enabled, or registered, or connected, then jump to 'REGISTER' right away. --- src/mm-iface-modem-simple.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'src/mm-iface-modem-simple.c') diff --git a/src/mm-iface-modem-simple.c b/src/mm-iface-modem-simple.c index 8a30a316..57ee18ed 100644 --- a/src/mm-iface-modem-simple.c +++ b/src/mm-iface-modem-simple.c @@ -765,17 +765,18 @@ connect_auth_ready (MMBaseModem *self, case MM_MODEM_STATE_ENABLING: case MM_MODEM_STATE_DISCONNECTING: - /* If we are transitioning to the ENABLED/REGISTERED state, - * wait to get there before going on */ + case MM_MODEM_STATE_SEARCHING: + case MM_MODEM_STATE_CONNECTING: + /* Wait to get to a final state before going on */ ctx->step = CONNECTION_STEP_WAIT_FOR_ENABLED; break; case MM_MODEM_STATE_ENABLED: - case MM_MODEM_STATE_SEARCHING: case MM_MODEM_STATE_REGISTERED: - case MM_MODEM_STATE_CONNECTING: case MM_MODEM_STATE_CONNECTED: - ctx->step = CONNECTION_STEP_ENABLE + 1; + /* If we are at least already enabled, start at the registration check + * right away */ + ctx->step = CONNECTION_STEP_REGISTER; break; default: -- cgit v1.2.3-70-g09d2