All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH] doc: Add experimental handsfree-audio API
Date: Wed, 20 Feb 2013 09:45:04 -0600	[thread overview]
Message-ID: <5124EF80.2060606@gmail.com> (raw)
In-Reply-To: <CANT-zCWRxEiYLNV07fsm6xfV8s+kDHE9dsgxrq8SmxOamLWzgQ@mail.gmail.com>

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

Hi Mikel,

> However, I still see some value in having a more coupled relation
> between modems and audio-cards, beyond the RemoteAddress/LocalAddress
> pair. This is specially interesting for the HS role, since otherwise
> both modems and audio-cards need to be tracked and matched.
>
> The two roles might actually need to be separated. After all, there
> are some fundamental differences: audio-cards representing a remote
> gateway would be statically associated to a modem. Actually the modem
> *represents* this device, so signals such as CardAdded/CardRemoved
> provide little value.
>

This API is meant specifically for PA or another Audio subsystem.  What 
you want to be tracking and cross-referencing I'm not quite sure of.

> On the other hand, audio-cards representing remote headsets are more
> dynamic: they come and go depending on the number of paired headsets
> (or more likely, the connected ones - it's not clear what "attached"
> means in the doc).

In practice, attached means an established SLC connection.

>
> One possibility would be to register HandsfreeAudioCard in the modem
> object path for the HS role (as I proposed before) and dynamically
> register the necessary objects for the AG role. In the later case, the
> connected headsets could be announced as in Denis' original proposal
> (perhaps renamed to HeadsetAdded/HeadsetRemoved).
>

Perhaps, but why would you be using this API and not one from PA?

> The ability to distinguish HS and AG roles in the client's side is
> desirable (at least PulseAudio heavily depends on this information).
> So even if the proposal above is not accepted, at least the role
> (UUID) would have to be exposed somehow.
>

The type information was left out on purpose from this proposal.  Likely 
it is necessary, but for the initial phase it does not seem relevant.  I 
also have yet to hear concise and clear summary on the utility of 
providing this information.

>>> If so, I have some concerns about the potential race conditions. If
>>> SCO was closed and reopen immediately afterwards, the client could
>>> receive the (second) NewConnection before the HUP.
>>>

You are going over-the-air vs local machine.  I really do not see this 
ever happening.  But even if it does, is this really a hard problem for 
the Agent to deal with?

Regards,
-Denis

  reply	other threads:[~2013-02-20 15:45 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-20  8:04 [PATCH] doc: Add experimental handsfree-audio API Mikel Astiz
2013-02-20  8:55 ` Mikel Astiz
2013-02-20  9:19   ` Frederic Danis
2013-02-20  9:19   ` Johan Hedberg
2013-02-20 11:04     ` Mikel Astiz
2013-02-20 15:45       ` Denis Kenzior [this message]
2013-02-20 16:50         ` Mikel Astiz
2013-02-20 17:34           ` Denis Kenzior
2013-02-22 10:06             ` Mikel Astiz
2013-02-22 19:14               ` Denis Kenzior

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=5124EF80.2060606@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.