Re: DUN client for oFono and BlueZ

2010-04-27 Thread Kalle Valo
Gustavo F. Padovan gust...@padovan.org writes: Hi all, Hello Gustavo, I'm starting the DUN Client implementation for the Linux Stack. DUN is the Bluetooth dial-up network profile. It makes possible share internet connection between two Bluetooth devices. That is my Google Summer of Code

Re: Option GTM 382

2010-04-27 Thread Kalle Valo
Teemu Piiroinen teemu.piiroi...@gmail.com writes: No luck yet.. Did I setup the sw correcty? - I run configure, make and make install - Added ofono.rules to /etc/udev/rules.d and /lib/udev/rules.d/ - Started ofono by running ofonod from shell Is there something else I need to do? Enable

Re: [PATCH v0] Read EF_ICCID property of SIM

2010-04-27 Thread Daniel Wagner
Hi Denis, + + extract_bcd_number(data, len, number); + number[OFONO_MAX_ICCID_LENGTH] = '\0'; + sim-iccid = g_strdup(number); + + DBG(ICCID %s, sim-iccid); +} + You might want to emit the property changed signal with the new ICCID value here. I had already the

Re: DUN client for oFono and BlueZ

2010-04-27 Thread Kalle Valo
Gustavo F. Padovan gust...@padovan.org writes: Hi Kalle, Hi Gustavo, 4. Agent server on BlueZ. This one is very similar to the HFP Agent server. At the end of the DUN agent project I plan to merge the both agent servers. SAP will take advantage of that merge too. 5. oFono DUN

Modem emulator and DUN server side for oFono and BlueZ

2010-04-27 Thread Zhang, Zhenhua
Hi, I am now working on modem emulator and one usage is for DUN server role. Since Padovan is working on client role, it's good to share my rough thinking for server side implementation. Here are the simple steps I have: 1. Create an oFono emulator atom in oFono. It's the emulator manager that

Re: [PATCH v0] Read EF_ICCID property of SIM

2010-04-27 Thread Daniel Wagner
On Tue, Apr 27, 2010 at 08:36:31AM +0200, Daniel Wagner wrote: Please use the sim file queue like EFpl does, no need to invent your own solution. I used the ofono_sim_read function but that didn't work. On the terminal I saw only: CRSM=192,12258 With read_file_transparent it is:

RE: [PATCH 2/6] stkutil: Move advance of ctlv to decicated proactive commands parser

2010-04-27 Thread Gu, Yang
Hi Andrew and Denis, -Original Message- From: ofono-boun...@ofono.org [mailto:ofono-boun...@ofono.org] On Behalf Of Denis Kenzior Sent: Monday, April 26, 2010 11:39 PM To: ofono@ofono.org Subject: Re: [PATCH 2/6] stkutil: Move advance of ctlv to decicated proactive commands parser Hi

[PATCH 1/7] stkutil: Move advance of ctlv to parse_dataobj

2010-04-27 Thread Yang Gu
Not all the proactive commands have comprehension tlv other than command details and device identities. So the appropriate code should be moved to parse_dataobj() from stk_command_new_from_pdu(). --- src/stkutil.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git

[PATCH 5/7] test-stkutil: Add unit test for more time parser

2010-04-27 Thread Yang Gu
--- unit/test-stkutil.c | 38 ++ 1 files changed, 38 insertions(+), 0 deletions(-) diff --git a/unit/test-stkutil.c b/unit/test-stkutil.c index 0961368..427d7a2 100644 --- a/unit/test-stkutil.c +++ b/unit/test-stkutil.c @@ -405,6 +405,41 @@ static void

[PATCH 4/7] stkutil: Add parser for more time proactive commands

