From: Sergey Matyukevich <[email protected]>
Implement read_settings function to get configuration for automatic
contexts. Fix the issue uncovered by added support for automatic
context activation: do not use AT+CGACT for the contexts handled
by AT^SWWAN. As per modem specs, CGACT context can not be reused
for SWWAN. Though that worked for some reason when automatic
context was reactivated without proper deactivation.
---
drivers/gemaltomodem/gprs-context.c | 110 +++++++++++++++-------------
1 file changed, 59 insertions(+), 51 deletions(-)
diff --git a/drivers/gemaltomodem/gprs-context.c
b/drivers/gemaltomodem/gprs-context.c
index 322a5f98..680f01ab 100644
--- a/drivers/gemaltomodem/gprs-context.c
+++ b/drivers/gemaltomodem/gprs-context.c
@@ -50,70 +50,61 @@ struct gprs_context_data {
void *cb_data;
};
-static void cgact_enable_cb(gboolean ok, GAtResult *result,
- gpointer user_data)
+static void set_gprs_context_interface(struct ofono_gprs_context *gc)
{
- struct ofono_gprs_context *gc = user_data;
- struct gprs_context_data *gcd = ofono_gprs_context_get_data(gc);
struct ofono_modem *modem;
const char *interface;
- char buf[64];
+
+ modem = ofono_gprs_context_get_modem(gc);
+ interface = ofono_modem_get_string(modem, "NetworkInterface");
+ ofono_gprs_context_set_interface(gc, interface);
+}
+
+static void swwan_cb(gboolean ok, GAtResult *result, gpointer user_data)
+{
+ struct ofono_gprs_context *gc = user_data;
+ struct gprs_context_data *gcd = ofono_gprs_context_get_data(gc);
+ struct ofono_error error;
DBG("ok %d", ok);
if (!ok) {
- struct ofono_error error;
-
+ ofono_error("Unable to activate context");
+ ofono_gprs_context_deactivated(gc, gcd->active_context);
gcd->active_context = 0;
-
decode_at_error(&error, g_at_result_final_response(result));
gcd->cb(&error, gcd->cb_data);
-
return;
}
-
- snprintf(buf, sizeof(buf), "AT^SWWAN=1,%u", gcd->active_context);
- g_at_chat_send(gcd->chat, buf, none_prefix, NULL, NULL, NULL);
-
- modem = ofono_gprs_context_get_modem(gc);
- interface = ofono_modem_get_string(modem, "NetworkInterface");
- ofono_gprs_context_set_interface(gc, interface);
-
- /* Use DHCP */
- ofono_gprs_context_set_ipv4_address(gc, NULL, 0);
-
- CALLBACK_WITH_SUCCESS(gcd->cb, gcd->cb_data);
}
static void cgdcont_enable_cb(gboolean ok, GAtResult *result,
- gpointer user_data)
+ gpointer user_data)
{
struct ofono_gprs_context *gc = user_data;
struct gprs_context_data *gcd = ofono_gprs_context_get_data(gc);
+ struct ofono_error error;
char buf[64];
DBG("ok %d", ok);
if (!ok) {
- struct ofono_error error;
-
gcd->active_context = 0;
-
decode_at_error(&error, g_at_result_final_response(result));
gcd->cb(&error, gcd->cb_data);
-
return;
}
- snprintf(buf, sizeof(buf), "AT+CGACT=1,%u", gcd->active_context);
-
- if (g_at_chat_send(gcd->chat, buf, none_prefix,
- cgact_enable_cb, gc, NULL) == 0)
- goto error;
+ snprintf(buf, sizeof(buf), "AT^SWWAN=1,%u", gcd->active_context);
+ if (g_at_chat_send(gcd->chat, buf, none_prefix, swwan_cb, gc, NULL)) {
+ set_gprs_context_interface(gc);
+ /* Use DHCP */
+ ofono_gprs_context_set_ipv4_address(gc, NULL, 0);
- return;
+ CALLBACK_WITH_SUCCESS(gcd->cb, gcd->cb_data);
+ return;
+ }
-error:
CALLBACK_WITH_FAILURE(gcd->cb, gcd->cb_data);
}
@@ -141,34 +132,28 @@ static void gemalto_gprs_activate_primary(struct
ofono_gprs_context *gc,
snprintf(buf + len, sizeof(buf) - len - 3, ",\"%s\"", ctx->apn);
if (g_at_chat_send(gcd->chat, buf, none_prefix,
- cgdcont_enable_cb, gc, NULL) == 0)
- goto error;
-
- return;
+ cgdcont_enable_cb, gc, NULL))
+ return;
error:
CALLBACK_WITH_FAILURE(cb, data);
}
-static void cgact_disable_cb(gboolean ok, GAtResult *result,
- gpointer user_data)
+static void deactivate_cb(gboolean ok, GAtResult *result,
+ gpointer user_data)
{
struct ofono_gprs_context *gc = user_data;
struct gprs_context_data *gcd = ofono_gprs_context_get_data(gc);
- char buf[64];
DBG("ok %d", ok);
+ gcd->active_context = 0;
+
if (!ok) {
CALLBACK_WITH_FAILURE(gcd->cb, gcd->cb_data);
return;
}
- snprintf(buf, sizeof(buf), "AT^SWWAN=0,%u", gcd->active_context);
- g_at_chat_send(gcd->chat, buf, none_prefix, NULL, NULL, NULL);
-
- gcd->active_context = 0;
-
CALLBACK_WITH_SUCCESS(gcd->cb, gcd->cb_data);
}
@@ -185,17 +170,39 @@ static void gemalto_gprs_deactivate_primary(struct
ofono_gprs_context *gc,
gcd->cb = cb;
gcd->cb_data = data;
- snprintf(buf, sizeof(buf), "AT+CGACT=0,%u", cid);
+ snprintf(buf, sizeof(buf), "AT^SWWAN=0,%u", gcd->active_context);
if (g_at_chat_send(gcd->chat, buf, none_prefix,
- cgact_disable_cb, gc, NULL) == 0)
- goto error;
-
- return;
+ deactivate_cb, gc, NULL))
+ return;
-error:
CALLBACK_WITH_FAILURE(cb, data);
+}
+
+static void gemalto_gprs_read_settings(struct ofono_gprs_context *gc,
+ unsigned int cid,
+ ofono_gprs_context_cb_t cb, void *data)
+{
+ struct gprs_context_data *gcd = ofono_gprs_context_get_data(gc);
+ char buf[64];
+
+ DBG("cid %u", cid);
+
+ gcd->active_context = cid;
+ gcd->cb = cb;
+ gcd->cb_data = data;
+
+ snprintf(buf, sizeof(buf), "AT^SWWAN=1,%u", gcd->active_context);
+ if (g_at_chat_send(gcd->chat, buf, none_prefix, swwan_cb, gc, NULL)) {
+ set_gprs_context_interface(gc);
+ /* Use DHCP */
+ ofono_gprs_context_set_ipv4_address(gc, NULL, 0);
+ CALLBACK_WITH_SUCCESS(gcd->cb, gcd->cb_data);
+ return;
+ }
+
+ CALLBACK_WITH_FAILURE(gcd->cb, gcd->cb_data);
}
static void cgev_notify(GAtResult *result, gpointer user_data)
@@ -267,6 +274,7 @@ static const struct ofono_gprs_context_driver driver = {
.remove = gemalto_gprs_context_remove,
.activate_primary = gemalto_gprs_activate_primary,
.deactivate_primary = gemalto_gprs_deactivate_primary,
+ .read_settings = gemalto_gprs_read_settings,
};
void gemalto_gprs_context_init(void)
--
2.29.2
_______________________________________________
ofono mailing list -- [email protected]
To unsubscribe send an email to [email protected]
