[PATCH] stk: Add busy error for the display text command

2010-11-25 Thread Lucas, GuillaumeX
From: Guillaume Lucas guillaumex.lu...@intel.com According to the sequence 1.2 of the ETSI TS 102 384 a busy screen error type must be returned for the display text proactive command when the message is not urgent and if the ME is not able to display the text. --- doc/stk-api.txt |1 +

RE: How to get the OMA DRM right object file coming from WAP Push?

2010-11-25 Thread Marcel Holtmann
Hi Aki, Can the two methods help me to handle files from WAP push? Yes. You need to implement the PushNotificationAgent interface, and register this object with oFono using the RegisterAgent() method. there is actually a test/test-push-notification Python example in the code base that

RE: [PATCH v2 5/7] text-telephony: add documentation

2010-11-25 Thread Kai.Vehmanen
Hi, On 23 Nov 2010, Lucas De Marchi wrote: +Interfaceorg.ofono.TextTelephony [...] +Properties boolean Powered [readwrite] + + This property will enable or disable the text + telephony feature in the modem. + + Text

[PATCH 2/3] Use %m instead of strerror() in syslog messages

2010-11-25 Thread Rémi Denis-Courmont
strerror() is not thread-safe, nor in the glibc implementation, nor in the POSIX specification. In my experience, it will infrequently crash when another thread uses locale data in any way, including string formatting and such... Since the Huawei driver uses thread, we might as well avoid

[PATCH 1/3] Remove stray newlines in DBG and ofono log messages

2010-11-25 Thread Rémi Denis-Courmont
--- drivers/isimodem/call-barring.c| 10 +- drivers/isimodem/call-forwarding.c |8 drivers/isimodem/call-settings.c |8 drivers/isimodem/cbs.c |2 +- drivers/isimodem/sms.c |2 +- drivers/stemodem/caif_rtnl.c |2

[PATCH 3/3] gisi: use strerror_r() instead of strerror()

2010-11-25 Thread Rémi Denis-Courmont
--- gisi/netlink.c |8 ++-- gisi/server.c | 11 --- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/gisi/netlink.c b/gisi/netlink.c index 44a4a8d..dcc322c 100644 --- a/gisi/netlink.c +++ b/gisi/netlink.c @@ -240,8 +240,12 @@ static gboolean

Re: [PATCH v2 4/7] text-telephony: implement interface/atom

2010-11-25 Thread Lucas De Marchi
Hi Denis On Wed, Nov 24, 2010 at 7:29 PM, Denis Kenzior denk...@gmail.com wrote: Hi Lucas, +     tt_set_powered(tt, enable); +     tt_send_properties_reply(tt); The oFono convention is to reply to the pending D-Bus method call and then signal the property changed signals. The examples I

[PATCH 3/3] test-stk: Add provide local info

2010-11-25 Thread Yang Gu
--- test/test-stk-menu | 36 1 files changed, 36 insertions(+), 0 deletions(-) diff --git a/test/test-stk-menu b/test/test-stk-menu index 916a527..db3f8b7 100755 --- a/test/test-stk-menu +++ b/test/test-stk-menu @@ -134,6 +134,42 @@ class

[PATCH 0/3] Patch Description

2010-11-25 Thread Yang Gu
This series of patch is to add provide local info support by requesting the terminal to send time and language info. Please comment on the following aspects as I'm not sure after reading the spec: 1. Timezone may be a number in the range -47 through +48. In struct sms_scts, timezone is defined

[PATCH 1/3] network: Use bit as size instead of byte

2010-11-25 Thread Yang Gu
--- src/network.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/network.c b/src/network.c index d203bee..83014bb 100644 --- a/src/network.c +++ b/src/network.c @@ -129,7 +129,7 @@ static char **network_operator_technologies(struct network_operator_data *opd)

Re: [PATCH 2/3] Use %m instead of strerror() in syslog messages

