Update the at phonebook driver to the new API

This commit is contained in:
Denis Kenzior 2009-08-13 16:40:08 -05:00
parent ecdfbebc51
commit 1c25111c27
3 changed files with 111 additions and 103 deletions

View File

@ -28,7 +28,7 @@ struct at_data {
struct netreg_data *netreg; struct netreg_data *netreg;
struct voicecall_data *voicecall; struct voicecall_data *voicecall;
struct sms_data *sms; struct sms_data *sms;
struct pb_data *pb; struct ofono_phonebook *pb;
}; };
void decode_at_error(struct ofono_error *error, const char *final); void decode_at_error(struct ofono_error *error, const char *final);
@ -92,5 +92,5 @@ extern void at_sim_exit(struct ofono_modem *modem);
extern void at_sms_init(struct ofono_modem *modem); extern void at_sms_init(struct ofono_modem *modem);
extern void at_sms_exit(struct ofono_modem *modem); extern void at_sms_exit(struct ofono_modem *modem);
extern void at_phonebook_init(struct ofono_modem *modem); extern void at_phonebook_init();
extern void at_phonebook_exit(struct ofono_modem *modem); extern void at_phonebook_exit();

View File

@ -369,7 +369,7 @@ static void create_cb(GIOChannel *io, gboolean success, gpointer user)
ofono_call_meter_create(at->modem, "generic_at", at->parser); ofono_call_meter_create(at->modem, "generic_at", at->parser);
ofono_call_barring_create(at->modem, "generic_at", at->parser); ofono_call_barring_create(at->modem, "generic_at", at->parser);
at_sms_init(at->modem); at_sms_init(at->modem);
at_phonebook_init(at->modem); ofono_phonebook_create(at->modem, "generic_at", at->parser);
at->io = io; at->io = io;
at->driver = g_strdup(driver); at->driver = g_strdup(driver);
@ -530,6 +530,7 @@ static int atmodem_init(void)
at_call_forwarding_init(); at_call_forwarding_init();
at_call_meter_init(); at_call_meter_init();
at_call_settings_init(); at_call_settings_init();
at_phonebook_init();
manager_init(conn); manager_init(conn);
@ -542,6 +543,7 @@ static void atmodem_exit(void)
manager_exit(conn); manager_exit(conn);
at_phonebook_exit();
at_call_settings_exit(); at_call_settings_exit();
at_call_meter_exit(); at_call_meter_exit();
at_call_forwarding_exit(); at_call_forwarding_exit();

View File

