From 5b080915200d0ba5de28d50bb088eba0aef9b362 Mon Sep 17 00:00:00 2001 From: Aleksander Morgado Date: Mon, 3 Jun 2013 12:44:40 +0200 Subject: api,introspection: Simple.Connect() won't change neither bands nor allowed modes We won't allow changing modes or bands through Simple.Connect(). Applications should instead look at the corresponding SupportedBands or SupportedModes, and then use SetCurrentBands() or SetCurrentModes() explicitly. --- src/mm-iface-modem-simple.c | 160 -------------------------------------------- 1 file changed, 160 deletions(-) (limited to 'src/mm-iface-modem-simple.c') diff --git a/src/mm-iface-modem-simple.c b/src/mm-iface-modem-simple.c index 5d1e1b21..bb850a18 100644 --- a/src/mm-iface-modem-simple.c +++ b/src/mm-iface-modem-simple.c @@ -174,8 +174,6 @@ typedef enum { CONNECTION_STEP_WAIT_FOR_INITIALIZED, CONNECTION_STEP_ENABLE, CONNECTION_STEP_WAIT_FOR_ENABLED, - CONNECTION_STEP_CURRENT_MODES, - CONNECTION_STEP_CURRENT_BANDS, CONNECTION_STEP_REGISTER, CONNECTION_STEP_BEARER, CONNECTION_STEP_CONNECT, @@ -187,8 +185,6 @@ typedef struct { GDBusMethodInvocation *invocation; MMIfaceModemSimple *self; ConnectionStep step; - gulong state_changed_id; - guint state_changed_wait_id; /* Expected input properties */ GVariant *dictionary; @@ -201,9 +197,6 @@ typedef struct { static void connection_context_free (ConnectionContext *ctx) { - g_assert (ctx->state_changed_id == 0); - g_assert (ctx->state_changed_wait_id == 0); - g_variant_unref (ctx->dictionary); if (ctx->properties) g_object_unref (ctx->properties); @@ -274,82 +267,6 @@ register_in_3gpp_or_cdma_network_ready (MMIfaceModemSimple *self, connection_step (ctx); } -static gboolean -after_set_current_modes_timeout_cb (ConnectionContext *ctx) -{ - /* Allowed modes set... almost there! */ - ctx->step++; - connection_step (ctx); - return FALSE; -} - -static void -set_current_modes_ready (MMBaseModem *self, - GAsyncResult *res, - ConnectionContext *ctx) -{ - GError *error = NULL; - - if (!mm_iface_modem_set_current_modes_finish (MM_IFACE_MODEM (self), res, &error)) { - if (g_error_matches (error, - MM_CORE_ERROR, - MM_CORE_ERROR_UNSUPPORTED)) { - g_error_free (error); - /* If setting bands is unsupported, keep on without sleep */ - ctx->step++; - connection_step (ctx); - } else { - g_dbus_method_invocation_take_error (ctx->invocation, error); - connection_context_free (ctx); - } - return; - } - - /* Setting allowed modes will reset the current registration, so we'll need - * a couple of seconds to settle down. This sleep time just makes sure that - * the modem has enough time to report being unregistered. */ - mm_dbg ("Will wait to settle down after updating allowed modes"); - g_timeout_add_seconds (2, (GSourceFunc)after_set_current_modes_timeout_cb, ctx); -} - -static gboolean -after_set_current_bands_timeout_cb (ConnectionContext *ctx) -{ - /* Bands set... almost there! */ - ctx->step++; - connection_step (ctx); - return FALSE; -} - -static void -set_current_bands_ready (MMBaseModem *self, - GAsyncResult *res, - ConnectionContext *ctx) -{ - GError *error = NULL; - - if (!mm_iface_modem_set_current_bands_finish (MM_IFACE_MODEM (self), res, &error)) { - if (g_error_matches (error, - MM_CORE_ERROR, - MM_CORE_ERROR_UNSUPPORTED)) { - g_error_free (error); - /* If setting bands is unsupported, keep on without sleep */ - ctx->step++; - connection_step (ctx); - } else { - g_dbus_method_invocation_take_error (ctx->invocation, error); - connection_context_free (ctx); - } - return; - } - - /* Setting bands will reset the current registration, so we'll need a couple - * of seconds to settle down. This sleep time just makes sure that the modem - * has enough time to report being unregistered. */ - mm_dbg ("Will wait to settle down after updating bands"); - g_timeout_add_seconds (2, (GSourceFunc)after_set_current_bands_timeout_cb, ctx); -} - static void wait_for_enabled_ready (MMIfaceModem *self, GAsyncResult *res, @@ -556,59 +473,6 @@ connection_step (ConnectionContext *ctx) ctx); return; - case CONNECTION_STEP_CURRENT_MODES: { - MMModemMode allowed_modes = MM_MODEM_MODE_ANY; - MMModemMode preferred_mode = MM_MODEM_MODE_NONE; - - mm_info ("Simple connect state (%d/%d): Current modes", - ctx->step, CONNECTION_STEP_LAST); - - /* Don't set modes unless explicitly requested to do so */ - if (mm_simple_connect_properties_get_current_modes (ctx->properties, - &allowed_modes, - &preferred_mode)) { - mm_iface_modem_set_current_modes (MM_IFACE_MODEM (ctx->self), - allowed_modes, - preferred_mode, - (GAsyncReadyCallback)set_current_modes_ready, - ctx); - return; - } - - /* Fall down to next step */ - ctx->step++; - } - - case CONNECTION_STEP_CURRENT_BANDS: { - const MMModemBand *bands = NULL; - guint n_bands = 0; - - mm_info ("Simple connect state (%d/%d): Current Bands", - ctx->step, CONNECTION_STEP_LAST); - - /* Don't set bands unless explicitly requested to do so */ - if (mm_simple_connect_properties_get_current_bands (ctx->properties, &bands, &n_bands)) { - GArray *array; - guint i; - - if (bands && *bands) { - array = g_array_sized_new (FALSE, FALSE, sizeof (MMModemBand), n_bands); - for (i = 0; i < n_bands; i++) - g_array_insert_val (array, i, bands[i]); - - mm_iface_modem_set_current_bands (MM_IFACE_MODEM (ctx->self), - array, - (GAsyncReadyCallback)set_current_bands_ready, - ctx); - g_array_unref (array); - return; - } - } - - /* Fall down to next step */ - ctx->step++; - } - case CONNECTION_STEP_REGISTER: mm_info ("Simple connect state (%d/%d): Register", ctx->step, CONNECTION_STEP_LAST); @@ -778,10 +642,6 @@ connect_auth_ready (MMBaseModem *self, /* Log about all the parameters being used for the simple connect */ { - const MMModemBand *bands; - guint n_bands; - MMModemMode allowed; - MMModemMode preferred; MMBearerAllowedAuth allowed_auth; gchar *str; MMBearerIpFamily ip_family; @@ -790,25 +650,6 @@ connect_auth_ready (MMBaseModem *self, mm_dbg (" PIN: %s", VALIDATE_UNSPECIFIED (mm_simple_connect_properties_get_pin (ctx->properties))); - if (mm_simple_connect_properties_get_current_modes (ctx->properties, &allowed, &preferred)) { - str = mm_modem_mode_build_string_from_mask (allowed); - mm_dbg (" Allowed mode: %s", str); - g_free (str); - str = mm_modem_mode_build_string_from_mask (preferred); - mm_dbg (" Preferred mode: %s", str); - g_free (str); - } else { - mm_dbg (" Allowed mode: %s", VALIDATE_UNSPECIFIED (NULL)); - mm_dbg (" Preferred mode: %s", VALIDATE_UNSPECIFIED (NULL)); - } - - if (mm_simple_connect_properties_get_current_bands (ctx->properties, &bands, &n_bands)) { - str = mm_common_build_bands_string (bands, n_bands); - mm_dbg (" Bands: %s", str); - g_free (str); - } else - mm_dbg (" Bands: %s", VALIDATE_UNSPECIFIED (NULL)); - mm_dbg (" Operator ID: %s", VALIDATE_UNSPECIFIED (mm_simple_connect_properties_get_operator_id (ctx->properties))); mm_dbg (" Allowed roaming: %s", mm_simple_connect_properties_get_allow_roaming (ctx->properties) ? "yes" : "no"); @@ -831,7 +672,6 @@ connect_auth_ready (MMBaseModem *self, } else mm_dbg (" Allowed authentication: %s", VALIDATE_UNSPECIFIED (NULL)); - mm_dbg (" User: %s", VALIDATE_UNSPECIFIED (mm_simple_connect_properties_get_user (ctx->properties))); mm_dbg (" Password: %s", VALIDATE_UNSPECIFIED (mm_simple_connect_properties_get_password (ctx->properties))); -- cgit v1.2.3-70-g09d2