[PATCH_v4 0/5] Private network request to ConnMan
by Guillaume Zajac
Hi,
Changelog from v3 is:
- Add private-network source/include
- ConnMan plugin is independant from emulator
- Each application that need VPN will pass a callback as argument
when private network is requested. This callback will contain the
private network settings.
Guillaume Zajac (5):
gatppp: Add new contructor to use external fd
private-network: add callback typedef drivers and settings
private-network: add request/release functions and new feature to
Makefile.am
emulator: add request/release private network calls
connman: add plugin in oFono to request request/release private
network
Makefile.am | 10 +-
gatchat/gatppp.c | 33 +++++-
gatchat/gatppp.h | 1 +
gatchat/ppp.h | 2 +-
gatchat/ppp_net.c | 40 ++++---
include/private-network.h | 59 +++++++++
plugins/connman.c | 297 +++++++++++++++++++++++++++++++++++++++++++++
src/emulator.c | 49 ++++++--
src/ofono.h | 6 +
src/private-network.c | 89 ++++++++++++++
10 files changed, 556 insertions(+), 30 deletions(-)
create mode 100644 include/private-network.h
create mode 100644 plugins/connman.c
create mode 100644 src/private-network.c
1 month, 1 week
ofono with sim5320 module
by David Ashley
Hello, I'm at my wits' end trying to get ofono working with the
sim5320 module. I'm using the plugins/sim900.c module as a starting
point. I think the issue has something to do with the difference
between the MUX functionality between the 900 and the 5320. The sim900
supports the elaborate parameters sent on the
AT+CMUX=0,x,x,x,x, etc.
but the SIM5320 only supports
AT+CMUX=0
There's that... but also the way the sim900 plugin creates a
SETUP_DLC, initiates muxing, then deletes the setup DLC and creates 4
new DLC's... it didn't work for the sim5320 until I remapped the DLC's
somewhat like this:
#define NUM_DLC 4
#define VOICE_DLC 2
#define NETREG_DLC 1
//#define SMS_DLC 2
#define GPRS_DLC 3
#define SETUP_DLC 0
static char *dlc_prefixes[NUM_DLC] = {
[VOICE_DLC]="Voice: ",
[NETREG_DLC]="Net: ",
// [SMS_DLC]= "SMS: ",
[GPRS_DLC]= "GPRS: " ,
[SETUP_DLC]= "Setup: ",
};
Note I have to eliminate the SMS_DLC usage later in sim5320_post_sim:
// ofono_sms_create(modem, OFONO_VENDOR_SIMCOM, "atmodem",
// data->dlcs[SMS_DLC]);
OK everything is *ALMOST* working. ofonod interacts fine with
connmand, connmand tells ofonod to activate the sim5320, which
actually establishes a ppp connection and sets up a ppp device:
ppp0 Link encap:UNSPEC HWaddr 00-00-00-00-00-00-00-00-00-00-00-00-00-00-0
inet addr:30.97.132.47 P-t-P:30.97.132.47 Mask:255.255.255.255
UP POINTOPOINT RUNNING NOARP MULTICAST MTU:1500 Metric:1
RX packets:0 errors:0 dropped:0 overruns:0 frame:0
TX packets:2 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:500
RX bytes:0 (0.0 B) TX bytes:124 (124.0 B)
Here's the rub: No matter what I do, I never get any RX packets from
that ppp device, and even when it appears to TX packets (I'm trying to
ping out) the machine on the internet isn't actually receiving them.
I'm running on a beaglebone with a custom board with a sim5320 module on it.
I have no idea what to try... Any advice would be appreciated...
Thanks very much!!!!
-Dave
4 years, 1 month
qmimodem: GPRS context stay Active when GPRS attach is lost
by Christophe Ronco
Hello,
Even with detach_shutdown method implemented in qmimodem (patch done by
Jonas Bonn two or three weeks ago), I still sometimes have problems when
I have a lot of GPRS deconnections.
I use a MC7304 modem and to simulate short GPRS network losses, I simply
unplug and replug modem antenna.
I figured out that my problems come from QMI_WDS_PKT_STATUS_IND that is
received after GPRS network is back and context activation started once
again (by connman).
I have posted a trace with connman and ofono traces here (available for
the next 7 days): https://we.tl/FliZAdaVN5
I have added some additional traces in ofono to narrow down the problem.
Here is the trace explanation:
What happens before Apr 27 16:05:59:
1) Apr 27 16:05:44: GPRS attach lost, we asked to deactivate GPRS context.
2) Apr 27 16:05:53: GPRS attach is back
What happens at Apr 27 16:05:59:
1) Answer OK to qmi_deactivate_primary (stop_net_cb without error)
2) ofono_gprs_context_deactivated called with correct cid, property
Active set to False
3) Connman immediately reacts and ask for context reactivation.
qmi_activate_primary is called
4) pkt_status_notify received to say that packet status is disconnected
=> ofono_gprs_context_deactivated called again but context not activated
yet. Nothing happens outside of gprs-context driver but active_context
is set to 0.
What happens at Apr 27 16:10:28:
1) GPRS attach lost, we asked to deactivate GPRS context.
2) stop_net_cb called without error BUT active_context is 0.
3) ofono_gprs_context_deactivated called with wrong context id, nothing
happens
And property Active of gprs context is not modified. Connman will never
try to reconnect the service when GPRS attach is back (Apr 27 16:10:39).
I've done a patch that seems to correct the problem for me. I simply do
not handle anymore QMI_WDS_PKT_STATUS_IND notifications. I assume it is
not needed anymore (after Jonas Bonn patch implementing detach_shutdown
method in qmimodem driver). But I might be wrong...
I'll send the patch after this message.
Best Regards,
Christophe
4 years, 8 months
[PATCH v3 1/3] hfp_hf: Fix not creating modem during NewConnection
by Luiz Augusto von Dentz
From: Luiz Augusto von Dentz <luiz.von.dentz(a)intel.com>
In case the UUIDs are not updated, as they are still being resolved, when
Paired property changes a modem will never be registered.
In order to fix this problem allow modems to be registered directly
during NewConnection.
---
plugins/hfp_hf_bluez5.c | 193 +++++++++++++++++++++++++-----------------------
1 file changed, 102 insertions(+), 91 deletions(-)
diff --git a/plugins/hfp_hf_bluez5.c b/plugins/hfp_hf_bluez5.c
index 6ae98b3..fb7cc3f 100644
--- a/plugins/hfp_hf_bluez5.c
+++ b/plugins/hfp_hf_bluez5.c
@@ -499,6 +499,96 @@ static int get_version(DBusMessageIter *iter, uint16_t *version)
return -ENOENT;
}
+static gboolean has_hfp_ag_uuid(DBusMessageIter *array)
+{
+ DBusMessageIter value;
+
+ if (dbus_message_iter_get_arg_type(array) != DBUS_TYPE_ARRAY)
+ return FALSE;
+
+ dbus_message_iter_recurse(array, &value);
+
+ while (dbus_message_iter_get_arg_type(&value) == DBUS_TYPE_STRING) {
+ const char *uuid;
+
+ dbus_message_iter_get_basic(&value, &uuid);
+
+ if (g_str_equal(uuid, HFP_AG_UUID) == TRUE)
+ return TRUE;
+
+ dbus_message_iter_next(&value);
+ }
+
+ return FALSE;
+}
+
+static void modem_removed(GDBusProxy *proxy, void *user_data)
+{
+ struct ofono_modem *modem = user_data;
+
+ ofono_modem_remove(modem);
+}
+
+static void alias_changed(GDBusProxy *proxy, const char *name,
+ DBusMessageIter *iter, void *user_data)
+{
+ const char *alias;
+ struct ofono_modem *modem = user_data;
+
+ if (g_str_equal("Alias", name) == FALSE)
+ return;
+
+ dbus_message_iter_get_basic(iter, &alias);
+ ofono_modem_set_name(modem, alias);
+}
+
+static struct ofono_modem *modem_register_from_proxy(GDBusProxy *proxy,
+ const char *path)
+{
+ const char *alias, *remote;
+ DBusMessageIter iter;
+ dbus_bool_t paired;
+ struct ofono_modem *modem;
+
+ if (g_dbus_proxy_get_property(proxy, "Paired", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &paired);
+
+ if (paired == FALSE) {
+ modem = ofono_modem_find(device_path_compare, (void *) path);
+
+ if (modem != NULL) {
+ ofono_modem_remove(modem);
+ g_dbus_proxy_set_removed_watch(proxy, NULL, NULL);
+ g_dbus_proxy_set_property_watch(proxy, NULL, NULL);
+ }
+ return NULL;
+ }
+
+ if (g_dbus_proxy_get_property(proxy, "UUIDs", &iter) == FALSE)
+ return NULL;
+
+ if (has_hfp_ag_uuid(&iter) == FALSE)
+ return NULL;
+
+ if (g_dbus_proxy_get_property(proxy, "Alias", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &alias);
+
+ if (g_dbus_proxy_get_property(proxy, "Address", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &remote);
+
+ modem = modem_register(path, remote, alias);
+ g_dbus_proxy_set_property_watch(proxy, alias_changed, modem);
+ g_dbus_proxy_set_removed_watch(proxy, modem_removed, modem);
+
+ return modem;
+}
+
static DBusMessage *profile_new_connection(DBusConnection *conn,
DBusMessage *msg, void *user_data)
{
@@ -541,10 +631,18 @@ static DBusMessage *profile_new_connection(DBusConnection *conn,
modem = ofono_modem_find(device_path_compare, (void *) device);
if (modem == NULL) {
- close(fd);
- return g_dbus_create_error(msg, BLUEZ_ERROR_INTERFACE
- ".Rejected",
- "Unknown Bluetooth device");
+ GDBusProxy *proxy;
+
+ proxy = g_dbus_proxy_new(bluez, device, BLUEZ_DEVICE_INTERFACE);
+ modem = modem_register_from_proxy(proxy, device);
+ g_dbus_proxy_unref(proxy);
+
+ if (!modem) {
+ close(fd);
+ return g_dbus_create_error(msg, BLUEZ_ERROR_INTERFACE
+ ".Rejected",
+ "Unknown Bluetooth device");
+ }
}
err = service_level_connection(modem, fd, version);
@@ -686,93 +784,6 @@ static void connect_handler(DBusConnection *conn, void *user_data)
HFP_EXT_PROFILE_PATH, NULL, features);
}
-static gboolean has_hfp_ag_uuid(DBusMessageIter *array)
-{
- DBusMessageIter value;
-
- if (dbus_message_iter_get_arg_type(array) != DBUS_TYPE_ARRAY)
- return FALSE;
-
- dbus_message_iter_recurse(array, &value);
-
- while (dbus_message_iter_get_arg_type(&value) == DBUS_TYPE_STRING) {
- const char *uuid;
-
- dbus_message_iter_get_basic(&value, &uuid);
-
- if (g_str_equal(uuid, HFP_AG_UUID) == TRUE)
- return TRUE;
-
- dbus_message_iter_next(&value);
- }
-
- return FALSE;
-}
-
-static void modem_removed(GDBusProxy *proxy, void *user_data)
-{
- struct ofono_modem *modem = user_data;
-
- ofono_modem_remove(modem);
-}
-
-static void alias_changed(GDBusProxy *proxy, const char *name,
- DBusMessageIter *iter, void *user_data)
-{
- const char *alias;
- struct ofono_modem *modem = user_data;
-
- if (g_str_equal("Alias", name) == FALSE)
- return;
-
- dbus_message_iter_get_basic(iter, &alias);
- ofono_modem_set_name(modem, alias);
-}
-
-static void modem_register_from_proxy(GDBusProxy *proxy, const char *path)
-{
- const char *alias, *remote;
- DBusMessageIter iter;
- dbus_bool_t paired;
- struct ofono_modem *modem;
-
- if (g_dbus_proxy_get_property(proxy, "Paired", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &paired);
-
- if (paired == FALSE) {
- modem = ofono_modem_find(device_path_compare, (void *) path);
-
- if (modem != NULL) {
- ofono_modem_remove(modem);
- g_dbus_proxy_set_removed_watch(proxy, NULL, NULL);
- g_dbus_proxy_set_property_watch(proxy, NULL, NULL);
- }
- return;
- }
-
- if (g_dbus_proxy_get_property(proxy, "UUIDs", &iter) == FALSE)
- return;
-
- if (has_hfp_ag_uuid(&iter) == FALSE)
- return;
-
- if (g_dbus_proxy_get_property(proxy, "Alias", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &alias);
-
- if (g_dbus_proxy_get_property(proxy, "Address", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &remote);
-
- modem = modem_register(path, remote, alias);
- g_dbus_proxy_set_property_watch(proxy, alias_changed, modem);
- g_dbus_proxy_set_removed_watch(proxy, modem_removed, modem);
-}
-
static void proxy_added(GDBusProxy *proxy, void *user_data)
{
const char *interface, *path;
--
2.9.3
4 years, 8 months
[PATCH qmi lte v5 0/4] QMI LTE series
by Jonas Bonn
Changes since v4:
* Address feedback from Denis
* Lots of testing; this is robust against loss of network connectivity
and transitions across technologies.
* Dropped LTE atom for now since it's not doing anything interesting
currently... I'll resend it separately.
Original cover letter follows:
LTE is different from other 'GPRS' technologies in that a default bearer
always gets set up when the modem registers to the network. Ofono has
some support for this and this series tries to set this up for QMI
modems.
The key to understanding this series is to understand that QMI modems
do not automatically enable the network interface when connecting to
an LTE network, even though the default bearer is already set up. A
call to "start network" needs to be made in order to allow packets to
flow. So the jist of this series is:
i) detect network registration to LTE network
ii) call ofono_gprs_cid_activated
iii) in read_settings, start by calling "start network"
...and then there are some other adjustments in the series to make
this all work.
I've tested this with my EC21, both on LTE and UMTS networks. It
all seems to work, including moving out of network connectivity and
bringing the modem repeatedly online/offline.
One open question at this point:
* What happens if the connection manager calls deactivate_context
on the default bearer context? I don't see that any other drivers
guard against this, and the QMI driver will happily stop the network
interface even though the default bearer is still up in the background.
Jonas Bonn (4):
qmi: watch packet status for spontaneous disconnection only
qmi: use shared services
qmi: use named status values
qmi: activate default bearer context for LTE networks
drivers/qmimodem/gprs-context.c | 12 ++++--
drivers/qmimodem/gprs.c | 72 +++++++++++++++++++++++++--------
drivers/qmimodem/network-registration.c | 2 +-
drivers/qmimodem/radio-settings.c | 2 +-
4 files changed, 66 insertions(+), 22 deletions(-)
--
2.9.3
4 years, 8 months
[PATCH v2 1/3] hfp_hf: Fix not creating modem during NewConnection
by Luiz Augusto von Dentz
From: Luiz Augusto von Dentz <luiz.von.dentz(a)intel.com>
In case the UUIDs are not updated, as they are still being resolved, when
Paired property changes a modem will never be registered.
In order to fix this problem allow modems to be registered directly
during NewConnection.
---
plugins/hfp_hf_bluez5.c | 193 +++++++++++++++++++++++++-----------------------
1 file changed, 102 insertions(+), 91 deletions(-)
diff --git a/plugins/hfp_hf_bluez5.c b/plugins/hfp_hf_bluez5.c
index 6ae98b3..fb7cc3f 100644
--- a/plugins/hfp_hf_bluez5.c
+++ b/plugins/hfp_hf_bluez5.c
@@ -499,6 +499,96 @@ static int get_version(DBusMessageIter *iter, uint16_t *version)
return -ENOENT;
}
+static gboolean has_hfp_ag_uuid(DBusMessageIter *array)
+{
+ DBusMessageIter value;
+
+ if (dbus_message_iter_get_arg_type(array) != DBUS_TYPE_ARRAY)
+ return FALSE;
+
+ dbus_message_iter_recurse(array, &value);
+
+ while (dbus_message_iter_get_arg_type(&value) == DBUS_TYPE_STRING) {
+ const char *uuid;
+
+ dbus_message_iter_get_basic(&value, &uuid);
+
+ if (g_str_equal(uuid, HFP_AG_UUID) == TRUE)
+ return TRUE;
+
+ dbus_message_iter_next(&value);
+ }
+
+ return FALSE;
+}
+
+static void modem_removed(GDBusProxy *proxy, void *user_data)
+{
+ struct ofono_modem *modem = user_data;
+
+ ofono_modem_remove(modem);
+}
+
+static void alias_changed(GDBusProxy *proxy, const char *name,
+ DBusMessageIter *iter, void *user_data)
+{
+ const char *alias;
+ struct ofono_modem *modem = user_data;
+
+ if (g_str_equal("Alias", name) == FALSE)
+ return;
+
+ dbus_message_iter_get_basic(iter, &alias);
+ ofono_modem_set_name(modem, alias);
+}
+
+static struct ofono_modem *modem_register_from_proxy(GDBusProxy *proxy,
+ const char *path)
+{
+ const char *alias, *remote;
+ DBusMessageIter iter;
+ dbus_bool_t paired;
+ struct ofono_modem *modem;
+
+ if (g_dbus_proxy_get_property(proxy, "Paired", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &paired);
+
+ if (paired == FALSE) {
+ modem = ofono_modem_find(device_path_compare, (void *) path);
+
+ if (modem != NULL) {
+ ofono_modem_remove(modem);
+ g_dbus_proxy_set_removed_watch(proxy, NULL, NULL);
+ g_dbus_proxy_set_property_watch(proxy, NULL, NULL);
+ }
+ return NULL;
+ }
+
+ if (g_dbus_proxy_get_property(proxy, "UUIDs", &iter) == FALSE)
+ return NULL;
+
+ if (has_hfp_ag_uuid(&iter) == FALSE)
+ return NULL;
+
+ if (g_dbus_proxy_get_property(proxy, "Alias", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &alias);
+
+ if (g_dbus_proxy_get_property(proxy, "Address", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &remote);
+
+ modem = modem_register(path, remote, alias);
+ g_dbus_proxy_set_property_watch(proxy, alias_changed, modem);
+ g_dbus_proxy_set_removed_watch(proxy, modem_removed, modem);
+
+ return modem;
+}
+
static DBusMessage *profile_new_connection(DBusConnection *conn,
DBusMessage *msg, void *user_data)
{
@@ -541,10 +631,18 @@ static DBusMessage *profile_new_connection(DBusConnection *conn,
modem = ofono_modem_find(device_path_compare, (void *) device);
if (modem == NULL) {
- close(fd);
- return g_dbus_create_error(msg, BLUEZ_ERROR_INTERFACE
- ".Rejected",
- "Unknown Bluetooth device");
+ GDBusProxy *proxy;
+
+ proxy = g_dbus_proxy_new(bluez, device, BLUEZ_DEVICE_INTERFACE);
+ modem = modem_register_from_proxy(proxy, device);
+ g_dbus_proxy_unref(proxy);
+
+ if (!modem) {
+ close(fd);
+ return g_dbus_create_error(msg, BLUEZ_ERROR_INTERFACE
+ ".Rejected",
+ "Unknown Bluetooth device");
+ }
}
err = service_level_connection(modem, fd, version);
@@ -686,93 +784,6 @@ static void connect_handler(DBusConnection *conn, void *user_data)
HFP_EXT_PROFILE_PATH, NULL, features);
}
-static gboolean has_hfp_ag_uuid(DBusMessageIter *array)
-{
- DBusMessageIter value;
-
- if (dbus_message_iter_get_arg_type(array) != DBUS_TYPE_ARRAY)
- return FALSE;
-
- dbus_message_iter_recurse(array, &value);
-
- while (dbus_message_iter_get_arg_type(&value) == DBUS_TYPE_STRING) {
- const char *uuid;
-
- dbus_message_iter_get_basic(&value, &uuid);
-
- if (g_str_equal(uuid, HFP_AG_UUID) == TRUE)
- return TRUE;
-
- dbus_message_iter_next(&value);
- }
-
- return FALSE;
-}
-
-static void modem_removed(GDBusProxy *proxy, void *user_data)
-{
- struct ofono_modem *modem = user_data;
-
- ofono_modem_remove(modem);
-}
-
-static void alias_changed(GDBusProxy *proxy, const char *name,
- DBusMessageIter *iter, void *user_data)
-{
- const char *alias;
- struct ofono_modem *modem = user_data;
-
- if (g_str_equal("Alias", name) == FALSE)
- return;
-
- dbus_message_iter_get_basic(iter, &alias);
- ofono_modem_set_name(modem, alias);
-}
-
-static void modem_register_from_proxy(GDBusProxy *proxy, const char *path)
-{
- const char *alias, *remote;
- DBusMessageIter iter;
- dbus_bool_t paired;
- struct ofono_modem *modem;
-
- if (g_dbus_proxy_get_property(proxy, "Paired", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &paired);
-
- if (paired == FALSE) {
- modem = ofono_modem_find(device_path_compare, (void *) path);
-
- if (modem != NULL) {
- ofono_modem_remove(modem);
- g_dbus_proxy_set_removed_watch(proxy, NULL, NULL);
- g_dbus_proxy_set_property_watch(proxy, NULL, NULL);
- }
- return;
- }
-
- if (g_dbus_proxy_get_property(proxy, "UUIDs", &iter) == FALSE)
- return;
-
- if (has_hfp_ag_uuid(&iter) == FALSE)
- return;
-
- if (g_dbus_proxy_get_property(proxy, "Alias", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &alias);
-
- if (g_dbus_proxy_get_property(proxy, "Address", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &remote);
-
- modem = modem_register(path, remote, alias);
- g_dbus_proxy_set_property_watch(proxy, alias_changed, modem);
- g_dbus_proxy_set_removed_watch(proxy, modem_removed, modem);
-}
-
static void proxy_added(GDBusProxy *proxy, void *user_data)
{
const char *interface, *path;
--
2.9.3
4 years, 9 months
[PATCH 1/2] hfp_hf: Fix not creating modem during NewConnection
by Luiz Augusto von Dentz
From: Luiz Augusto von Dentz <luiz.von.dentz(a)intel.com>
In case the UUIDs are not updated, or are still being resolved, when
Paired property changes a modem will never be registered.
In order to fix this problem allow modems to be registered directly
during NewConnection.
---
plugins/hfp_hf_bluez5.c | 193 +++++++++++++++++++++++++-----------------------
1 file changed, 102 insertions(+), 91 deletions(-)
diff --git a/plugins/hfp_hf_bluez5.c b/plugins/hfp_hf_bluez5.c
index 6ae98b3..fb7cc3f 100644
--- a/plugins/hfp_hf_bluez5.c
+++ b/plugins/hfp_hf_bluez5.c
@@ -499,6 +499,96 @@ static int get_version(DBusMessageIter *iter, uint16_t *version)
return -ENOENT;
}
+static gboolean has_hfp_ag_uuid(DBusMessageIter *array)
+{
+ DBusMessageIter value;
+
+ if (dbus_message_iter_get_arg_type(array) != DBUS_TYPE_ARRAY)
+ return FALSE;
+
+ dbus_message_iter_recurse(array, &value);
+
+ while (dbus_message_iter_get_arg_type(&value) == DBUS_TYPE_STRING) {
+ const char *uuid;
+
+ dbus_message_iter_get_basic(&value, &uuid);
+
+ if (g_str_equal(uuid, HFP_AG_UUID) == TRUE)
+ return TRUE;
+
+ dbus_message_iter_next(&value);
+ }
+
+ return FALSE;
+}
+
+static void modem_removed(GDBusProxy *proxy, void *user_data)
+{
+ struct ofono_modem *modem = user_data;
+
+ ofono_modem_remove(modem);
+}
+
+static void alias_changed(GDBusProxy *proxy, const char *name,
+ DBusMessageIter *iter, void *user_data)
+{
+ const char *alias;
+ struct ofono_modem *modem = user_data;
+
+ if (g_str_equal("Alias", name) == FALSE)
+ return;
+
+ dbus_message_iter_get_basic(iter, &alias);
+ ofono_modem_set_name(modem, alias);
+}
+
+static struct ofono_modem *modem_register_from_proxy(GDBusProxy *proxy,
+ const char *path)
+{
+ const char *alias, *remote;
+ DBusMessageIter iter;
+ dbus_bool_t paired;
+ struct ofono_modem *modem;
+
+ if (g_dbus_proxy_get_property(proxy, "Paired", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &paired);
+
+ if (paired == FALSE) {
+ modem = ofono_modem_find(device_path_compare, (void *) path);
+
+ if (modem != NULL) {
+ ofono_modem_remove(modem);
+ g_dbus_proxy_set_removed_watch(proxy, NULL, NULL);
+ g_dbus_proxy_set_property_watch(proxy, NULL, NULL);
+ }
+ return NULL;
+ }
+
+ if (g_dbus_proxy_get_property(proxy, "UUIDs", &iter) == FALSE)
+ return NULL;
+
+ if (has_hfp_ag_uuid(&iter) == FALSE)
+ return NULL;
+
+ if (g_dbus_proxy_get_property(proxy, "Alias", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &alias);
+
+ if (g_dbus_proxy_get_property(proxy, "Address", &iter) == FALSE)
+ return NULL;
+
+ dbus_message_iter_get_basic(&iter, &remote);
+
+ modem = modem_register(path, remote, alias);
+ g_dbus_proxy_set_property_watch(proxy, alias_changed, modem);
+ g_dbus_proxy_set_removed_watch(proxy, modem_removed, modem);
+
+ return modem;
+}
+
static DBusMessage *profile_new_connection(DBusConnection *conn,
DBusMessage *msg, void *user_data)
{
@@ -541,10 +631,18 @@ static DBusMessage *profile_new_connection(DBusConnection *conn,
modem = ofono_modem_find(device_path_compare, (void *) device);
if (modem == NULL) {
- close(fd);
- return g_dbus_create_error(msg, BLUEZ_ERROR_INTERFACE
- ".Rejected",
- "Unknown Bluetooth device");
+ GDBusProxy *proxy;
+
+ proxy = g_dbus_proxy_new(bluez, device, BLUEZ_DEVICE_INTERFACE);
+ modem = modem_register_from_proxy(proxy, device);
+ g_dbus_proxy_unref(proxy);
+
+ if (!modem) {
+ close(fd);
+ return g_dbus_create_error(msg, BLUEZ_ERROR_INTERFACE
+ ".Rejected",
+ "Unknown Bluetooth device");
+ }
}
err = service_level_connection(modem, fd, version);
@@ -686,93 +784,6 @@ static void connect_handler(DBusConnection *conn, void *user_data)
HFP_EXT_PROFILE_PATH, NULL, features);
}
-static gboolean has_hfp_ag_uuid(DBusMessageIter *array)
-{
- DBusMessageIter value;
-
- if (dbus_message_iter_get_arg_type(array) != DBUS_TYPE_ARRAY)
- return FALSE;
-
- dbus_message_iter_recurse(array, &value);
-
- while (dbus_message_iter_get_arg_type(&value) == DBUS_TYPE_STRING) {
- const char *uuid;
-
- dbus_message_iter_get_basic(&value, &uuid);
-
- if (g_str_equal(uuid, HFP_AG_UUID) == TRUE)
- return TRUE;
-
- dbus_message_iter_next(&value);
- }
-
- return FALSE;
-}
-
-static void modem_removed(GDBusProxy *proxy, void *user_data)
-{
- struct ofono_modem *modem = user_data;
-
- ofono_modem_remove(modem);
-}
-
-static void alias_changed(GDBusProxy *proxy, const char *name,
- DBusMessageIter *iter, void *user_data)
-{
- const char *alias;
- struct ofono_modem *modem = user_data;
-
- if (g_str_equal("Alias", name) == FALSE)
- return;
-
- dbus_message_iter_get_basic(iter, &alias);
- ofono_modem_set_name(modem, alias);
-}
-
-static void modem_register_from_proxy(GDBusProxy *proxy, const char *path)
-{
- const char *alias, *remote;
- DBusMessageIter iter;
- dbus_bool_t paired;
- struct ofono_modem *modem;
-
- if (g_dbus_proxy_get_property(proxy, "Paired", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &paired);
-
- if (paired == FALSE) {
- modem = ofono_modem_find(device_path_compare, (void *) path);
-
- if (modem != NULL) {
- ofono_modem_remove(modem);
- g_dbus_proxy_set_removed_watch(proxy, NULL, NULL);
- g_dbus_proxy_set_property_watch(proxy, NULL, NULL);
- }
- return;
- }
-
- if (g_dbus_proxy_get_property(proxy, "UUIDs", &iter) == FALSE)
- return;
-
- if (has_hfp_ag_uuid(&iter) == FALSE)
- return;
-
- if (g_dbus_proxy_get_property(proxy, "Alias", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &alias);
-
- if (g_dbus_proxy_get_property(proxy, "Address", &iter) == FALSE)
- return;
-
- dbus_message_iter_get_basic(&iter, &remote);
-
- modem = modem_register(path, remote, alias);
- g_dbus_proxy_set_property_watch(proxy, alias_changed, modem);
- g_dbus_proxy_set_removed_watch(proxy, modem_removed, modem);
-}
-
static void proxy_added(GDBusProxy *proxy, void *user_data)
{
const char *interface, *path;
--
2.9.3
4 years, 9 months
[PATCH 0/1] atmodem: Fix SMS reception for Gemalto modems
by Vincent Cesson
The following patch fixes SMS reception for Gemalto modems. It impacts sensible
code in atmodem driver. I need to add a case for Gemalto concerning +CMT
parsing. This parsing was executed in at_parse_cmt(), which is called only by
at_cmt_notify(). Based on at_cmgr_notify(), I moved parsing code from
at_parse_cmt() to at_cmt_notify() and I added a switch case for Gemalto modems.
I moved code because I need access to data->vendor.
Vincent Cesson (1):
atmodem: Fix SMS reception for Gemalto modems
drivers/atmodem/sms.c | 66 +++++++++++++++++++++++++++++----------------------
plugins/gemalto.c | 2 +-
2 files changed, 39 insertions(+), 29 deletions(-)
--
1.9.1
4 years, 9 months
[PATCH qmi lte v4 1/6] qmi: add missing header inclusion
by Jonas Bonn
---
drivers/qmimodem/nas.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/qmimodem/nas.h b/drivers/qmimodem/nas.h
index b39b425..b4bc012 100644
--- a/drivers/qmimodem/nas.h
+++ b/drivers/qmimodem/nas.h
@@ -19,6 +19,8 @@
*
*/
+#include <stdint.h>
+
#define QMI_NAS_RESET 0 /* Reset NAS service state variables */
#define QMI_NAS_ABORT 1 /* Abort previously issued NAS command */
#define QMI_NAS_EVENT 2 /* Connection state report indication */
--
2.9.3
4 years, 9 months