2010-11-25 Thread Rémi Denis-Courmont
On Thursday 25 November 2010 14:12:20 ext Marcel Holtmann, you wrote: Hi Remi, strerror() is not thread-safe, nor in the glibc implementation, nor in the POSIX specification. In my experience, it will infrequently crash when another thread uses locale data in any way, including string

[RFC] [PATCH 0/6] GPRS registration and bearer technology

2010-11-25 Thread Rémi Denis-Courmont
Hello, The following patch series adds oFono core support to report the network advertized data technology, and the per-context actual packet bearers. Rémi Denis-Courmont (6): Distinguish HSDPA, HSUPA or both as access technologies ConnectionManager: expose available data

[PATCH 2/6] ConnectionManager: expose available data technology

2010-11-25 Thread Rémi Denis-Courmont
--- drivers/atmodem/gprs.c | 18 +- drivers/isimodem/gprs.c |6 +++--- include/gprs.h |4 ++-- src/gprs.c | 31 --- 4 files changed, 42 insertions(+), 17 deletions(-) diff --git a/drivers/atmodem/gprs.c

[PATCH 1/6] Distinguish HSDPA, HSUPA or both as access technologies

2010-11-25 Thread Rémi Denis-Courmont
--- src/common.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/common.c b/src/common.c index f32153c..1088b5d 100644 --- a/src/common.c +++ b/src/common.c @@ -661,9 +661,9 @@ const char *registration_tech_to_string(int tech) case

[PATCH 3/6] Update documentation for GPRS technology

2010-11-25 Thread Rémi Denis-Courmont
--- doc/connman-api.txt |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/doc/connman-api.txt b/doc/connman-api.txt index 22e02e5..b13efd1 100644 --- a/doc/connman-api.txt +++ b/doc/connman-api.txt @@ -77,6 +77,14 @@ Properties boolean Attached [readonly]

[PATCH 4/6] Define packet switched bearers

2010-11-25 Thread Rémi Denis-Courmont
--- src/common.c | 14 ++ src/common.h | 13 + 2 files changed, 27 insertions(+), 0 deletions(-) diff --git a/src/common.c b/src/common.c index 1088b5d..e403756 100644 --- a/src/common.c +++ b/src/common.c @@ -673,6 +673,20 @@ const char

[PATCH 5/6] Core support for packet switched bearer reporting

