colin/ModemManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

iface-modem-simple: explicit short wait after enabling

Browse Source 
When we have just enabled, we want to give it some time before
starting the registration process, so that any pending registration
update that may have been scheduled during the enabling phase is
applied. We don't want to trigger a new automatic registration if
e.g. we're already registered.
This commit is contained in:
Aleksander Morgado
2021-12-17 11:16:54 +01:00
parent 1ffd693f54
commit 3fac73a113
1 changed files with 21 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/mm-iface-modem-simple.c
View File

@@ -215,6 +215,7 @@ typedef enum {
CONNECTION_STEP_WAIT_FOR_INITIALIZED,
CONNECTION_STEP_ENABLE,
CONNECTION_STEP_WAIT_FOR_ENABLED,
CONNECTION_STEP_WAIT_AFTER_ENABLED,
CONNECTION_STEP_REGISTER,
CONNECTION_STEP_BEARER,
CONNECTION_STEP_CONNECT,
@@ -331,6 +332,15 @@ register_in_3gpp_or_cdma_network_ready (MMIfaceModemSimple *self,
connection_step (ctx);
}
static gboolean
wait_after_enabled_ready (ConnectionContext *ctx)
{
/* Just go on now */
ctx->step++;
connection_step (ctx);
return G_SOURCE_REMOVE;
}
static void
wait_for_enabled_ready (MMIfaceModem *self,
GAsyncResult *res,
@@ -561,6 +571,17 @@ connection_step (ConnectionContext *ctx)
ctx);
return;
case CONNECTION_STEP_WAIT_AFTER_ENABLED:
mm_obj_info (ctx->self, "simple connect state (%d/%d): wait after enabled",
ctx->step, CONNECTION_STEP_LAST);
/* When we have just enabled, we want to give it some time before starting
* the registration process, so that any pending registration update that may
* have been scheduled during the enabling phase is applied. We don't want to
* trigger a new automatic registration if e.g. we're already registered. */
g_timeout_add (100, (GSourceFunc)wait_after_enabled_ready, ctx);
return;
case CONNECTION_STEP_REGISTER:
mm_obj_info (ctx->self, "simple connect state (%d/%d): register",
ctx->step, CONNECTION_STEP_LAST);