aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/mm-broadband-modem.c28
-rw-r--r--src/mm-iface-modem-3gpp.c130
-rw-r--r--src/mm-iface-modem-3gpp.h24
-rw-r--r--src/mm-iface-modem.c127
-rw-r--r--src/mm-iface-modem.h24
5 files changed, 167 insertions, 166 deletions
diff --git a/src/mm-broadband-modem.c b/src/mm-broadband-modem.c
index 3e8d6e1b..920326d3 100644
--- a/src/mm-broadband-modem.c
+++ b/src/mm-broadband-modem.c
@@ -984,7 +984,7 @@ load_signal_quality (MMIfaceModem *self,
/* SETTING UP INDICATORS */
static gboolean
-setup_indicators_finish (MMIfaceModem *self,
+setup_indicators_finish (MMIfaceModem3gpp *self,
GAsyncResult *res,
GError **error)
{
@@ -1036,7 +1036,7 @@ setup_indicators_ready (MMBroadbandModem *self,
}
static void
-setup_indicators (MMIfaceModem *self,
+setup_indicators (MMIfaceModem3gpp *self,
GAsyncReadyCallback callback,
gpointer user_data)
{
@@ -1114,7 +1114,7 @@ unsolicited_events_context_complete_and_free (UnsolicitedEventsContext *ctx)
}
static gboolean
-unsolicited_events_finish (MMIfaceModem *self,
+unsolicited_events_finish (MMIfaceModem3gpp *self,
GAsyncResult *res,
GError **error)
{
@@ -1198,7 +1198,7 @@ unsolicited_events (UnsolicitedEventsContext *ctx)
}
static void
-enable_unsolicited_events (MMIfaceModem *self,
+enable_unsolicited_events (MMIfaceModem3gpp *self,
GAsyncReadyCallback callback,
gpointer user_data)
{
@@ -1217,9 +1217,9 @@ enable_unsolicited_events (MMIfaceModem *self,
}
static void
-disable_unsolicited_events (MMIfaceModem *self,
- GAsyncReadyCallback callback,
- gpointer user_data)
+disable_unsolicited_events (MMIfaceModem3gpp *self,
+ GAsyncReadyCallback callback,
+ gpointer user_data)
{
UnsolicitedEventsContext *ctx;
@@ -3351,14 +3351,6 @@ iface_modem_init (MMIfaceModem *iface)
iface->load_supported_charsets_finish = load_supported_charsets_finish;
iface->setup_charset = setup_charset;
iface->setup_charset_finish = setup_charset_finish;
- iface->setup_indicators = setup_indicators;
- iface->setup_indicators_finish = setup_indicators_finish;
- iface->enable_unsolicited_events = enable_unsolicited_events;
- iface->enable_unsolicited_events_finish = unsolicited_events_finish;
-
- /* Disabling steps */
- iface->disable_unsolicited_events = disable_unsolicited_events;
- iface->disable_unsolicited_events_finish = unsolicited_events_finish;
/* Additional actions */
iface->load_signal_quality = load_signal_quality;
@@ -3375,6 +3367,10 @@ iface_modem_3gpp_init (MMIfaceModem3gpp *iface)
iface->load_imei_finish = load_imei_finish;
/* Enabling steps */
+ iface->setup_indicators = setup_indicators;
+ iface->setup_indicators_finish = setup_indicators_finish;
+ iface->enable_unsolicited_events = enable_unsolicited_events;
+ iface->enable_unsolicited_events_finish = unsolicited_events_finish;
iface->setup_unsolicited_registration = setup_unsolicited_registration;
iface->setup_unsolicited_registration_finish = setup_unsolicited_registration_finish;
iface->setup_cs_registration = setup_cs_registration;
@@ -3383,6 +3379,8 @@ iface_modem_3gpp_init (MMIfaceModem3gpp *iface)
iface->setup_ps_registration_finish = setup_ps_registration_finish;
/* Disabling steps */
+ iface->disable_unsolicited_events = disable_unsolicited_events;
+ iface->disable_unsolicited_events_finish = unsolicited_events_finish;
iface->cleanup_unsolicited_registration = cleanup_unsolicited_registration;
iface->cleanup_unsolicited_registration_finish = cleanup_unsolicited_registration_finish;
iface->cleanup_cs_registration = cleanup_cs_registration;
diff --git a/src/mm-iface-modem-3gpp.c b/src/mm-iface-modem-3gpp.c
index ac12bbdd..30b05935 100644
--- a/src/mm-iface-modem-3gpp.c
+++ b/src/mm-iface-modem-3gpp.c
@@ -24,6 +24,12 @@
#include "mm-modem-helpers.h"
#include "mm-log.h"
+#define INDICATORS_CHECKED_TAG "indicators-checked-tag"
+#define UNSOLICITED_EVENTS_SUPPORTED_TAG "unsolicited-events-supported-tag"
+
+static GQuark indicators_checked_quark;
+static GQuark unsolicited_events_supported_quark;
+
/*****************************************************************************/
void
@@ -900,6 +906,7 @@ typedef enum {
DISABLING_STEP_CLEANUP_PS_REGISTRATION,
DISABLING_STEP_CLEANUP_CS_REGISTRATION,
DISABLING_STEP_CLEANUP_UNSOLICITED_REGISTRATION,
+ DISABLING_STEP_DISABLE_UNSOLICITED_EVENTS,
DISABLING_STEP_LAST
} DisablingStep;
@@ -997,6 +1004,8 @@ VOID_REPLY_READY_FN (cleanup_ps_registration,
"cleanup PS registration")
VOID_REPLY_READY_FN (cleanup_cs_registration,
"cleanup CS registration")
+VOID_REPLY_READY_FN (disable_unsolicited_events,
+ "disable unsolicited events")
static void
interface_disabling_step (DisablingContext *ctx)
@@ -1053,6 +1062,26 @@ interface_disabling_step (DisablingContext *ctx)
/* Fall down to next step */
ctx->step++;
+ case DISABLING_STEP_DISABLE_UNSOLICITED_EVENTS:
+ if (G_UNLIKELY (!unsolicited_events_supported_quark))
+ unsolicited_events_supported_quark = (g_quark_from_static_string (
+ UNSOLICITED_EVENTS_SUPPORTED_TAG));
+
+ /* Only try to disable if supported */
+ if (GPOINTER_TO_UINT (g_object_get_qdata (G_OBJECT (ctx->self),
+ unsolicited_events_supported_quark))) {
+ if (MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->disable_unsolicited_events &&
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->disable_unsolicited_events_finish) {
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->disable_unsolicited_events (
+ ctx->self,
+ (GAsyncReadyCallback)disable_unsolicited_events_ready,
+ ctx);
+ return;
+ }
+ }
+ /* Fall down to next step */
+ ctx->step++;
+
case DISABLING_STEP_LAST:
/* We are done without errors! */
g_simple_async_result_set_op_res_gboolean (ctx->result, TRUE);
@@ -1080,6 +1109,8 @@ static void interface_enabling_step (EnablingContext *ctx);
typedef enum {
ENABLING_STEP_FIRST,
+ ENABLING_STEP_SETUP_INDICATORS,
+ ENABLING_STEP_ENABLE_UNSOLICITED_EVENTS,
ENABLING_STEP_SETUP_UNSOLICITED_REGISTRATION,
ENABLING_STEP_SETUP_CS_REGISTRATION,
ENABLING_STEP_SETUP_PS_REGISTRATION,
@@ -1159,6 +1190,60 @@ mm_iface_modem_3gpp_enable_finish (MMIfaceModem3gpp *self,
}
static void
+setup_indicators_ready (MMIfaceModem3gpp *self,
+ GAsyncResult *res,
+ EnablingContext *ctx)
+{
+ GError *error = NULL;
+
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (self)->setup_indicators_finish (self, res, &error);
+ if (error) {
+ /* This error shouldn't be treated as critical */
+ mm_dbg ("Indicator control setup failed: '%s'", error->message);
+ g_error_free (error);
+
+ /* If we get an error setting up indicators, don't even bother trying to
+ * enable unsolicited events. */
+ ctx->step = ENABLING_STEP_ENABLE_UNSOLICITED_EVENTS + 1;
+ interface_enabling_step (ctx);
+ return;
+ }
+
+ /* Indicators setup, so assume we support unsolicited events */
+ g_object_set_qdata (G_OBJECT (self),
+ unsolicited_events_supported_quark,
+ GUINT_TO_POINTER (TRUE));
+
+ /* Go on to next step */
+ ctx->step++;
+ interface_enabling_step (ctx);
+}
+
+static void
+enable_unsolicited_events_ready (MMIfaceModem3gpp *self,
+ GAsyncResult *res,
+ EnablingContext *ctx)
+{
+ GError *error = NULL;
+
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (self)->enable_unsolicited_events_finish (self, res, &error);
+ if (error) {
+ /* This error shouldn't be treated as critical */
+ mm_dbg ("Enabling unsolicited events failed: '%s'", error->message);
+ g_error_free (error);
+
+ /* Reset support flag */
+ g_object_set_qdata (G_OBJECT (self),
+ unsolicited_events_supported_quark,
+ GUINT_TO_POINTER (FALSE));
+ }
+
+ /* Go on to next step */
+ ctx->step++;
+ interface_enabling_step (ctx);
+}
+
+static void
setup_cs_registration_ready (MMIfaceModem3gpp *self,
GAsyncResult *res,
EnablingContext *ctx)
@@ -1226,6 +1311,51 @@ interface_enabling_step (EnablingContext *ctx)
{
switch (ctx->step) {
case ENABLING_STEP_FIRST:
+ /* Setup quarks if we didn't do it before */
+ if (G_UNLIKELY (!indicators_checked_quark))
+ indicators_checked_quark = (g_quark_from_static_string (
+ INDICATORS_CHECKED_TAG));
+ if (G_UNLIKELY (!unsolicited_events_supported_quark))
+ unsolicited_events_supported_quark = (g_quark_from_static_string (
+ UNSOLICITED_EVENTS_SUPPORTED_TAG));
+ /* Fall down to next step */
+ ctx->step++;
+
+ case ENABLING_STEP_SETUP_INDICATORS:
+ if (!GPOINTER_TO_UINT (g_object_get_qdata (G_OBJECT (ctx->self),
+ indicators_checked_quark))) {
+ /* Set the checked flag so that we don't run it again */
+ g_object_set_qdata (G_OBJECT (ctx->self),
+ indicators_checked_quark,
+ GUINT_TO_POINTER (TRUE));
+ /* Initially, assume we don't support unsolicited events */
+ g_object_set_qdata (G_OBJECT (ctx->self),
+ unsolicited_events_supported_quark,
+ GUINT_TO_POINTER (FALSE));
+ if (MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->setup_indicators &&
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->setup_indicators_finish) {
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->setup_indicators (
+ ctx->self,
+ (GAsyncReadyCallback)setup_indicators_ready,
+ ctx);
+ return;
+ }
+ }
+ /* Fall down to next step */
+ ctx->step++;
+
+ case ENABLING_STEP_ENABLE_UNSOLICITED_EVENTS:
+ if (!GPOINTER_TO_UINT (g_object_get_qdata (G_OBJECT (ctx->self),
+ unsolicited_events_supported_quark))) {
+ if (MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->enable_unsolicited_events &&
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->enable_unsolicited_events_finish) {
+ MM_IFACE_MODEM_3GPP_GET_INTERFACE (ctx->self)->enable_unsolicited_events (
+ ctx->self,
+ (GAsyncReadyCallback)enable_unsolicited_events_ready,
+ ctx);
+ return;
+ }
+ }
/* Fall down to next step */
ctx->step++;
diff --git a/src/mm-iface-modem-3gpp.h b/src/mm-iface-modem-3gpp.h
index 52a4d45d..35f1194b 100644
--- a/src/mm-iface-modem-3gpp.h
+++ b/src/mm-iface-modem-3gpp.h
@@ -44,6 +44,30 @@ struct _MMIfaceModem3gpp {
GAsyncResult *res,
GError **error);
+ /* Asynchronous setup of indicators */
+ void (*setup_indicators) (MMIfaceModem3gpp *self,
+ GAsyncReadyCallback callback,
+ gpointer user_data);
+ gboolean (*setup_indicators_finish) (MMIfaceModem3gpp *self,
+ GAsyncResult *res,
+ GError **error);
+
+ /* Asynchronous enabling of unsolicited events */
+ void (*enable_unsolicited_events) (MMIfaceModem3gpp *self,
+ GAsyncReadyCallback callback,
+ gpointer user_data);
+ gboolean (*enable_unsolicited_events_finish) (MMIfaceModem3gpp *self,
+ GAsyncResult *res,
+ GError **error);
+
+ /* Asynchronous disabling of unsolicited events */
+ void (*disable_unsolicited_events) (MMIfaceModem3gpp *self,
+ GAsyncReadyCallback callback,
+ gpointer user_data);
+ gboolean (*disable_unsolicited_events_finish) (MMIfaceModem3gpp *self,
+ GAsyncResult *res,
+ GError **error);
+
/* Setup unsolicited registration messages */
void (* setup_unsolicited_registration) (MMIfaceModem3gpp *self,
GAsyncReadyCallback callback,
diff --git a/src/mm-iface-modem.c b/src/mm-iface-modem.c
index 88128539..27970e7d 100644
--- a/src/mm-iface-modem.c
+++ b/src/mm-iface-modem.c
@@ -27,13 +27,9 @@
#define SIGNAL_QUALITY_RECENT_TIMEOUT_SEC 60
#define SIGNAL_QUALITY_CHECK_TIMEOUT_SEC 30
-#define INDICATORS_CHECKED_TAG "indicators-checked-tag"
-#define UNSOLICITED_EVENTS_SUPPORTED_TAG "unsolicited-events-supported-tag"
#define SIGNAL_QUALITY_UPDATE_CONTEXT_TAG "signal-quality-update-context-tag"
#define SIGNAL_QUALITY_CHECK_CONTEXT_TAG "signal-quality-check-context-tag"
-static GQuark indicators_checked_quark;
-static GQuark unsolicited_events_supported_quark;
static GQuark signal_quality_update_context_quark;
static GQuark signal_quality_check_context_quark;
@@ -1580,7 +1576,6 @@ static void interface_disabling_step (DisablingContext *ctx);
typedef enum {
DISABLING_STEP_FIRST,
- DISABLING_STEP_DISABLE_UNSOLICITED_EVENTS,
DISABLING_STEP_MODEM_POWER_DOWN,
DISABLING_STEP_CLOSE_PORTS,
DISABLING_STEP_LAST
@@ -1687,7 +1682,6 @@ mm_iface_modem_disable_finish (MMIfaceModem *self,
interface_disabling_step (ctx); \
}
-VOID_REPLY_READY_FN (disable_unsolicited_events)
VOID_REPLY_READY_FN (modem_power_down)
static void
@@ -1698,26 +1692,6 @@ interface_disabling_step (DisablingContext *ctx)
/* Fall down to next step */
ctx->step++;
- case DISABLING_STEP_DISABLE_UNSOLICITED_EVENTS:
- if (G_UNLIKELY (!unsolicited_events_supported_quark))
- unsolicited_events_supported_quark = (g_quark_from_static_string (
- UNSOLICITED_EVENTS_SUPPORTED_TAG));
-
- /* Only try to disable if supported */
- if (GPOINTER_TO_UINT (g_object_get_qdata (G_OBJECT (ctx->self),
- unsolicited_events_supported_quark))) {
- if (MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->disable_unsolicited_events &&
- MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->disable_unsolicited_events_finish) {
- MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->disable_unsolicited_events (
- ctx->self,
- (GAsyncReadyCallback)disable_unsolicited_events_ready,
- ctx);
- return;
- }
- }
- /* Fall down to next step */
- ctx->step++;
-
case DISABLING_STEP_MODEM_POWER_DOWN:
/* CFUN=0 is dangerous and often will shoot devices in the head (that's
* what it's supposed to do). So don't use CFUN=0 by default, but let
@@ -1789,8 +1763,6 @@ typedef enum {
ENABLING_STEP_FLOW_CONTROL,
ENABLING_STEP_SUPPORTED_CHARSETS,
ENABLING_STEP_CHARSET,
- ENABLING_STEP_SETUP_INDICATORS,
- ENABLING_STEP_ENABLE_UNSOLICITED_EVENTS,
ENABLING_STEP_LAST
} EnablingStep;
@@ -1915,60 +1887,6 @@ VOID_REPLY_READY_FN (modem_after_power_up);
VOID_REPLY_READY_FN (setup_flow_control);
static void
-setup_indicators_ready (MMIfaceModem *self,
- GAsyncResult *res,
- EnablingContext *ctx)
-{
- GError *error = NULL;
-
- MM_IFACE_MODEM_GET_INTERFACE (self)->setup_indicators_finish (self, res, &error);
- if (error) {
- /* This error shouldn't be treated as critical */
- mm_dbg ("Indicator control setup failed: '%s'", error->message);
- g_error_free (error);
-
- /* If we get an error setting up indicators, don't even bother trying to
- * enable unsolicited events. */
- ctx->step = ENABLING_STEP_ENABLE_UNSOLICITED_EVENTS + 1;
- interface_enabling_step (ctx);
- return;
- }
-
- /* Indicators setup, so assume we support unsolicited events */
- g_object_set_qdata (G_OBJECT (self),
- unsolicited_events_supported_quark,
- GUINT_TO_POINTER (TRUE));
-
- /* Go on to next step */
- ctx->step++;
- interface_enabling_step (ctx);
-}
-
-static void
-enable_unsolicited_events_ready (MMIfaceModem *self,
- GAsyncResult *res,
- EnablingContext *ctx)
-{
- GError *error = NULL;
-
- MM_IFACE_MODEM_GET_INTERFACE (self)->enable_unsolicited_events_finish (self, res, &error);
- if (error) {
- /* This error shouldn't be treated as critical */
- mm_dbg ("Enabling unsolicited events failed: '%s'", error->message);
- g_error_free (error);
-
- /* Reset support flag */
- g_object_set_qdata (G_OBJECT (self),
- unsolicited_events_supported_quark,
- GUINT_TO_POINTER (FALSE));
- }
-
- /* Go on to next step */
- ctx->step++;
- interface_enabling_step (ctx);
-}
-
-static void
load_supported_charsets_ready (MMIfaceModem *self,
GAsyncResult *res,
EnablingContext *ctx)
@@ -2040,13 +1958,6 @@ interface_enabling_step (EnablingContext *ctx)
{
switch (ctx->step) {
case ENABLING_STEP_FIRST:
- /* Setup quarks if we didn't do it before */
- if (G_UNLIKELY (!indicators_checked_quark))
- indicators_checked_quark = (g_quark_from_static_string (
- INDICATORS_CHECKED_TAG));
- if (G_UNLIKELY (!unsolicited_events_supported_quark))
- unsolicited_events_supported_quark = (g_quark_from_static_string (
- UNSOLICITED_EVENTS_SUPPORTED_TAG));
/* Fall down to next step */
ctx->step++;
@@ -2195,44 +2106,6 @@ interface_enabling_step (EnablingContext *ctx)
/* Fall down to next step */
ctx->step++;
- case ENABLING_STEP_SETUP_INDICATORS:
- if (!GPOINTER_TO_UINT (g_object_get_qdata (G_OBJECT (ctx->self),
- indicators_checked_quark))) {
- /* Set the checked flag so that we don't run it again */
- g_object_set_qdata (G_OBJECT (ctx->self),
- indicators_checked_quark,
- GUINT_TO_POINTER (TRUE));
- /* Initially, assume we don't support unsolicited events */
- g_object_set_qdata (G_OBJECT (ctx->self),
- unsolicited_events_supported_quark,
- GUINT_TO_POINTER (FALSE));
- if (MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->setup_indicators &&
- MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->setup_indicators_finish) {
- MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->setup_indicators (
- ctx->self,
- (GAsyncReadyCallback)setup_indicators_ready,
- ctx);
- return;
- }
- }
- /* Fall down to next step */
- ctx->step++;
-
- case ENABLING_STEP_ENABLE_UNSOLICITED_EVENTS:
- if (!GPOINTER_TO_UINT (g_object_get_qdata (G_OBJECT (ctx->self),
- unsolicited_events_supported_quark))) {
- if (MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->enable_unsolicited_events &&
- MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->enable_unsolicited_events_finish) {
- MM_IFACE_MODEM_GET_INTERFACE (ctx->self)->enable_unsolicited_events (
- ctx->self,
- (GAsyncReadyCallback)enable_unsolicited_events_ready,
- ctx);
- return;
- }
- }
- /* Fall down to next step */
- ctx->step++;
-
case ENABLING_STEP_LAST:
/* We are done without errors! */
g_simple_async_result_set_op_res_gboolean (ctx->result, TRUE);
diff --git a/src/mm-iface-modem.h b/src/mm-iface-modem.h
index 3aba24c9..cd25a6bb 100644
--- a/src/mm-iface-modem.h
+++ b/src/mm-iface-modem.h
@@ -222,30 +222,6 @@ struct _MMIfaceModem {
GAsyncResult *res,
GError **error);
- /* Asynchronous setup of indicators */
- void (*setup_indicators) (MMIfaceModem *self,
- GAsyncReadyCallback callback,
- gpointer user_data);
- gboolean (*setup_indicators_finish) (MMIfaceModem *self,
- GAsyncResult *res,
- GError **error);
-
- /* Asynchronous enabling of unsolicited events */
- void (*enable_unsolicited_events) (MMIfaceModem *self,
- GAsyncReadyCallback callback,
- gpointer user_data);
- gboolean (*enable_unsolicited_events_finish) (MMIfaceModem *self,
- GAsyncResult *res,
- GError **error);
-
- /* Asynchronous disabling of unsolicited events */
- void (*disable_unsolicited_events) (MMIfaceModem *self,
- GAsyncReadyCallback callback,
- gpointer user_data);
- gboolean (*disable_unsolicited_events_finish) (MMIfaceModem *self,
- GAsyncResult *res,
- GError **error);
-
/* Asynchronous modem power-down operation */
void (*modem_power_down) (MMIfaceModem *self,
GAsyncReadyCallback callback,