2010-11-25 Thread Rémi Denis-Courmont
--- src/gprs.c | 32 1 files changed, 32 insertions(+), 0 deletions(-) diff --git a/src/gprs.c b/src/gprs.c index 3b0733e..0226985 100644 --- a/src/gprs.c +++ b/src/gprs.c @@ -107,6 +107,7 @@ struct context_settings { struct pri_context {

Re: [PATCH 5/6] Core support for packet switched bearer reporting

2010-11-25 Thread Rémi Denis-Courmont
Hmm, noticed I forgot to declare the new function, which makes the patch kinda useless. On Thursday 25 November 2010 17:14:37 Denis-Courmont Remi (Nokia-MS/Helsinki), you wrote: --- src/gprs.c | 32 1 files changed, 32 insertions(+), 0 deletions(-) diff

RE: CDMA Support

2010-11-25 Thread Rajesh.Nagaiah
Hi Denis, The proposal covers the basics of voice calls, data connections, messaging and network registration details. The documentation can be found under: doc/cdma-network-api.txt doc/cdma-voicecall-manager-api.txt Initially we discussed not to use voicecall manager for cdma, rather

[PATCH] stkutil: Use sms_dcs_decode to retrieve the STK character set.

2010-11-25 Thread Lucas, GuillaumeX
From: Guillaume Lucas guillaumex.lu...@intel.com --- src/stkutil.c | 12 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/src/stkutil.c b/src/stkutil.c index 377ceff..78efbf0 100644 --- a/src/stkutil.c +++ b/src/stkutil.c @@ -77,9 +77,13 @@ struct gsm_sms_tpdu {

[PATCH 2/5] text-telephony: implement interface/atom

2010-11-25 Thread Lucas De Marchi
--- Makefile.am |2 +- src/ctm.c | 334 +++ src/ofono.h |2 + 3 files changed, 337 insertions(+), 1 deletions(-) create mode 100644 src/ctm.c diff --git a/Makefile.am b/Makefile.am index 2615be7..93db7ea 100644 --- a/Makefile.am

[PATCH 3/5] text-telephony: add documentation

2010-11-25 Thread Lucas De Marchi
--- Makefile.am|2 +- doc/text-telephony-api.txt | 39 +++ 2 files changed, 40 insertions(+), 1 deletions(-) create mode 100644 doc/text-telephony-api.txt diff --git a/Makefile.am b/Makefile.am index 93db7ea..77965d4 100644 ---

[PATCH 5/5] Add script to enable/disable text-telephony support

2010-11-25 Thread Lucas De Marchi
--- test/set-tty | 25 + 1 files changed, 25 insertions(+), 0 deletions(-) create mode 100755 test/set-tty diff --git a/test/set-tty b/test/set-tty new file mode 100755 index 000..67876af --- /dev/null +++ b/test/set-tty @@ -0,0 +1,25 @@ +#!/usr/bin/python +

[PATCH 2/5] phonesim: Add modem reset trigger

2010-11-25 Thread Gustavo F. Padovan
--- plugins/phonesim.c | 27 +++ 1 files changed, 27 insertions(+), 0 deletions(-) diff --git a/plugins/phonesim.c b/plugins/phonesim.c index d2faf42..14dae47 100644 --- a/plugins/phonesim.c +++ b/plugins/phonesim.c @@ -237,6 +237,30 @@ static void

[PATCH 3/5] modem: add support to restore state when resetting the modem

2010-11-25 Thread Gustavo F. Padovan
--- src/modem.c | 70 +- 1 files changed, 40 insertions(+), 30 deletions(-) diff --git a/src/modem.c b/src/modem.c index 6233944..1828a3b 100644 --- a/src/modem.c +++ b/src/modem.c @@ -70,6 +70,7 @@ struct ofono_modem { guint

[PATCH 4/5] Add Lockdown property to Modem interface

2010-11-25 Thread Gustavo F. Padovan
Setting Lockdown to TRUE means power down the modem and hold a lock that only permits the lock's owner power up the modem back. When released it restores the last state of the modem before holding the lock. --- doc/modem-api.txt | 10 + src/modem.c | 105

Re: [PATCH 0/3] Patch Description

2010-11-25 Thread andrzej zaborowski
Hi Yang, On 25 November 2010 13:28, Yang Gu yang...@intel.com wrote: This series of patch is to add provide local info support by requesting the terminal to send time and language info. Please comment on the following aspects as I'm not sure after reading the spec: 1. Timezone may be a

Re: [PATCH 2/3] stk: Handle provide local info proactive command

2010-11-25 Thread andrzej zaborowski
On 25 November 2010 13:29, Yang Gu yang...@intel.com wrote: ---  src/smsutil.c  |    6 +-  src/stk.c      |  113 +  src/stkagent.c |  153  src/stkagent.h |   14 +  src/stkutil.c  |    2 +-

RE: How to get the OMA DRM right object file coming from WAP Push?

2010-11-25 Thread Lin, Mengdong
Thank you so much for your information! But I'm a little confused about the relationship between WAP push and SMS. The spec said WAP push is running on top of HTTP (OTA-HTTP) or WSP (OTA-WSP). But from the oFono source, it seems WAP push is on top of SMS. Is this a contradiction? Or I

RE: [PATCH 0/3] Patch Description

2010-11-25 Thread Gu, Yang
Hi Andrew, -Original Message- From: ofono-boun...@ofono.org [mailto:ofono-boun...@ofono.org] On Behalf Of andrzej zaborowski Sent: Friday, November 26, 2010 7:26 AM To: ofono@ofono.org Subject: Re: [PATCH 0/3] Patch Description Hi Yang, On 25 November 2010 13:28, Yang Gu

[RFC 1/7] simutil: Add EFcfis and EFcphs-cff file Ids

2010-11-25 Thread Jeevaka Badrappan
--- src/simutil.h |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/simutil.h b/src/simutil.h index 5b56099..92b2e0f 100644 --- a/src/simutil.h +++ b/src/simutil.h @@ -25,6 +25,7 @@ enum sim_fileid { SIM_EFIMG_FILEID = 0x4F20, SIM_EFLI_FILEID = 0x6f05,

[RFC 2/7] call-forwarding: Read/Write cfis/cphs-cff

2010-11-25 Thread Jeevaka Badrappan
--- src/call-forwarding.c | 260 ++--- 1 files changed, 248 insertions(+), 12 deletions(-) diff --git a/src/call-forwarding.c b/src/call-forwarding.c index 715ce02..6d65400 100644 --- a/src/call-forwarding.c +++ b/src/call-forwarding.c @@ -34,6 +34,7

Read/Write EFcfis/EFcphs-cff files

2010-11-25 Thread Jeevaka Badrappan
Hi, This patch reads and writes the call forwarding unconditional status from and to the SIM depending on the SIM file availability. New property needs to be added due to the fact that number won't be available from the cphs-cff file. Incase of SIM, EFcphs-cff file holds call forwarding status

[RFC 3/7] ifx: Move call forwarding to post sim

2010-11-25 Thread Jeevaka Badrappan
--- plugins/ifx.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/plugins/ifx.c b/plugins/ifx.c index 037273a..cd6e667 100644 --- a/plugins/ifx.c +++ b/plugins/ifx.c @@ -703,6 +703,7 @@ static void ifx_post_sim(struct ofono_modem *modem) ofono_stk_create(modem,

[RFC 4/7] isigen: Move call forwarding to post sim

2010-11-25 Thread Jeevaka Badrappan
--- plugins/isigen.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/plugins/isigen.c b/plugins/isigen.c index fad4e20..f75ec0a 100644 --- a/plugins/isigen.c +++ b/plugins/isigen.c @@ -402,6 +402,7 @@ static void isigen_post_sim(struct ofono_modem *modem)

[RFC 5/7] plugins/n900: Move call forwarding to post sim

2010-11-25 Thread Jeevaka Badrappan
--- plugins/n900.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/plugins/n900.c b/plugins/n900.c index a9e6b59..3f6aa39 100644 --- a/plugins/n900.c +++ b/plugins/n900.c @@ -491,6 +491,7 @@ static void n900_post_sim(struct ofono_modem *modem) DBG();

[RFC 6/7] phonesim: Move call forwarding to post sim

2010-11-25 Thread Jeevaka Badrappan
--- plugins/phonesim.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/plugins/phonesim.c b/plugins/phonesim.c index d2faf42..e29d04d 100644 --- a/plugins/phonesim.c +++ b/plugins/phonesim.c @@ -470,6 +470,8 @@ static void phonesim_post_sim(struct ofono_modem *modem)

[RFC 7/7] doc: Add new property to call forwarding

2010-11-25 Thread Jeevaka Badrappan
--- doc/call-forwarding-api.txt |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/doc/call-forwarding-api.txt b/doc/call-forwarding-api.txt index 067531a..5dfb48e 100644 --- a/doc/call-forwarding-api.txt +++ b/doc/call-forwarding-api.txt @@ -57,3 +57,8 @@ Properties

Re: [RFC 7/7] doc: Add new property to call forwarding

2010-11-25 Thread Sankar
Hi Jeevaka, On Fri, Nov 26, 2010 at 11:53 AM, Jeevaka Badrappan jeevaka.badrap...@elektrobit.com wrote: --- doc/call-forwarding-api.txt |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/doc/call-forwarding-api.txt b/doc/call-forwarding-api.txt index

RE: [RFC 7/7] doc: Add new property to call forwarding

2010-11-25 Thread Jeevaka.Badrappan
Hi Sankar, As per the specification, querying of group of supplementary services is not allowed. Do you propose a solution where core queries all the three call forwardings namely CF busy, CF Not Reachable, CF noreply and updating the clients with the status using the above property after