From 521349d5fcd5d4b52d86a7c827cc12309f7c2080 Mon Sep 17 00:00:00 2001 From: Florian Eckert Date: Wed, 7 Feb 2024 16:10:08 +0100 Subject: [PATCH] broadband-modem-qmi: add debug messages on operator description set With the current implementation it was not possible to find out which operator description is now exactly used. By adding a log message, it is now clear which operator description is now set. Signed-off-by: Florian Eckert --- src/mm-broadband-modem-qmi.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/src/mm-broadband-modem-qmi.c b/src/mm-broadband-modem-qmi.c index 0f31e3c3..d2c6d5cd 100644 --- a/src/mm-broadband-modem-qmi.c +++ b/src/mm-broadband-modem-qmi.c @@ -3467,10 +3467,18 @@ get_plmn_name_ready (QmiClientNas *client, mm_obj_dbg (self, "current operator short name: %s", short_name); mm_obj_dbg (self, "current operator service name: %s", service_name); if (!self->priv->current_operator_description) { - self->priv->current_operator_description = (service_name ? g_steal_pointer (&service_name) : - (long_name ? g_steal_pointer (&long_name) : - (short_name ? g_steal_pointer (&short_name) : - NULL))); + if (service_name) { + mm_obj_dbg (self, "setting operator description to service name: %s", service_name); + self->priv->current_operator_description = g_steal_pointer (&service_name); + } else if (long_name) { + mm_obj_dbg (self, "setting operator description to long name: %s", long_name); + self->priv->current_operator_description = g_steal_pointer (&long_name); + } else if (short_name) { + mm_obj_dbg (self, "setting operator description to short name: %s", short_name); + self->priv->current_operator_description = g_steal_pointer (&short_name); + } else { + mm_obj_dbg (self, "unable to set operator description"); + } } }