@ -32,7 +32,7 @@
#include <ofono/log.h> #include <ofono/log.h>
#include <ofono/modem.h> #include <ofono/modem.h>
#include "driver.h" #include <ofono/phonebook.h>
#include "util.h" #include "util.h"
#include "gatchat.h" #include "gatchat.h"
@ -56,21 +56,9 @@ struct pb_data {
int index_min, index_max; int index_min, index_max;
char *old_charset; char *old_charset;
int supported; int supported;
GAtChat *chat;
}; };
static struct pb_data *phonebook_create()
{
struct pb_data *pb = g_try_new0(struct pb_data, 1);
return pb;
}
static void phonebook_destroy(struct pb_data *data)
{
if (data->old_charset)
g_free(data->old_charset);
g_free(data);
}
static char *ucs2_to_utf8(const char *str) static char *ucs2_to_utf8(const char *str)
{ {
long len; long len;
@ -102,20 +90,20 @@ static const char *best_charset(int supported)
static void at_cpbr_notify(GAtResult *result, gpointer user_data) static void at_cpbr_notify(GAtResult *result, gpointer user_data)
{ {
struct cb_data *cbd = user_data; struct cb_data *cbd = user_data;
struct ofono_modem *modem = cbd->modem; struct ofono_phonebook *pb = cbd->user;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
GAtResultIter iter; GAtResultIter iter;
int current; int current;
dump_response("at_cbpr_notify", 1, result); dump_response("at_cbpr_notify", 1, result);
if (at->pb->supported & CHARSET_IRA) if (pbd->supported & CHARSET_IRA)
current = CHARSET_IRA; current = CHARSET_IRA;
if (at->pb->supported & CHARSET_UCS2) if (pbd->supported & CHARSET_UCS2)
current = CHARSET_UCS2; current = CHARSET_UCS2;
if (at->pb->supported & CHARSET_UTF8) if (pbd->supported & CHARSET_UTF8)
current = CHARSET_UTF8; current = CHARSET_UTF8;
g_at_result_iter_init(&iter, result); g_at_result_iter_init(&iter, result);
@ -186,7 +174,7 @@ static void at_cpbr_notify(GAtResult *result, gpointer user_data)
if (tel_uri) if (tel_uri)
tel_uri_utf8 = ucs2_to_utf8(tel_uri); tel_uri_utf8 = ucs2_to_utf8(tel_uri);
ofono_phonebook_entry(cbd->modem, index, number, type, ofono_phonebook_entry(pb, index, number, type,
text_utf8, hidden, group_utf8, adnumber, text_utf8, hidden, group_utf8, adnumber,
adtype, secondtext_utf8, email_utf8, adtype, secondtext_utf8, email_utf8,
sip_uri_utf8, tel_uri_utf8); sip_uri_utf8, tel_uri_utf8);
@ -201,7 +189,7 @@ static void at_cpbr_notify(GAtResult *result, gpointer user_data)
/* In the case of IRA charset, assume these are Latin1 /* In the case of IRA charset, assume these are Latin1
* characters, same as in UTF8 * characters, same as in UTF8
*/ */
ofono_phonebook_entry(cbd->modem, index, number, type, ofono_phonebook_entry(pb, index, number, type,
text, hidden, group, adnumber, text, hidden, group, adnumber,
adtype, secondtext, email, adtype, secondtext, email,
sip_uri, tel_uri); sip_uri, tel_uri);
@ -212,9 +200,9 @@ static void at_cpbr_notify(GAtResult *result, gpointer user_data)
static void export_failed(struct cb_data *cbd) static void export_failed(struct cb_data *cbd)
{ {
struct ofono_modem *modem = cbd->modem; struct ofono_phonebook *pb = cbd->user;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
ofono_generic_cb_t cb = cbd->cb; ofono_phonebook_cb_t cb = cbd->cb;
{ {
DECLARE_FAILURE(error); DECLARE_FAILURE(error);
@ -223,9 +211,9 @@ static void export_failed(struct cb_data *cbd)
g_free(cbd); g_free(cbd);
if (at->pb->old_charset) { if (pbd->old_charset) {
g_free(at->pb->old_charset); g_free(pbd->old_charset);
at->pb->old_charset = NULL; pbd->old_charset = NULL;
} }
} }
@ -233,9 +221,9 @@ static void at_read_entries_cb(gboolean ok, GAtResult *result,
gpointer user_data) gpointer user_data)
{ {
struct cb_data *cbd = user_data; struct cb_data *cbd = user_data;
struct ofono_modem *modem = cbd->modem; struct ofono_phonebook *pb = cbd->user;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
ofono_generic_cb_t cb = cbd->cb; ofono_phonebook_cb_t cb = cbd->cb;
const char *charset; const char *charset;
struct ofono_error error; struct ofono_error error;
char buf[32]; char buf[32];
@ -244,25 +232,25 @@ static void at_read_entries_cb(gboolean ok, GAtResult *result,
cb(&error, cbd->data); cb(&error, cbd->data);
g_free(cbd); g_free(cbd);
charset = best_charset(at->pb->supported); charset = best_charset(pbd->supported);
if (strcmp(at->pb->old_charset, charset)) { if (strcmp(pbd->old_charset, charset)) {
sprintf(buf, "AT+CSCS=\"%s\"", at->pb->old_charset); sprintf(buf, "AT+CSCS=\"%s\"", pbd->old_charset);
g_at_chat_send(at->parser, buf, none_prefix, NULL, NULL, NULL); g_at_chat_send(pbd->chat, buf, none_prefix, NULL, NULL, NULL);
} }
g_free(at->pb->old_charset); g_free(pbd->old_charset);
at->pb->old_charset = NULL; pbd->old_charset = NULL;
} }
static void at_read_entries(struct cb_data *cbd) static void at_read_entries(struct cb_data *cbd)
{ {
struct ofono_modem *modem = cbd->modem; struct ofono_phonebook *pb = cbd->user;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
char buf[32]; char buf[32];
sprintf(buf, "AT+CPBR=%d,%d", at->pb->index_min, at->pb->index_max); sprintf(buf, "AT+CPBR=%d,%d", pbd->index_min, pbd->index_max);
if (g_at_chat_send_listing(at->parser, buf, cpbr_prefix, if (g_at_chat_send_listing(pbd->chat, buf, cpbr_prefix,
at_cpbr_notify, at_read_entries_cb, at_cpbr_notify, at_read_entries_cb,
cbd, NULL) > 0) cbd, NULL) > 0)
return; return;
@ -290,8 +278,8 @@ static void at_read_charset_cb(gboolean ok, GAtResult *result,
gpointer user_data) gpointer user_data)
{ {
struct cb_data *cbd = user_data; struct cb_data *cbd = user_data;
struct ofono_modem *modem = cbd->modem; struct ofono_phonebook *pb = cbd->user;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
GAtResultIter iter; GAtResultIter iter;
const char *charset; const char *charset;
char buf[32]; char buf[32];
@ -308,17 +296,17 @@ static void at_read_charset_cb(gboolean ok, GAtResult *result,
g_at_result_iter_next_string(&iter, &charset); g_at_result_iter_next_string(&iter, &charset);
at->pb->old_charset = g_strdup(charset); pbd->old_charset = g_strdup(charset);
charset = best_charset(at->pb->supported); charset = best_charset(pbd->supported);
if (!strcmp(at->pb->old_charset, charset)) { if (!strcmp(pbd->old_charset, charset)) {
at_read_entries(cbd); at_read_entries(cbd);
return; return;
} }
sprintf(buf, "AT+CSCS=\"%s\"", charset); sprintf(buf, "AT+CSCS=\"%s\"", charset);
if (g_at_chat_send(at->parser, buf, none_prefix, if (g_at_chat_send(pbd->chat, buf, none_prefix,
at_set_charset_cb, cbd, NULL) > 0) at_set_charset_cb, cbd, NULL) > 0)
return; return;
@ -330,8 +318,8 @@ static void at_list_indices_cb(gboolean ok, GAtResult *result,
gpointer user_data) gpointer user_data)
{ {
struct cb_data *cbd = user_data; struct cb_data *cbd = user_data;
struct ofono_modem *modem = cbd->modem; struct ofono_phonebook *pb = cbd->user;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
GAtResultIter iter; GAtResultIter iter;
if (!ok) if (!ok)
@ -347,14 +335,14 @@ static void at_list_indices_cb(gboolean ok, GAtResult *result,
/* retrieve index_min and index_max from indices /* retrieve index_min and index_max from indices
* which seems like "(1-150),32,16" * which seems like "(1-150),32,16"
*/ */
if (!g_at_result_iter_next_range(&iter, &at->pb->index_min, if (!g_at_result_iter_next_range(&iter, &pbd->index_min,
&at->pb->index_max)) &pbd->index_max))
goto error; goto error;
if (!g_at_result_iter_close_list(&iter)) if (!g_at_result_iter_close_list(&iter))
goto error; goto error;
if (g_at_chat_send(at->parser, "AT+CSCS?", cscs_prefix, if (g_at_chat_send(pbd->chat, "AT+CSCS?", cscs_prefix,
at_read_charset_cb, cbd, NULL) > 0) at_read_charset_cb, cbd, NULL) > 0)
return; return;
@ -366,15 +354,15 @@ static void at_select_storage_cb(gboolean ok, GAtResult *result,
gpointer user_data) gpointer user_data)
{ {
struct cb_data *cbd = user_data; struct cb_data *cbd = user_data;
struct ofono_modem *modem = cbd->modem; struct ofono_phonebook *pb = cbd->user;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
dump_response("at_select_storage_cb", ok, result); dump_response("at_select_storage_cb", ok, result);
if (!ok) if (!ok)
goto error; goto error;
if (g_at_chat_send(at->parser, "AT+CPBR=?", cpbr_prefix, if (g_at_chat_send(pbd->chat, "AT+CPBR=?", cpbr_prefix,
at_list_indices_cb, cbd, NULL) > 0) at_list_indices_cb, cbd, NULL) > 0)
return; return;
@ -382,18 +370,20 @@ error:
export_failed(cbd); export_failed(cbd);
} }
static void at_export_entries(struct ofono_modem *modem, const char *storage, static void at_export_entries(struct ofono_phonebook *pb, const char *storage,
ofono_generic_cb_t cb, void *data) ofono_phonebook_cb_t cb, void *data)
{ {
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
struct cb_data *cbd = cb_data_new(modem, cb, data); struct cb_data *cbd = cb_data_new(NULL, cb, data);
char buf[32]; char buf[32];
if (!cbd) if (!cbd)
goto error; goto error;
cbd->user = pb;
sprintf(buf, "AT+CPBS=\"%s\"", storage); sprintf(buf, "AT+CPBS=\"%s\"", storage);
if (g_at_chat_send(at->parser, buf, none_prefix, if (g_at_chat_send(pbd->chat, buf, none_prefix,
at_select_storage_cb, cbd, NULL) > 0) at_select_storage_cb, cbd, NULL) > 0)
return; return;
@ -407,26 +397,19 @@ error:
} }
} }
static struct ofono_phonebook_ops ops = { static void phonebook_not_supported(struct ofono_phonebook *pb)
.export_entries = at_export_entries
};
static void phonebook_not_supported(struct ofono_modem *modem)
{ {
struct at_data *at = ofono_modem_get_userdata(modem);
ofono_error("Phonebook not supported by this modem. If this is in " ofono_error("Phonebook not supported by this modem. If this is in "
"error please submit patches to support this hardware"); "error please submit patches to support this hardware");
if (at->pb) {
phonebook_destroy(at->pb); ofono_phonebook_remove(pb);
at->pb = NULL;
}
} }
static void at_list_storages_cb(gboolean ok, GAtResult *result, static void at_list_storages_cb(gboolean ok, GAtResult *result,
gpointer user_data) gpointer user_data)
{ {
struct ofono_modem *modem = user_data; struct ofono_phonebook *pb = user_data;
struct pb_data *pbd = ofono_phonebook_get_data(pb);
gboolean sm_supported = FALSE; gboolean sm_supported = FALSE;
gboolean me_supported = FALSE; gboolean me_supported = FALSE;
gboolean in_list = FALSE; gboolean in_list = FALSE;
@ -459,18 +442,18 @@ static void at_list_storages_cb(gboolean ok, GAtResult *result,
if (!me_supported && !sm_supported) if (!me_supported && !sm_supported)
goto error; goto error;
ofono_phonebook_register(modem, &ops); ofono_phonebook_register(pb);
return; return;
error: error:
phonebook_not_supported(modem); phonebook_not_supported(pb);
} }
static void at_list_charsets_cb(gboolean ok, GAtResult *result, static void at_list_charsets_cb(gboolean ok, GAtResult *result,
gpointer user_data) gpointer user_data)
{ {
struct ofono_modem *modem = user_data; struct ofono_phonebook *pb = user_data;
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
gboolean in_list = FALSE; gboolean in_list = FALSE;
GAtResultIter iter; GAtResultIter iter;
const char *charset; const char *charset;
@ -490,24 +473,24 @@ static void at_list_charsets_cb(gboolean ok, GAtResult *result,
while (g_at_result_iter_next_string(&iter, &charset)) { while (g_at_result_iter_next_string(&iter, &charset)) {
if (!strcmp(charset, "UTF-8")) if (!strcmp(charset, "UTF-8"))
at->pb->supported |= CHARSET_UTF8; pbd->supported |= CHARSET_UTF8;
else if (!strcmp(charset, "UCS2")) else if (!strcmp(charset, "UCS2"))
at->pb->supported |= CHARSET_UCS2; pbd->supported |= CHARSET_UCS2;
else if (!strcmp(charset, "IRA")) else if (!strcmp(charset, "IRA"))
at->pb->supported |= CHARSET_IRA; pbd->supported |= CHARSET_IRA;
} }
if (in_list && !g_at_result_iter_close_list(&iter)) if (in_list && !g_at_result_iter_close_list(&iter))
goto error; goto error;
if (!(at->pb->supported & CHARSET_SUPPORT)) { if (!(pbd->supported & CHARSET_SUPPORT)) {
/* Some modems, like the Google G1, do not support UCS2 or UTF8 /* Some modems, like the Google G1, do not support UCS2 or UTF8
* Such modems are effectively junk, but we can still get some * Such modems are effectively junk, but we can still get some
* useful information out of them by using IRA charset, which * useful information out of them by using IRA charset, which
* is essentially Latin1. Still, all bets are off if a SIM * is essentially Latin1. Still, all bets are off if a SIM
* with UCS2 encoded entries is present. * with UCS2 encoded entries is present.
*/ */
if (at->pb->supported & CHARSET_IRA) { if (pbd->supported & CHARSET_IRA) {
ofono_error("This modem does not support UCS2 or UTF8 " ofono_error("This modem does not support UCS2 or UTF8 "
"character sets. This means no i18n " "character sets. This means no i18n "
"phonebook is possible on this modem," "phonebook is possible on this modem,"
@ -517,42 +500,65 @@ static void at_list_charsets_cb(gboolean ok, GAtResult *result,
goto error; goto error;
} }
if (g_at_chat_send(at->parser, "AT+CPBS=?", cpbs_prefix, if (g_at_chat_send(pbd->chat, "AT+CPBS=?", cpbs_prefix,
at_list_storages_cb, modem, NULL) > 0) at_list_storages_cb, pb, NULL) > 0)
return; return;
error: error:
phonebook_not_supported(modem); phonebook_not_supported(pb);
} }
static void at_list_charsets(struct ofono_modem *modem) static void at_list_charsets(struct ofono_phonebook *pb)
{ {
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
if (g_at_chat_send(at->parser, "AT+CSCS=?", cscs_prefix, if (g_at_chat_send(pbd->chat, "AT+CSCS=?", cscs_prefix,
at_list_charsets_cb, modem, NULL) > 0) at_list_charsets_cb, pb, NULL) > 0)
return; return;
phonebook_not_supported(modem); phonebook_not_supported(pb);
} }
void at_phonebook_init(struct ofono_modem *modem) static int at_phonebook_probe(struct ofono_phonebook *pb)
{ {
struct at_data *at = ofono_modem_get_userdata(modem); GAtChat *chat = ofono_phonebook_get_data(pb);
struct pb_data *pbd;
at->pb = phonebook_create(); pbd = g_new0(struct pb_data, 1);
at_list_charsets(modem); pbd->chat = chat;
ofono_phonebook_set_data(pb, pbd);
at_list_charsets(pb);
return 0;
} }
void at_phonebook_exit(struct ofono_modem *modem) static int at_phonebook_remove(struct ofono_phonebook *pb)
{ {
struct at_data *at = ofono_modem_get_userdata(modem); struct pb_data *pbd = ofono_phonebook_get_data(pb);
if (!at->pb) if (pbd->old_charset)
return; g_free(pbd->old_charset);
phonebook_destroy(at->pb); g_free(pbd);
at->pb = NULL;
ofono_phonebook_unregister(modem); return 0;
}
static struct ofono_phonebook_driver driver = {
.name = "generic_at",
.probe = at_phonebook_probe,
.remove = at_phonebook_remove,
.export_entries = at_export_entries
};
void at_phonebook_init()
{
ofono_phonebook_driver_register(&driver);
}
void at_phonebook_exit()
{
ofono_phonebook_driver_unregister(&driver);
} }