Option modem use OCTI and OUWCTI commands for reporting the cell capability. --- This patch needs some more testing. v2: simplify v1 and use two callbacks instead of one v1: do not depend on unsoliced message. use only octi & ouwcti, no ossysi v0: based on "[RFC v3] Readout technology form Option HSO modems" drivers/atmodem/network-registration.c | 194 ++++++++++++++++++++------------ 1 files changed, 121 insertions(+), 73 deletions(-) diff --git a/drivers/atmodem/network-registration.c b/drivers/atmodem/network-registration.c index f5ccbb1..f61cb25 100644 --- a/drivers/atmodem/network-registration.c +++ b/drivers/atmodem/network-registration.c @@ -46,6 +46,7 @@ static const char *creg_prefix[] = { "+CREG:", NULL }; static const char *cops_prefix[] = { "+COPS:", NULL }; static const char *csq_prefix[] = { "+CSQ:", NULL }; static const char *cind_prefix[] = { "+CIND:", NULL }; +static const char *option_tech_prefix[] = { "_OCTI:", "_OUWCTI:", NULL }; struct netreg_data { GAtChat *chat; @@ -54,6 +55,9 @@ struct netreg_data { int signal_index; /* If strength is reported via CIND */ int signal_min; /* min strength reported via CIND */ int signal_max; /* max strength reported via CIND */ + int status; + int lac; + int ci; int tech; unsigned int vendor; }; @@ -69,11 +73,62 @@ static void extract_mcc_mnc(const char *str, char *mcc, char *mnc) mnc[OFONO_MAX_MNC_LENGTH] = '\0'; } +static int option_parse_tech(GAtResult *result) +{ + GAtResultIter iter; + int s, octi, ouwcti; + int tech = -1; + + g_at_result_iter_init(&iter, result); + + if (!g_at_result_iter_next(&iter, "_OCTI:")) + return -1; + + if (!g_at_result_iter_next_number(&iter, &s)) + return -1; + + if (!g_at_result_iter_next_number(&iter, &octi)) + return -1; + + if (!g_at_result_iter_next(&iter, "_OUWCTI:")) + return -1; + + if (!g_at_result_iter_next_number(&iter, &s)) + return -1; + + if (!g_at_result_iter_next_number(&iter, &ouwcti)) + return -1; + + switch (octi) { + case 1: /* GSM */ + tech = 0; + break; + case 2: /* GPRS */ + tech = 1; + break; + case 3: /* EDGE */ + tech = 3; + } + + switch (ouwcti) { + case 1: /* UMTS */ + case 2: /* HSDPA */ + case 3: /* HSUPA */ + case 4: /* HSPA */ + tech = 2; + break; + } + + DBG("octi %d ouwcti %d tech %d", octi, ouwcti, tech); + + return tech; +} + static void at_creg_cb(gboolean ok, GAtResult *result, gpointer user_data) { struct cb_data *cbd = user_data; ofono_netreg_status_cb_t cb = cbd->cb; - int status, lac, ci, tech; + int tech; struct ofono_error error; struct netreg_data *nd = cbd->user; @@ -84,16 +139,16 @@ static void at_creg_cb(gboolean ok, GAtResult *result, gpointer user_data) return; } - if (at_util_parse_reg(result, "+CREG:", NULL, &status, - &lac, &ci, &tech, nd->vendor) == FALSE) { + if (at_util_parse_reg(result, "+CREG:", NULL, &nd->status, + &nd->lac, &nd->ci, &tech, nd->vendor) == FALSE) { CALLBACK_WITH_FAILURE(cb, -1, -1, -1, -1, cbd->data); return; } - - if ((status == 1 || status == 5) && (tech == -1)) + + if ((nd->status == 1 || nd->status == 5) && (tech == -1)) tech = nd->tech; - cb(&error, status, lac, ci, tech, cbd->data); + cb(&error, nd->status, nd->lac, nd->ci, tech, cbd->data); } static void at_registration_status(struct ofono_netreg *netreg, @@ -125,6 +180,14 @@ static void at_registration_status(struct ofono_netreg *netreg, g_at_chat_send(nd->chat, "AT$CNTI=0", none_prefix, NULL, NULL, NULL); break; + case OFONO_VENDOR_OPTION_HSO: + /* + * Send AT_OCTI?;_OUWCTI? to find out the current tech, it will be + * intercepted in option_tech_notify + */ + g_at_chat_send(nd->chat, "AT_OCTI?;_OUWCTI?", + option_tech_prefix, NULL, NULL, NULL); + break; } if (g_at_chat_send(nd->chat, "AT+CREG?", creg_prefix, @@ -515,6 +578,14 @@ static void calypso_csq_notify(GAtResult *result, gpointer user_data) at_util_convert_signal_strength(strength)); } +static void option_tech_notify(GAtResult *result, gpointer user_data) +{ + struct ofono_netreg *netreg = user_data; + struct netreg_data *nd = ofono_netreg_get_data(netreg); + + nd->tech = option_parse_tech(result); +} + static void option_osigq_notify(GAtResult *result, gpointer user_data) { struct ofono_netreg *netreg = user_data; @@ -533,38 +604,6 @@ static void option_osigq_notify(GAtResult *result, gpointer user_data) at_util_convert_signal_strength(strength)); } -static void option_ouwcti_notify(GAtResult *result, gpointer user_data) -{ - int mode; - GAtResultIter iter; - - g_at_result_iter_init(&iter, result); - - if (!g_at_result_iter_next(&iter, "_OUWCTI:")) - return; - - if (!g_at_result_iter_next_number(&iter, &mode)) - return; - - ofono_info("OWCTI mode: %d", mode); -} - -static void option_octi_notify(GAtResult *result, gpointer user_data) -{ - int mode; - GAtResultIter iter; - - g_at_result_iter_init(&iter, result); - - if (!g_at_result_iter_next(&iter, "_OCTI:")) - return; - - if (!g_at_result_iter_next_number(&iter, &mode)) - return; - - ofono_info("OCTI mode: %d", mode); -} - static void ciev_notify(GAtResult *result, gpointer user_data) { struct ofono_netreg *netreg = user_data; @@ -624,22 +663,6 @@ static void cind_cb(gboolean ok, GAtResult *result, gpointer user_data) cb(&error, strength, cbd->data); } -static void option_ossysi_notify(GAtResult *result, gpointer user_data) -{ - int mode; - GAtResultIter iter; - - g_at_result_iter_init(&iter, result); - - if (!g_at_result_iter_next(&iter, "_OSSYSI:")) - return; - - if (!g_at_result_iter_next_number(&iter, &mode)) - return; - - ofono_info("OSSYSI mode: %d", mode); -} - static void huawei_rssi_notify(GAtResult *result, gpointer user_data) { struct ofono_netreg *netreg = user_data; @@ -803,20 +826,51 @@ static void nw_cnti_notify(GAtResult *result, gpointer user_data) ofono_info("CNTI: %s", tech); } +static void option_tech_cb(gboolean ok, GAtResult *result, gpointer user_data) +{ + struct cb_data *cbd = user_data; + struct ofono_netreg *netreg = cbd->data; + struct netreg_data *nd = ofono_netreg_get_data(netreg); + struct ofono_error error; + int tech; + + decode_at_error(&error, g_at_result_final_response(result)); + + if (!ok) + return; + + tech = option_parse_tech(result); + ofono_netreg_status_notify(netreg, nd->status, nd->lac, nd->ci, tech); +} + static void creg_notify(GAtResult *result, gpointer user_data) { struct ofono_netreg *netreg = user_data; - int status, lac, ci, tech; + int tech; struct netreg_data *nd = ofono_netreg_get_data(netreg); + struct cb_data *cbd; - if (at_util_parse_reg_unsolicited(result, "+CREG:", &status, - &lac, &ci, &tech, nd->vendor) == FALSE) + if (at_util_parse_reg_unsolicited(result, "+CREG:", &nd->status, + &nd->lac, &nd->ci, &tech, nd->vendor) == FALSE) return; - if ((status == 1 || status == 5) && tech == -1) - tech = nd->tech; + switch (nd->vendor) { + case OFONO_VENDOR_OPTION_HSO: + cbd = cb_data_new(NULL, netreg); + if (!cbd) + return; - ofono_netreg_status_notify(netreg, status, lac, ci, tech); + cbd->user = netreg; + + g_at_chat_send(nd->chat, "AT_OCTI?;_OUWCTI?", + option_tech_prefix, option_tech_cb, cbd, g_free); + break; + default: + if ((nd->status == 1 || nd->status == 5) && (tech == -1)) + tech = nd->tech; + + ofono_netreg_status_notify(netreg, nd->status, nd->lac, nd->ci, tech); + } } static void cind_support_cb(gboolean ok, GAtResult *result, gpointer user_data) @@ -909,29 +963,20 @@ static void at_creg_set_cb(gboolean ok, GAtResult *result, gpointer user_data) case OFONO_VENDOR_OPTION_HSO: g_at_chat_send(nd->chat, "AT_OSSYS=1", none_prefix, NULL, NULL, NULL); - g_at_chat_send(nd->chat, "AT_OUWCTI=1", none_prefix, - NULL, NULL, NULL); - g_at_chat_send(nd->chat, "AT_OCTI=1", none_prefix, - NULL, NULL, NULL); g_at_chat_send(nd->chat, "AT_OSQI=1", none_prefix, NULL, NULL, NULL); g_at_chat_register(nd->chat, "_OSIGQ:", option_osigq_notify, FALSE, netreg, NULL); - g_at_chat_register(nd->chat, "_OUWCTI:", option_ouwcti_notify, - FALSE, netreg, NULL); - g_at_chat_register(nd->chat, "_OCTI:", option_octi_notify, - FALSE, netreg, NULL); - g_at_chat_register(nd->chat, "_OSSYSI:", option_ossysi_notify, + g_at_chat_register(nd->chat, "AT_OCTI?;_OUWCTI?", option_tech_notify, FALSE, netreg, NULL); g_at_chat_send(nd->chat, "AT_OSSYS?", none_prefix, NULL, NULL, NULL); - g_at_chat_send(nd->chat, "AT_OWCTI?", none_prefix, - NULL, NULL, NULL); - g_at_chat_send(nd->chat, "AT_OCTI?", none_prefix, - NULL, NULL, NULL); g_at_chat_send(nd->chat, "AT_OSQI?", none_prefix, NULL, NULL, NULL); + g_at_chat_send(nd->chat, "AT_OCTI?;_OUWCTI?", + option_tech_prefix, NULL, NULL, NULL); + break; case OFONO_VENDOR_MBM: g_at_chat_send(nd->chat, "AT*ERINFO=1", none_prefix, @@ -1021,6 +1066,9 @@ static int at_netreg_probe(struct ofono_netreg *netreg, unsigned int vendor, nd->chat = chat; nd->vendor = vendor; + nd->status = -1; + nd->lac = -1; + nd->ci = -1; nd->tech = -1; ofono_netreg_set_data(netreg, nd); -- 1.6.6.1