From: Frederic Danis <frederic.danis@linux.intel.com>
To: Mikel Astiz <mikel.astiz.oss@gmail.com>
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH v11 01/12] audio: Move telephony drivers to D-Bus interface
Date: Fri, 06 Jul 2012 15:07:42 +0200 [thread overview]
Message-ID: <4FF6E31E.70507@linux.intel.com> (raw)
In-Reply-To: <CANT-zCVZx5QcPkh4Zeru1GvZrOyt2twqB9sy2Zz4KCRZWq_+YQ@mail.gmail.com>
Hello Mikel,
On 05/07/2012 16:21, Mikel Astiz wrote:
<snip>
>> +Interaction with the audio component (i.e. PulseAudio) will be done through the
>> +MediaTransport object (passed to telephony agent during NewConnection call).
>
> This is where the interesting part begins: how exactly are you
> planning to do this?
>
> Perhaps I'm missing something but I see no relation between Media
> transport and telephony agent in your patchset. It seems this part of
> the documentation is inconsistent with the code.
The Bluez MediaTransport will act as central point for messages related
to audio.
e.g. for volume management, PulseAudio or oFono will call SetProperty
method of MediaTransport which will send a property changed signal.
This behavior is implemented through existing MediaTransport signals and
patch 8.
> I don't understand your original motivation to do this, but from my
> point of view, something similar would actually be interesting to
> propagate call-states (HFP) from oFono to PulseAudio. This information
> can later be useful for audio routing policies in PulseAudio.
IMO, component implementing audio routing policy should listen oFono
state changes directly.
On call, it may have to stop current audio playback then route audio:
- from telephony chipset to speaker if there is no HFP connection
- or from telephony chipset to bluetooth device if a HFP endpoint exists
I do not think this needs to pass through BlueZ.
> That being said, I'm not sure if the Telephony API is the proper way
> to address this.
Regards
Fred
--
Frederic Danis Open Source Technology Center
frederic.danis@intel.com Intel Corporation
next prev parent reply other threads:[~2012-07-06 13:07 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-28 16:15 [PATCH v11 00/12] Add org.bluez.Telephony interface Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 01/12] audio: Move telephony drivers to D-Bus interface Frédéric Danis
2012-07-05 14:21 ` Mikel Astiz
2012-07-06 13:07 ` Frederic Danis [this message]
2012-06-28 16:15 ` [PATCH v11 02/12] audio: Simplify org.bluez.Headset Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 03/12] audio: Remove dummy telephony driver Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 04/12] audio: Remove maemo5 " Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 05/12] audio: Remove maemo6 " Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 06/12] audio: Remove oFono " Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 07/12] audio: Move HFP/HSP AG servers to telephony.c Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 08/12] audio: Send transport path to telephony agent Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 09/12] audio: Move HFP HF server to telephony.c Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 10/12] audio: Replace headset and gateway by telephony Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 11/12] audio: Add DUN GW to org.bluez.Telephony Frédéric Danis
2012-06-28 16:15 ` [PATCH v11 12/12] audio: Add SAP " Frédéric Danis
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=4FF6E31E.70507@linux.intel.com \
--to=frederic.danis@linux.intel.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=mikel.astiz.oss@gmail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).