diff options
| author | Forest Bond <forest.bond@rapidrollout.com> | 2013-03-28 09:46:29 -0400 |
|---|---|---|
| committer | Patrik Flykt <patrik.flykt@linux.intel.com> | 2013-04-03 11:11:12 +0300 |
| commit | 4c09e05f07a7777929e05552e659cae09cf5a4cb (patch) | |
| tree | dfa8330f79a475cb8d1478496a77a72e2d71463c /plugins | |
| parent | 21dc8816dc7970baf3871cbbaafb01752a0e25a7 (diff) | |
| download | connman-4c09e05f07a7777929e05552e659cae09cf5a4cb.tar.gz connman-4c09e05f07a7777929e05552e659cae09cf5a4cb.tar.bz2 connman-4c09e05f07a7777929e05552e659cae09cf5a4cb.zip | |
ofono: Add CDMA network when modem is registered
Connecting the network requires powering the CDMA connection manager,
which fails with an error if network registration is not complete. So
we only add the network when registration is complete and remove it when
the modem is no longer registered.
Diffstat (limited to 'plugins')
| -rw-r--r-- | plugins/ofono.c | 27 |
1 files changed, 16 insertions, 11 deletions
diff --git a/plugins/ofono.c b/plugins/ofono.c index 082c8b26..09718a4b 100644 --- a/plugins/ofono.c +++ b/plugins/ofono.c @@ -179,6 +179,7 @@ struct modem_data { char *name; uint8_t strength; uint8_t data_strength; /* 1xEVDO signal strength */ + connman_bool_t registered; connman_bool_t roaming; /* pending calls */ @@ -1457,7 +1458,7 @@ static void netreg_update_datastrength(struct modem_data *modem, connman_network_update(modem->network); } -static void netreg_update_roaming(struct modem_data *modem, +static void netreg_update_status(struct modem_data *modem, DBusMessageIter *value) { char *status; @@ -1465,10 +1466,8 @@ static void netreg_update_roaming(struct modem_data *modem, dbus_message_iter_get_basic(value, &status); - if (g_str_equal(status, "roaming") == TRUE) - roaming = TRUE; - else - roaming = FALSE; + roaming = g_str_equal(status, "roaming"); + modem->registered = roaming || g_str_equal(status, "registered"); if (roaming == modem->roaming) return; @@ -1532,7 +1531,7 @@ static gboolean netreg_changed(DBusConnection *conn, DBusMessage *message, else if (g_str_equal(key, "Strength") == TRUE) netreg_update_strength(modem, &value); else if (g_str_equal(key, "Status") == TRUE) - netreg_update_roaming(modem, &value); + netreg_update_status(modem, &value); else if (g_str_equal(key, "MobileCountryCode") == TRUE) netreg_update_regdom(modem, &value); @@ -1559,7 +1558,7 @@ static void netreg_properties_reply(struct modem_data *modem, else if (g_str_equal(key, "Strength") == TRUE) netreg_update_strength(modem, &value); else if (g_str_equal(key, "Status") == TRUE) - netreg_update_roaming(modem, &value); + netreg_update_status(modem, &value); else if (g_str_equal(key, "MobileCountryCode") == TRUE) netreg_update_regdom(modem, &value); @@ -1646,9 +1645,12 @@ static gboolean cdma_netreg_changed(DBusConnection *conn, else if (g_str_equal(key, "DataStrength") == TRUE) netreg_update_datastrength(modem, &value); else if (g_str_equal(key, "Status") == TRUE) - netreg_update_roaming(modem, &value); + netreg_update_status(modem, &value); - add_cdma_network(modem); + if (modem->registered == TRUE) + add_cdma_network(modem); + else + remove_network(modem); return TRUE; } @@ -1675,12 +1677,15 @@ static void cdma_netreg_properties_reply(struct modem_data *modem, else if (g_str_equal(key, "DataStrength") == TRUE) netreg_update_datastrength(modem, &value); else if (g_str_equal(key, "Status") == TRUE) - netreg_update_roaming(modem, &value); + netreg_update_status(modem, &value); dbus_message_iter_next(dict); } - add_cdma_network(modem); + if (modem->registered == TRUE) + add_cdma_network(modem); + else + remove_network(modem); } static int cdma_netreg_get_properties(struct modem_data *modem) |