[PATCH] qmi/sms: require WMS version >= 1.2 for bearer calls
by Alexander Couzens
I've never seen a major 0 service. The gobi2000 comes with
WMS 1.0 and doesn't support the bearer command. Guessing
it's 1.2 required.
---
drivers/qmimodem/sms.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/qmimodem/sms.c b/drivers/qmimodem/sms.c
index 91501b75c4f4..3f4bdeb2a09c 100644
--- a/drivers/qmimodem/sms.c
+++ b/drivers/qmimodem/sms.c
@@ -277,7 +277,7 @@ static void qmi_bearer_query(struct ofono_sms *sms,
DBG("");
- if (data->major < 1 && data->minor < 2)
+ if (data->major < 1 || (data->major == 1 && data->minor < 2))
goto error;
if (qmi_service_send(data->wms, QMI_WMS_GET_DOMAIN_PREF, NULL,
@@ -315,7 +315,7 @@ static void qmi_bearer_set(struct ofono_sms *sms, int bearer,
DBG("bearer %d", bearer);
- if (data->major < 1 && data->minor < 2)
+ if (data->major < 1 || (data->major == 1 && data->minor < 2))
goto error;
domain = bearer_to_domain(bearer);
--
2.14.1
3 years, 7 months
APN provisioning with connman (and duplicate profiles in MBPI)
by Matthijs Kooijman
Hey folks,
I just tried Connman and oFono on a Debian stretch system for the first
time, hoping to get a more robust solution for keeping a cellular and/or
ethernet connection up. It seems the duo is quite capable of doing what
I need, but there were a few bumps in setting this up, where I think
things could be improved.
Some of these issues might be more connman issues, but they're mostly on
the cross-over area between the two, so I'll post them here for now.
In somewhat chronological order:
- I could not find any documentation on how to get a cellular
connection up. Plugging in a dongle made the "cellular" technology
appear in connman, but no services, with no indication of what needed
to happen next.
- When provisioning in oFono fails, a single gprs context is created
without an APN. Setting the APN manually made the profile show up in
connman and (eventually) work.
It might be better to cange connman to create a service for a context
without an APN, and show an error when trying to connect it, instead
of completely hiding the context.
- There does not seem to be any official commandline interface to ofono
to figure out what is happening. There are "test" python scripts in
the source repo, but these are not shipped in the Debian package
(reported here: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=868377)
- When mobile-broadband-provider-info provision finds multiple matches
for the operator (that is, multiple apns all with the same usage
type), provisioning fails, disregarding *all* apns found. I believe
this even happens when there is just one type duplicate (e.g. if
multiple mms profiles are available and just one internet profile,
the internet one is not used). Not entirely sure, though.
- When multiple profiles match, it would not make sense for oFono to
just pick an arbitrary one to use. Ideally, it would just create
a context for each, and let the user select the right context. For
this to actually work, the user must be able to distinguish them, and
see all the relevant details for profile/context to allow choosing
the right one.
I tried simply passing "allow_duplicates=TRUE" to ..., which does
indeed create all contexts, but seems to show only the first two as
aservices in connman (not sure why just these two), and there is no
way to really distinguish them (the service name is just the
currently connected network name).
- When actually connecting from connman (or activating a context using
the ofono test/activate-context script), the reported errors are very
unclear. I only got a "Not implemented" error, which is very
non-descriptive and, looking at the code, is returned in quite some
different places. There's also nothing in the log to indicate what
went wrong.
The activation error in my case was due to a missing tun kernel
module, I submitted a separate patch for that.
- I also ran into an issue where my modem was not properly detected at
startup, debug output showed it couldn't detect the USB vidpid for
some subdevices. I suspect there might be a race condition involved
there, but I'm still investigating.
Gr.
Matthijs
3 years, 7 months
[PATCH] sms: Pass const pointer to dispatch_app_datagram
by Slava Monich
---
src/sms.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/sms.c b/src/sms.c
index 17c5a9c..b86158e 100644
--- a/src/sms.c
+++ b/src/sms.c
@@ -1171,7 +1171,8 @@ static gboolean compute_incoming_msgid(GSList *sms_list,
static void dispatch_app_datagram(struct ofono_sms *sms,
const struct ofono_uuid *uuid,
int dst, int src,
- unsigned char *buf, unsigned len,
+ const unsigned char *buf,
+ unsigned int len,
const struct sms_address *addr,
const struct sms_scts *scts)
{
--
1.9.1
3 years, 7 months
SIM not starting up reliably on rilmodem
by Joey Hewitt
I'm running oFono with the rilmodem driver on an MSM8930AB-based phone
(Samsung GS4 Mini GT-I9195). I'm having issues getting it to come online
reliably. oFono can always connect to rild and get device identifiers
etc., but often when I set the modem property Online to true, it doesn't
come all the way up. It will report being Online, and rild sends some
things like signal strength updates, but most of the interfaces and
properties oFono offers aren't available in this state (e.g., SimManager
reports SIM not Present.) When it happens I kill ofonod and try again
until it works. It's pretty easy to spot, because a successful
connection causes a flurry of SIM I/O calls. I haven't used it
extensively yet, but network state, SMS, and calls are working in my tests.
Here are a couple of snippets from logs at different times:
E/RILD (30928): radioState(0) so did not update simState
E/RILD ( 3655): SetSimOperator: invalid operator
E/RILD ( 3655): OnRadioStateChanged: sim operator not ready wait for read
ofonod[19106]: [0,0014]< RIL_REQUEST_SET_PREFERRED_NETWORK_TYPE failed
RADIO_NOT_AVAILABLE
E/RILD (19068): Blocked preferred net type(1) due to sim state(0)
It's almost as if there is a circular dependency between the SIM and the
radio state, that resolves itself if the timing comes out right. I can
give more detailed logs if something specific would be helpful.
From comparing some logs and making some guesses, I arrived at a hack
of sending RIL_REQUEST_SCREEN_STATE = TRUE as part of the rild
connection initialization, and sending TRUE again later in
ril_radio_settings_probe(). This makes the startup more likely to
succeed, but still not perfect (it changed from taking 10-20 tries to
something like five.) I don't know if one of these changes is more
effective than the other, but it's good enough for now.
So, has anyone else seen this kind of thing, or have ideas on how to
make it more reliable? Also, what is the state of the rilmodem fork at
https://github.com/rilmodem/ofono ? I'm using the mainline oFono code
right now.
Thanks,
Joey Hewitt
3 years, 8 months
Re: AW: AW: AW: connmand[186]: Online check failed but running dhclient manually fixes this issue
by Daniel Wagner
Hi Vinothkumar,
[sorry for the last mails where I got your last name mixed up...]
On 08/10/2017 07:03 AM, Eswaran Vinothkumar (BEG/PJ-IOT-EL) wrote:
> Could you send me the exact modem description you are using? So that we can write quirk?
>
> Hi,
>
> I am using TELIT 910 EUG Modem.
Okay, maybe that is already enough. I saw that in drivers/telit.c there
is already some code to handle variants of the modem. The matching is
done on these strings.
Are there any vid/pid available fot his modem?
Thanks,
Daniel
3 years, 8 months
Re: AW: AW: connmand[186]: Online check failed but running dhclient manually fixes this issue
by Daniel Wagner
[added ofono mailing list]
Summery: The qmi modem returns since 475b789f3de9 ("qmi: retrieve GPRS
context parameters") the IP configuration which is sometimes incorrect.
If ConnMan gets the IP configuration it doesn't do any DHCP and applies
the not working IP configuration.
On 07/26/2017 10:11 AM, Eswaran Vinothkumar (BEG/PJ-IOT-EL) wrote:
> On 07/21/2017 09:31 AM, Daniel Wagner wrote:
>> The first thing I noticed, is that we do not extract the DNS settings
>> correctly. oFono sends 10.105.16.254 and 10.105.144.254. I starred at
>> the oFono code base and also at ConnMan and I don't see what's going
>> wrong.
>>
>> Jul 21 05:47:02 alen-iot-gateway ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb()
>> Jul 21 05:47:02 alen-iot-gateway ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb() APN:
>> internetd.gdsp Jul 21 05:47:02 alen-iot-gateway ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb() PDP
>> type 0 Jul 21 05:47:02 alen-iot-gateway ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb() IP
>> addr: 10.249.97.226 Jul 21 05:47:02 alen-iot-gateway ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb()
>> Gateway: 10.249.97.225 Jul 21 05:47:02 alen-iot-gateway ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb()
>> Gateway netmask: 255.255.255.252 Jul 21 05:47:02 alen-iot-gateway
>> ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb()
>> Primary DNS: 10.105.16.254 Jul 21 05:47:02 alen-iot-gateway
>> ofonod[176]:
>> ../ofono-1.20/drivers/qmimodem/gprs-context.c:get_settings_cb()
>> Secondary DNS: 10.105.144.254
>>
>>
>> Jul 21 05:47:02 alen-iot-gateway connmand[175]:
>> ../connman-1.34/plugins/ofono.c:extract_ipv4_settings() Interface
>> wwan0 Jul 21 05:47:02 alen-iot-gateway connmand[175]:
>> ../connman-1.34/plugins/ofono.c:extract_ipv4_settings() index 6 Jul 21
>> 05:47:02 alen-iot-gateway connmand[175]:
>> ../connman-1.34/plugins/ofono.c:extract_ipv4_settings() Method static
>> Jul 21 05:47:02 alen-iot-gateway connmand[175]:
>> ../connman-1.34/plugins/ofono.c:extract_ipv4_settings() Address
>> 10.249.97.226 Jul 21 05:47:02 alen-iot-gateway connmand[175]:
>> ../connman-1.34/plugins/ofono.c:extract_ipv4_settings() Netmask
>> 255.255.255.252 Jul 21 05:47:02 alen-iot-gateway connmand[175]:
>> ../connman-1.34/plugins/ofono.c:extract_ipv4_settings() Gateway
>> 10.249.97.225 Jul 21 05:47:02 alen-iot-gateway connmand[175]:
>> ../connman-1.34/plugins/ofono.c:extract_ipv4_settings() Nameservers
>> 10.105.144.254 10.105.144.254
>>
>> Could you add some printfs to the extract_nameservers() in
>> plugins/ofono.c? And you might also want to run
>> ofono/test/monitor-ofono so that you can see what oFono sends to
>> ConnMan.
>>
>>
>> Address: 10.249.97.226 00001010.11111001.01100001.111000 10
>> Netmask: 255.255.255.252 = 30 11111111.11111111.11111111.111111 00
>> Wildcard: 0.0.0.3 00000000.00000000.00000000.000000 11
>> =>
>> Network: 10.249.97.224/30 00001010.11111001.01100001.111000 00 (Class A)
>> Broadcast: 10.249.97.227 00001010.11111001.01100001.111000 11
>> HostMin: 10.249.97.225 00001010.11111001.01100001.111000 01
>> HostMax: 10.249.97.226 00001010.11111001.01100001.111000 10
>> Hosts/Net: 2 (Private Internet)
>>
>> Looking through the log, it looks like ConnMan does all the math
>> correctly.
>>
>> One thing which looks inconsistent in your first report is:
>>
>> ofono_start_gsm.sh[192]: Interface is wwan0
>> ofono_start_gsm.sh[192]: IP address is 10.249.29.20
>> ofono_start_gsm.sh[192]: Gateway is 10.249.29.21
>> ofono_start_gsm.sh[192]: Nameserver is 10.105.144.254
>>
>>
>> Destination Gateway Genmask Flags Metric Ref Use Iface
>> 0.0.0.0 10.166.199.10 0.0.0.0 UG 0 0 0 wwan0
>> 10.105.144.254 10.166.199.10 255.255.255.255 UGH 0 0 0 wwan0
>> 10.166.199.8 0.0.0.0 255.255.255.252 U 0 0 0 wwan0
>> 10.166.199.10 0.0.0.0 255.255.255.255 UH 0 0 0 wwan0
>>
>> The gateway should be 10.249.29.21 and not 10.166.199.10.
>> The next three lines are adding a host routing for the DNS server and
>> maybe timeserver. At least the first line is the DNS server but again
>> the gateway is wrong.
>>
>> dhclient gets set a complete different gateway:
>>
>> Destination Gateway Genmask Flags Metric Ref Use Iface
>> 0.0.0.0 10.166.250.1 0.0.0.0 UG 0 0 0 wwan0
>> 10.166.250.0 0.0.0.0 255.255.255.240 U 0 0 0 wwan0
>> 192.168.1.0 0.0.0.0 255.255.255.0 U 0 0 0 eth0
>> 192.168.1.255 0.0.0.0 255.255.255.255 UH 0 0 0 eth0
>>
>> 10.166.250.1 with a different netmask!
>>
>> First thing I would do is to verify what the oFono modem is doing. If
>> that one extracts the wrong information and passes that to ConnMan it
>> would explain your problem. The code in oFono is pretty new
>>
>> 475b789f3de9 ("qmi: retrieve GPRS context parameters")
> Just my 2 cents on this topic.
> On my setup, I use another QMI modem: MC7304 with ofono 1.20 + a bunch of patches done internally and connman 1.34.
>
> I also had problem with this patch (475b789f3de9 ("qmi: retrieve GPRS context parameters")).
> I have reverted it or to be more precise I commented all calls to functions ofono_gprs_context_set_ipv4* functions.
> I have done that 3 months ago so I am not completely sure about the why:
> Here is what I wrote in my patch:
>
> The goal is to force connman to use DHCP.
> When connman don't do that, MC7304 fails to ping.
> This behavior has been seen by libqmi guys:
> https://lists.freedesktop.org/archives/libqmi-devel/2016-June/001617.html
>
> Maybe you have the same problem with your TELIT modem.
> I can send my patch if you want but it is not the best way to handle the problem.
Thanks for your report. So we have at least two independent reports for
this problem.
Just for the record which ISP are you using? The above is Vodafone.
> Christophe Ronco
>
> I have also fixed the issue by creating a patch for Connman to run DHCP even when network returns static/fixed IP. Its'a not the way to do, but it’s a quick fix like yours.
If we could find some sort of rule when to ignore the returned values
from oFono than we can fallback to DHCP.
Thanks,
Daniel
3 years, 8 months
[PATCH 1/5] common: create GList helper ofono_call_compare
by Alexander Couzens
replaces at_util_call_compare (atmodem) and
call_compare (rild).
---
drivers/atmodem/atutil.c | 17 ++---------------
drivers/atmodem/atutil.h | 1 -
drivers/atmodem/voicecall.c | 2 +-
drivers/hfpmodem/voicecall.c | 2 +-
drivers/huaweimodem/voicecall.c | 2 +-
drivers/ifxmodem/voicecall.c | 2 +-
drivers/rilmodem/voicecall.c | 16 +---------------
drivers/stemodem/voicecall.c | 2 +-
src/common.c | 14 ++++++++++++++
src/common.h | 2 ++
10 files changed, 24 insertions(+), 36 deletions(-)
diff --git a/drivers/atmodem/atutil.c b/drivers/atmodem/atutil.c
index 14873483..1a48b606 100644
--- a/drivers/atmodem/atutil.c
+++ b/drivers/atmodem/atutil.c
@@ -31,6 +31,7 @@
#define OFONO_API_SUBJECT_TO_CHANGE
#include <ofono/log.h>
#include <ofono/types.h>
+#include "../src/common.h"
#include "atutil.h"
#include "vendor.h"
@@ -101,20 +102,6 @@ gint at_util_call_compare_by_id(gconstpointer a, gconstpointer b)
return 0;
}
-gint at_util_call_compare(gconstpointer a, gconstpointer b)
-{
- const struct ofono_call *ca = a;
- const struct ofono_call *cb = b;
-
- if (ca->id < cb->id)
- return -1;
-
- if (ca->id > cb->id)
- return 1;
-
- return 0;
-}
-
GSList *at_util_parse_clcc(GAtResult *result, unsigned int *ret_mpty_ids)
{
GAtResultIter iter;
@@ -173,7 +160,7 @@ GSList *at_util_parse_clcc(GAtResult *result, unsigned int *ret_mpty_ids)
else
call->clip_validity = 2;
- l = g_slist_insert_sorted(l, call, at_util_call_compare);
+ l = g_slist_insert_sorted(l, call, ofono_call_compare);
if (mpty)
mpty_ids |= 1 << id;
diff --git a/drivers/atmodem/atutil.h b/drivers/atmodem/atutil.h
index 5cb88b78..95e09900 100644
--- a/drivers/atmodem/atutil.h
+++ b/drivers/atmodem/atutil.h
@@ -54,7 +54,6 @@ void decode_at_error(struct ofono_error *error, const char *final);
gint at_util_call_compare_by_status(gconstpointer a, gconstpointer b);
gint at_util_call_compare_by_phone_number(gconstpointer a, gconstpointer b);
gint at_util_call_compare_by_id(gconstpointer a, gconstpointer b);
-gint at_util_call_compare(gconstpointer a, gconstpointer b);
GSList *at_util_parse_clcc(GAtResult *result, unsigned int *mpty_ids);
gboolean at_util_parse_reg(GAtResult *result, const char *prefix,
int *mode, int *status,
diff --git a/drivers/atmodem/voicecall.c b/drivers/atmodem/voicecall.c
index e4c59c26..6fee39bc 100644
--- a/drivers/atmodem/voicecall.c
+++ b/drivers/atmodem/voicecall.c
@@ -132,7 +132,7 @@ static struct ofono_call *create_call(struct ofono_voicecall *vc, int type,
call->clip_validity = clip;
call->cnap_validity = CNAP_VALIDITY_NOT_AVAILABLE;
- d->calls = g_slist_insert_sorted(d->calls, call, at_util_call_compare);
+ d->calls = g_slist_insert_sorted(d->calls, call, ofono_call_compare);
return call;
}
diff --git a/drivers/hfpmodem/voicecall.c b/drivers/hfpmodem/voicecall.c
index ffdf4b7b..1a282903 100644
--- a/drivers/hfpmodem/voicecall.c
+++ b/drivers/hfpmodem/voicecall.c
@@ -128,7 +128,7 @@ static struct ofono_call *create_call(struct ofono_voicecall *vc, int type,
call->phone_number.type = num_type;
}
- d->calls = g_slist_insert_sorted(d->calls, call, at_util_call_compare);
+ d->calls = g_slist_insert_sorted(d->calls, call, ofono_call_compare);
call->clip_validity = clip;
diff --git a/drivers/huaweimodem/voicecall.c b/drivers/huaweimodem/voicecall.c
index f55568d3..0887f652 100644
--- a/drivers/huaweimodem/voicecall.c
+++ b/drivers/huaweimodem/voicecall.c
@@ -76,7 +76,7 @@ static struct ofono_call *create_call(struct ofono_voicecall *vc, int type,
call->clip_validity = clip;
- d->calls = g_slist_insert_sorted(d->calls, call, at_util_call_compare);
+ d->calls = g_slist_insert_sorted(d->calls, call, ofono_call_compare);
return call;
}
diff --git a/drivers/ifxmodem/voicecall.c b/drivers/ifxmodem/voicecall.c
index 45b5ca42..581a7906 100644
--- a/drivers/ifxmodem/voicecall.c
+++ b/drivers/ifxmodem/voicecall.c
@@ -107,7 +107,7 @@ static struct ofono_call *create_call(struct ofono_voicecall *vc, int type,
call->clip_validity = clip;
- d->calls = g_slist_insert_sorted(d->calls, call, at_util_call_compare);
+ d->calls = g_slist_insert_sorted(d->calls, call, ofono_call_compare);
return call;
}
diff --git a/drivers/rilmodem/voicecall.c b/drivers/rilmodem/voicecall.c
index b7180b90..f248d871 100644
--- a/drivers/rilmodem/voicecall.c
+++ b/drivers/rilmodem/voicecall.c
@@ -117,20 +117,6 @@ done:
ofono_voicecall_disconnected(vc, reqdata->id, reason, NULL);
}
-static int call_compare(gconstpointer a, gconstpointer b)
-{
- const struct ofono_call *ca = a;
- const struct ofono_call *cb = b;
-
- if (ca->id < cb->id)
- return -1;
-
- if (ca->id > cb->id)
- return 1;
-
- return 0;
-}
-
static void clcc_poll_cb(struct ril_msg *message, gpointer user_data)
{
struct ofono_voicecall *vc = user_data;
@@ -209,7 +195,7 @@ static void clcc_poll_cb(struct ril_msg *message, gpointer user_data)
call->id, call->status, call->type,
call->phone_number.number, call->name);
- calls = g_slist_insert_sorted(calls, call, call_compare);
+ calls = g_slist_insert_sorted(calls, call, ofono_call_compare);
}
no_calls:
diff --git a/drivers/stemodem/voicecall.c b/drivers/stemodem/voicecall.c
index 356ab7c6..19276111 100644
--- a/drivers/stemodem/voicecall.c
+++ b/drivers/stemodem/voicecall.c
@@ -128,7 +128,7 @@ static struct ofono_call *create_call(struct ofono_voicecall *vc, int type,
call->clip_validity = clip;
- d->calls = g_slist_insert_sorted(d->calls, call, at_util_call_compare);
+ d->calls = g_slist_insert_sorted(d->calls, call, ofono_call_compare);
return call;
}
diff --git a/src/common.c b/src/common.c
index 17d1d58a..75bb94dc 100644
--- a/src/common.c
+++ b/src/common.c
@@ -736,3 +736,17 @@ void ofono_call_init(struct ofono_call *call)
call->cnap_validity = CNAP_VALIDITY_NOT_AVAILABLE;
call->clip_validity = CLIP_VALIDITY_NOT_AVAILABLE;
}
+
+gint ofono_call_compare(gconstpointer a, gconstpointer b)
+{
+ const struct ofono_call *ca = a;
+ const struct ofono_call *cb = b;
+
+ if (ca->id < cb->id)
+ return -1;
+
+ if (ca->id > cb->id)
+ return 1;
+
+ return 0;
+}
diff --git a/src/common.h b/src/common.h
index 05f2a851..46bae577 100644
--- a/src/common.h
+++ b/src/common.h
@@ -184,3 +184,5 @@ const char *registration_tech_to_string(int tech);
const char *packet_bearer_to_string(int bearer);
gboolean is_valid_apn(const char *apn);
+
+gint ofono_call_compare(gconstpointer a, gconstpointer b);
--
2.13.3
3 years, 8 months
[PATCH] radio-settings: Fix typo in radio_band_umts_from_string
by John Ernberg
From: John Ernberg <john.ernberg(a)actia.se>
---
src/radio-settings.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/radio-settings.c b/src/radio-settings.c
index b988e3e..c551b04 100644
--- a/src/radio-settings.c
+++ b/src/radio-settings.c
@@ -167,7 +167,7 @@ static gboolean radio_band_umts_from_string(const char *str,
enum ofono_radio_band_umts *band)
{
if (g_str_equal(str, "any")) {
- *band = OFONO_RADIO_BAND_GSM_ANY;
+ *band = OFONO_RADIO_BAND_UMTS_ANY;
return TRUE;
} else if (g_str_equal(str, "850")) {
*band = OFONO_RADIO_BAND_UMTS_850;
--
2.7.4
3 years, 8 months
gemalto: AGPS functionality
by Vincent CESSON
Hi,
I am trying to add AGPS functionality to Gemalto modems. I took a look
at interface AssistedSatelliteNavigation and implementation in
drivers/atmodem/gnss.c. Unfortunately, Gemalto does not support AT+CPOS
command but it uses a custom command: AT^SBNW. Moreover, the positioning
file expected is not XML but binary format.
The process of loading the binary is:
- Download binary file from Gemalto server to local memory
- Initiate binary write command: AT^SBNW="agps",<binary_file_length>
- Copy binary file from local memory to modem over its application (or
modem) interface
AT^SBNW immediately returns "CONNECT". Then "AGPS READY: SEND FILE..."
After the modem receives the exact amount of bytes specified in the
command, it verifies the data and returns one of the following: "AGPS
END OK", "TIME INFO ERROR", "BAD CRC" or "OK".
So I am facing several issues:
- How to send binary data from local memory to oFono? DBus message
with array of bytes?
- How to send binary data from oFono to modem interface? Does gatchat
only support strings?
- How to handle custom result codes? I already tried some hacks, and I
had to blacklist the response "CONNECT" for instance.
Or maybe it isn't worth the effort implementing it in oFono?
Regards,
Vincent
3 years, 8 months
[PATCH] ussd: Switch the state from USER_ACTION to IDLE
by Slava Monich
... when a USSD notification is received. Some networks
send 0 (no further user action required) after the response
timeout expires. That should result in the user input form
getting removed from the ME screen.
---
src/ussd.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/src/ussd.c b/src/ussd.c
index 99fa753..84f64c6 100644
--- a/src/ussd.c
+++ b/src/ussd.c
@@ -512,6 +512,20 @@ void ofono_ussd_notify(struct ofono_ussd *ussd, int status, int dcs,
ussd_change_state(ussd, new_state);
goto free;
+ } else if (ussd->state == USSD_STATE_USER_ACTION &&
+ status != OFONO_USSD_STATUS_ACTION_REQUIRED) {
+ ussd_change_state(ussd, USSD_STATE_IDLE);
+
+ if (status == OFONO_USSD_STATUS_NOTIFY && str && str[0]) {
+ const char *path = __ofono_atom_get_path(ussd->atom);
+
+ g_dbus_emit_signal(conn, path,
+ OFONO_SUPPLEMENTARY_SERVICES_INTERFACE,
+ "NotificationReceived", DBUS_TYPE_STRING,
+ &str, DBUS_TYPE_INVALID);
+ }
+
+ goto free;
} else {
ofono_error("Received an unsolicited USSD but can't handle.");
DBG("USSD is: status: %d, %s", status, str);
--
1.9.1
3 years, 8 months
