From 051b15cca89670d943459f734e20117ca1b8afcf Mon Sep 17 00:00:00 2001 From: Ben Chan Date: Wed, 16 May 2018 12:45:57 -0700 Subject: iface-modem-3gpp: ignore initial registration check result when enabling This patch extends commit c9e85b671 ("iface-modem-3gpp: ignore initial registration check result when appropriate") to handle the situation where a registration check update may prematurely transition the modem to the 'enabled' state while the modem is still being enabled. The issue is identified by Dan Williams . More details can be found in https://bugs.freedesktop.org/show_bug.cgi?id=106468 --- src/mm-iface-modem-3gpp.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'src') diff --git a/src/mm-iface-modem-3gpp.c b/src/mm-iface-modem-3gpp.c index 4ac32fd5..4e19815d 100644 --- a/src/mm-iface-modem-3gpp.c +++ b/src/mm-iface-modem-3gpp.c @@ -1332,17 +1332,17 @@ update_registration_state (MMIfaceModem3gpp *self, if (ctx->reloading_registration_info) return; - /* If the modem isn't already enabled or being enabled, this - * registration state update is due to a previously scheduled - * initial registration check when the modem was being enabled. - * We need to ignore it as otherwise it may cause an incorrect - * transition of the registration state and modem state when the - * modem is being disabled. */ + /* If the modem isn't already enabled, this registration state update + * could be due to a previously scheduled initial registration check + * when the modem was being enabled. We need to ignore it as otherwise + * it may cause an incorrect transition of the registration state and + * modem state when the modem is being disabled or still going through + * enable steps */ modem_state = MM_MODEM_STATE_UNKNOWN; g_object_get (self, MM_IFACE_MODEM_STATE, &modem_state, NULL); - if (modem_state < MM_MODEM_STATE_ENABLING) { + if (modem_state < MM_MODEM_STATE_ENABLED) { mm_dbg ("Modem %s: 3GPP Registration state change ignored as modem isn't enabled", g_dbus_object_get_object_path (G_DBUS_OBJECT (self))); return; -- cgit v1.2.3-70-g09d2