All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH] Add voice call boolean property for MO/MT distinction
Date: Thu, 25 Nov 2010 17:51:18 -0600	[thread overview]
Message-ID: <4CEEF676.3020902@gmail.com> (raw)
In-Reply-To: <20101125082310.GA17294@jh-x301>

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

Hi Johan,

On 11/25/2010 02:23 AM, Johan Hedberg wrote:
> Hi,
> 
> On Thu, Nov 25, 2010, Rémi Denis-Courmont wrote:
>>>> +		boolean Originated [readonly]
>>>> +
>>>> +			Indicates whether the call was mobile-originated
>>>> +			(true) or mobile-terminated (false).
>>>> +
>>>
>>> What use case are you trying to enable with this API?  I don't see how
>>> this could be useful for the UI...
>>
>> This becomes a problem if we have more than one application "managing" voice 
>> calls. And I expect that will _be_ the most common case. You have the main 
>> call UI, but you also have the Bluetooth headset/carkit, the USB AT commands 
>> port, and possibly some third party call applications.
> 
> Right. At least for HFP the call direction needs to be in the AT+CLCC
> reply. I.e. if the bluetoothd ofono telephony driver is used and
> bluetoothd starts after one or more calls are already in progress
> bluetoothd wouldn't have seen the call setup process and therefore not
> know the direction of the call (without this property).

I really don't view +CLCC emulation outside oFono as a valid use case.
We already have a task to implement HFP AG role inside oFono, and
especially with CDMA on the horizon this is the only way to go.

So unless there are more convincing arguments, I'm against this API
change.  I really suggest that instead of doing this outside oFono,
someone picks up the AT modem emulator work and continue where Zhenhua
left off.

Regards,
-Denis

  reply	other threads:[~2010-11-25 23:51 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-24 15:05 [PATCH] Add voice call boolean property for MO/MT distinction =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-11-24 21:31 ` Denis Kenzior
2010-11-25  7:40   ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-11-25  8:23     ` Johan Hedberg
2010-11-25 23:51       ` Denis Kenzior [this message]
2010-11-26  8:21         ` Johan Hedberg
2010-11-26 12:06       ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-11-29  9:41         ` Marcel Holtmann
2010-12-01 23:57           ` Pekka Pessi
  -- strict thread matches above, loose matches on Subject: below --
2010-11-30  8:20 Remi.Denis-Courmont
2010-11-30 10:58 ` Marcel Holtmann
2010-11-30 11:24   ` Aki Niemi
2010-11-30 11:39     ` Marcel Holtmann
2010-11-30 11:17 Remi.Denis-Courmont

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=4CEEF676.3020902@gmail.com \
    --to=denkenz@gmail.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.