Re: [PATCH 2/2] netreg: Add telit version for ciev notification

2012-09-12 Thread Christopher Vogl
Hi Denis, Patch has been applied, however: @@ -1618,6 +1649,19 @@ static void cind_support_cb(gboolean ok, GAtResult *result, gpointer user_data) at_cmer_query_cb, netreg, NULL); g_at_chat_register(nd-chat, +CIEV:, ciev_notify, FALSE, netreg, NULL);

Re: Getting started (serial GPRS modem)

2012-09-12 Thread Erik
Hi Denis! Thanks a lot for your reply! On Wed, Sep 12, 2012 at 6:49 AM, Denis Kenzior denk...@gmail.com wrote: Formal getting started documentation is lacking at the moment. Documentation contributions are always welcome ;) I'll write something up if I end up doing something with ofono. I

Re: Getting started (serial GPRS modem)

2012-09-12 Thread Denis Kenzior
Hi Erik, I picked a driver at random. What I was looking for was basically to see if I could at least see some trace of AT commands being sent. I'll care about if they are the right ones when I understand how stuff work. Something apparently understood that it should use the port I specified.

[PATCHv2 0/6] mmsd: support delivery_ind notification

2012-09-12 Thread Ronald Tessier
These patches concern mmsd and add support for delivery report notification. I splitted the old Process delivery_ind notification patch in 2 parts to use MMS_META_UUID_SUFFIX in a separate patch and process delivery_ind notification in the other one. The ReportChanged signal documentation now

[PATCHv2 1/6] service: Use MMS_META_UUID_SUFFIX

2012-09-12 Thread Ronald Tessier
--- src/service.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/service.c b/src/service.c index b3ecc1e..ab88370 100644 --- a/src/service.c +++ b/src/service.c @@ -1514,15 +1514,17 @@ static void load_messages(struct mms_service *service)

[PATCHv2 3/6] service: Process delivery_ind notification

2012-09-12 Thread Ronald Tessier
--- src/service.c | 104 + 1 file changed, 104 insertions(+) diff --git a/src/service.c b/src/service.c index ab88370..e867aa1 100644 --- a/src/service.c +++ b/src/service.c @@ -1300,6 +1300,107 @@ static void emit_service_removed(struct

[PATCHv2 2/6] mmsutil: Decode delivery_ind msg

2012-09-12 Thread Ronald Tessier
--- src/mmsutil.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/src/mmsutil.c b/src/mmsutil.c index a9a12eb..69e4b1d 100644 --- a/src/mmsutil.c +++ b/src/mmsutil.c @@ -964,6 +964,23 @@ static gboolean decode_send_req(struct wsp_header_iter *iter,

[PATCHv2 4/6] doc: Add ReportChanged signal description

2012-09-12 Thread Ronald Tessier
--- doc/message-api.txt |8 1 file changed, 8 insertions(+) diff --git a/doc/message-api.txt b/doc/message-api.txt index 2922e5d..033253c 100644 --- a/doc/message-api.txt +++ b/doc/message-api.txt @@ -31,6 +31,14 @@ Signals PropertyChanged(string name, variant value)

[PATCHv2 6/6] service: Send a delivery changed signal

2012-09-12 Thread Ronald Tessier
--- src/service.c | 125 +++-- 1 file changed, 121 insertions(+), 4 deletions(-) diff --git a/src/service.c b/src/service.c index e867aa1..f72bb6f 100644 --- a/src/service.c +++ b/src/service.c @@ -80,6 +80,15 @@ static const char

[PATCHv2 5/6] test: Add ReportChanged to monitored signals

2012-09-12 Thread Ronald Tessier
--- test/monitor-mms | 13 + 1 file changed, 13 insertions(+) diff --git a/test/monitor-mms b/test/monitor-mms index 188d028..ab77765 100755 --- a/test/monitor-mms +++ b/test/monitor-mms @@ -29,6 +29,13 @@ def property_changed(name, value, member, path, interface): iface =

Re: [PATCH 2/2] netreg: Add telit version for ciev notification

2012-09-12 Thread Denis Kenzior
Hi Christopher, Also, I made failing to initialize +CMER fatal in commit b274e74102c7a4b96c06dba895cf0cb444fa54c3. Can you please double-check that it is working as intended, I only performed basic sanity checking on my end. I sent the CMER-string that is not supported by Telit (3,0,0,1) and

Re: [PATCH 2/2] netreg: Add telit version for ciev notification

2012-09-12 Thread Christopher Vogl
Hi Denis, Strange, ofono shouldn't even let you attach or activate the context until after the netreg atom is fully initialized. Do you have a log? I don't know how this could have happened, but before I tried it two times and I surely saw the ppp-setup after netreg was removed (should

[PATCH] TODO: Set number of significant digits in settings

2012-09-12 Thread Ronald Tessier
This patch concerns mmsd. The number of significant digits is country dependant, mmsd cannot know the local numbering plan and so needs this information from the upper layer. --- TODO | 13 + 1 file changed, 13 insertions(+) diff --git a/TODO b/TODO index 6e54f6f..3635a2f 100644

Re: Getting started (serial GPRS modem)

2012-09-12 Thread Daniele Palmas
Hi Erik, Da: ofono-boun...@ofono.org [ofono-boun...@ofono.org] per conto di Erik [stim...@gmail.com] Inviato: mercoledì 12 settembre 2012 10.40 A: ofono@ofono.org Oggetto: Re: Getting started (serial GPRS modem) I found Android's reference-ril code (not trying to offend anyone, not sure about

Re: Getting started (serial GPRS modem)

2012-09-12 Thread Andrea Galbusera
Hi Daniele, On Wed, Sep 12, 2012 at 5:55 PM, Daniele Palmas daniele.pal...@telit.com wrote: Hi Erik, Da: ofono-boun...@ofono.org [ofono-boun...@ofono.org] per conto di Erik [stim...@gmail.com] Inviato: mercoledì 12 settembre 2012 10.40 A: ofono@ofono.org Oggetto: Re: Getting started