Hi Antti,

>+static GSList *g_drivers = NULL;
>+
>+static GDBusMethodTable ci_methods[] = {
>+      { "GetProperties",      "",     "a{sv}aa{sv}",  ci_get_cells },
>+      { }
+};
+

Do we need to add 'G_DBUS_METHOD_FLAG_ASYNC' after ci_get_cells? 


>+static DBusMessage *ci_get_cells(DBusConnection *conn, DBusMessage *msg,
>+                                      void *data)
>+{
>+      struct ofono_cell_info *ci = data;
>+      DBG("");
>+
>+      ci->pending = dbus_message_ref(msg);
>+      ci->driver->query(ci, ofono_cell_info_query_cb, ci->driver_data);

The 3rd parameter should be 'ci' instead of 'ci->driver_data', right?



Regards,
Wei, Jun 韦峻
Email: jun....@intel.com
_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono

Reply via email to