All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jarkko Lehtoranta <devel@jlranta.com>
To: ofono@ofono.org
Subject: [PATCH] huawei: fix AT^DDSETEX=2 timing
Date: Sat, 10 Mar 2012 14:22:27 +0200	[thread overview]
Message-ID: <4F5B4783.6000303@jlranta.com> (raw)
In-Reply-To: <1331346388.14217.36.camel@aeonflux>

[-- Attachment #1: Type: text/plain, Size: 224 bytes --]

On 03/10/2012 04:26 AM, Marcel Holtmann wrote:
> Hi Jarkko,
>
> inline patches please. I am not digging patches out of Bugzilla.
>
> Regards
>
> Marcel
Here you go, patch for the Huawei voice output port issue.



[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: ofono_fix_huawei_ddsetex.patch --]
[-- Type: text/x-patch, Size: 1626 bytes --]

>From db13dd1dcdadefc9325fd159aebef10ac4b388c6 Mon Sep 17 00:00:00 2001
From: Jarkko Lehtoranta <devel@jlranta.com>
Date: Fri, 9 Mar 2012 19:51:29 +0200
Subject: [PATCH] huawei: fix AT^DDSETEX=2 timing

---
 drivers/huaweimodem/voicecall.c |   12 +++++++++---
 1 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/drivers/huaweimodem/voicecall.c b/drivers/huaweimodem/voicecall.c
index 1ea2613..3952b95 100644
--- a/drivers/huaweimodem/voicecall.c
+++ b/drivers/huaweimodem/voicecall.c
@@ -78,9 +78,6 @@ static struct ofono_call *create_call(struct ofono_voicecall *vc, int type,
 
 	d->calls = g_slist_insert_sorted(d->calls, call, at_util_call_compare);
 
-	g_at_chat_send(d->chat, "AT^DDSETEX=2", none_prefix,
-						NULL, NULL, NULL);
-
 	return call;
 }
 
@@ -117,6 +114,7 @@ static void huawei_dial(struct ofono_voicecall *vc,
 				ofono_voicecall_cb_t cb, void *data)
 {
 	char buf[256];
+	struct voicecall_data *vd = ofono_voicecall_get_data(vc);
 
 	if (ph->type == 145)
 		snprintf(buf, sizeof(buf), "ATD+%s", ph->number);
@@ -137,12 +135,20 @@ static void huawei_dial(struct ofono_voicecall *vc,
 	strcat(buf, ";");
 
 	huawei_template(vc, buf, cb, data);
+	
+	g_at_chat_send(vd->chat, "AT^DDSETEX=2", none_prefix,
+						NULL, NULL, NULL);
 }
 
 static void huawei_answer(struct ofono_voicecall *vc,
 				ofono_voicecall_cb_t cb, void *data)
 {
+	struct voicecall_data *vd = ofono_voicecall_get_data(vc);
+	
 	huawei_template(vc, "ATA", cb, data);
+	
+	g_at_chat_send(vd->chat, "AT^DDSETEX=2", none_prefix,
+						NULL, NULL, NULL);
 }
 
 static void huawei_hangup(struct ofono_voicecall *vc,
-- 
1.7.1


  reply	other threads:[~2012-03-10 12:22 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-09 23:49 Huawei voice output port issue + patch Jarkko Lehtoranta
2012-03-10  2:26 ` Marcel Holtmann
2012-03-10 12:22   ` Jarkko Lehtoranta [this message]
  -- strict thread matches above, loose matches on Subject: below --
2012-03-10 16:46 [PATCH] huawei: fix AT^DDSETEX=2 timing devel
2012-03-10 16:46 ` devel
2012-03-23 19:13   ` Marcel Holtmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F5B4783.6000303@jlranta.com \
    --to=devel@jlranta.com \
    --cc=ofono@ofono.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.