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

shared-qmi: make refresh all non-fatal during SIM hot swap setup

Browse Source 
If for any reason "Refresh all" fails, don't return an error, as we
may have already configured SIM slot status correctly.
This commit is contained in:
Aleksander Morgado
2022-07-16 14:43:56 +02:00
committed by Aleksander Morgado
parent d2bc73f394
commit 21d24657eb
1 changed files with 7 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/mm-shared-qmi.c
View File

@@ -3971,35 +3971,23 @@ uim_refresh_register_all_ready (QmiClientUim *client,
g_autoptr(QmiMessageUimRefreshRegisterAllOutput) output = NULL;
g_autoptr(GError) error = NULL;
MMIfaceModem *self;
Private *priv;
SetupSimHotSwapContext *ctx;
self = g_task_get_source_object (task);
priv = get_private (MM_SHARED_QMI (self));
ctx = g_task_get_task_data (task);
output = qmi_client_uim_refresh_register_all_finish (client, res, &error);
if (!output || !qmi_message_uim_refresh_register_all_output_get_result (output, &error)) {
if (g_error_matches (error, QMI_PROTOCOL_ERROR, QMI_PROTOCOL_ERROR_NOT_SUPPORTED) ||
g_error_matches (error, QMI_PROTOCOL_ERROR, QMI_PROTOCOL_ERROR_INVALID_QMI_COMMAND)) {
/* As last resort, if 'refresh register all' fails, try a plain 'refresh register'.
* Some older modems may not support 'refresh register all'. */
ctx->step++;
setup_sim_hot_swap_step (task);
return;
}
mm_obj_dbg (self, "refresh register all operation failed: %s", error->message);
g_clear_object (&priv->uim_client);
g_task_return_error (task, g_steal_pointer (&error));
g_object_unref (task);
return;
ctx->step++;
} else {
/* Jump to setup refresh indication signal */
mm_obj_dbg (self, "registered for all SIM refresh events");
ctx->step = SETUP_SIM_HOT_SWAP_STEP_UIM_REFRESH_INDICATION;
}
mm_obj_dbg (self, "registered for all SIM refresh events");
/* Jump to setup refresh indication signal */
ctx->step = SETUP_SIM_HOT_SWAP_STEP_UIM_REFRESH_INDICATION;
setup_sim_hot_swap_step (task);
}