2010-04-27 Thread Yang Gu
--- src/stkutil.c | 15 +++ 1 files changed, 15 insertions(+), 0 deletions(-) diff --git a/src/stkutil.c b/src/stkutil.c index ed48db4..2e0c957 100644 --- a/src/stkutil.c +++ b/src/stkutil.c @@ -2189,6 +2189,18 @@ static gboolean parse_get_input(struct stk_command *command,

[PATCH 3/7] test-stkutil: Refactor the implementation to test display text

2010-04-27 Thread Yang Gu
--- unit/test-stkutil.c | 160 ++ 1 files changed, 109 insertions(+), 51 deletions(-) diff --git a/unit/test-stkutil.c b/unit/test-stkutil.c index 2bea19b..0961368 100644 --- a/unit/test-stkutil.c +++ b/unit/test-stkutil.c @@ -35,15 +35,72 @@

[PATCH 6/7] stkutil: Change some dataobj parser to use char * instead of char **

2010-04-27 Thread Yang Gu
--- src/stkutil.c | 28 ++-- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git a/src/stkutil.c b/src/stkutil.c index 2e0c957..a0166d0 100644 --- a/src/stkutil.c +++ b/src/stkutil.c @@ -611,7 +611,7 @@ static gboolean parse_dataobj_location_info(struct

Re: [PATCH] Add send_method_call to g_dbus

2010-04-27 Thread Johan Hedberg
Hi, On Tue, Apr 27, 2010, Marcel Holtmann wrote: +gboolean g_dbus_send_method_call(DBusConnection *connection, const char *dest, + const char *path, const char *interface, + const char *method, int type, ...); +gboolean

Re: [PATCH v0] Read EF_ICCID property of SIM

2010-04-27 Thread Denis Kenzior
Hi Daniel, On Tue, Apr 27, 2010 at 08:36:31AM +0200, Daniel Wagner wrote: Please use the sim file queue like EFpl does, no need to invent your own solution. I used the ofono_sim_read function but that didn't work. On the terminal I saw only: CRSM=192,12258 With

Re: [PATCH 1/7] stkutil: Move advance of ctlv to parse_dataobj

2010-04-27 Thread Denis Kenzior
Hi Yang, Not all the proactive commands have comprehension tlv other than command details and device identities. So the appropriate code should be moved to parse_dataobj() from stk_command_new_from_pdu(). --- src/stkutil.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-)

Re: DUN client for oFono and BlueZ

2010-04-27 Thread Gustavo F. Padovan
Hi Bastien, * Bastien Nocera had...@hadess.net [2010-04-27 09:15:28 +0100]: On Mon, 2010-04-26 at 22:20 -0300, Gustavo F. Padovan wrote: Hi all, I'm starting the DUN Client implementation for the Linux Stack. DUN is the Bluetooth dial-up network profile. It makes possible share

Re: [PATCH v0] Read EF_ICCID property of SIM

2010-04-27 Thread Daniel Wagner
Hi Denis, On Tue, Apr 27, 2010 at 04:21:21PM +0200, Denis Kenzior wrote: Hi Daniel, On Tue, Apr 27, 2010 at 08:36:31AM +0200, Daniel Wagner wrote: Please use the sim file queue like EFpl does, no need to invent your own solution. I used the ofono_sim_read function but that

Re: [PATCH v0] Read EF_ICCID property of SIM

2010-04-27 Thread Denis Kenzior
Hi Daniel, It's a Option iCON 451. I didn't check what \r\r\n+CRSM: 144,0,03\r\n\r\nOK\r\n exaclty means. I guess I have to check the standard. (of course I did copy the wrong line) ofonod[18879]: Control: \r\r\n+CRSM: 111,0\r\n\r\nOK\r\n Yes the Option Modems always return +CRSM:

Re: [PATCH v0] Read EF_ICCID property of SIM

2010-04-27 Thread Daniel Wagner
Hi Denis, It's a Option iCON 451. I didn't check what \r\r\n+CRSM: 144,0,03\r\n\r\nOK\r\n exaclty means. I guess I have to check the standard. (of course I did copy the wrong line) ofonod[18879]: Control: \r\r\n+CRSM: 111,0\r\n\r\nOK\r\n Yes the Option Modems always

Re: [PATCH v0] Read EF_ICCID property of SIM

2010-04-27 Thread Denis Kenzior
Hi Daniel, Hi Denis, It's a Option iCON 451. I didn't check what \r\r\n+CRSM: 144,0,03\r\n\r\nOK\r\n exaclty means. I guess I have to check the standard. (of course I did copy the wrong line) ofonod[18879]: Control: \r\r\n+CRSM: 111,0\r\n\r\nOK\r\n Yes the Option

Re: [PATCH] ppp: silently discard packets if needed

2010-04-27 Thread Kristen Carlson Accardi
On Tue, 27 Apr 2010 18:12:03 -0300 Gustavo F. Padovan gust...@padovan.org wrote: Hi Kristen, * Kristen Carlson Accardi kris...@linux.intel.com [2010-04-27 13:26:49 -0700]: When receiving a packet, check the protocol and the phase to see if the packet should be dropped. ---

[PATCH] ppp: silently discard packets if needed

2010-04-27 Thread Kristen Carlson Accardi
When receiving a packet, check the protocol and the phase to see if the packet should be dropped. --- gatchat/gatppp.c | 30 +- 1 files changed, 25 insertions(+), 5 deletions(-) diff --git a/gatchat/gatppp.c b/gatchat/gatppp.c index c7bf089..e51c281 100644 ---