From: Johan Hedberg <johan.hedberg@gmail.com>
To: Denis Kenzior <denkenz@gmail.com>
Cc: linux-bluetooth <linux-bluetooth@vger.kernel.org>, ofono@ofono.org
Subject: Re: [RFC] HFP support into oFono and BlueZ
Date: Tue, 19 Jan 2010 10:02:23 +0200 [thread overview]
Message-ID: <20100119080223.GA20493@jh-x301> (raw)
In-Reply-To: <201001171637.22263.denkenz@gmail.com>
Hi Denis,
On Sun, Jan 17, 2010, Denis Kenzior wrote:
> > having it as a MediaEndpoint I guess it is probably a good idea to
> > have a similar design as we will have in endpoints, so the
> > registration actually happen in adapter path rather than device path,
> > which probably should avoid the having to resolve the device path
> > while probing if we just send the device path as a parameter of
> > NewConnection so we can do while in DEFER_SETUP stage so we don't risk
> > missing or getting timeouts while transferring the the fd to ofono.
>
> Personally I like this to be a per-device agent rather than per-adapter as it
> will make agent code inside oFono a bit easier to write. Marcel / Johan, care
> to weigh in?
My initial gut feeling is that this should be adapter based, though that
might be partly fueled by the fact that that's also how the current
pairing and authorization agent is designed. However, I do remember us
discussing this earlier and the outcome back then was (at least based on
your API suggestion) to have a per-device agent. I can't remember the
exact supporting arguments for this, but at least the API does get
simpler since then we don't need a new interface for both adapter and
device objects and the parameter lists of the HandsfreeAgent interface
methods become shorter since we don't need to have the remote device
object path there.
Johan
_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono
next prev parent reply other threads:[~2010-01-19 8:02 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-11 17:08 [RFC] HFP support into oFono and BlueZ Gustavo F. Padovan
2010-01-11 19:05 ` Gustavo F. Padovan
2010-01-13 23:39 ` Gustavo F. Padovan
2010-01-18 11:38 ` Luiz Augusto von Dentz
2010-01-17 22:37 ` Denis Kenzior
2010-01-19 8:02 ` Johan Hedberg [this message]
2010-01-19 9:30 ` Luiz Augusto von Dentz
2010-01-19 10:33 ` Johan Hedberg
2010-01-19 12:26 ` Luiz Augusto von Dentz
2010-01-20 19:58 ` Gustavo F. Padovan
2010-01-21 6:28 ` Zhang, Zhenhua
2010-01-21 7:54 ` Zhao Forrest
2010-01-21 2:56 ` Denis Kenzior
2012-08-21 22:46 ` Lucas De Marchi
2010-01-21 19:22 ` Johan Hedberg
2010-01-21 19:27 ` Johan Hedberg
2010-01-27 19:12 ` HFP support into BlueZ and oFono Gustavo F. Padovan
2010-01-27 19:12 ` [PATCH 1/2] clean up audio/gateway.c Gustavo F. Padovan
2010-01-27 19:12 ` [PATCH 2/2] Implement HandsfreeGateway Interface Gustavo F. Padovan
2010-01-27 19:12 ` [PATCH] Add HFP support through BlueZ Gustavo F. Padovan
2010-01-27 20:17 ` Marcel Holtmann
2010-02-01 3:15 ` [PATCH 1/2] clean up audio/gateway.c Zhenhua Zhang
2010-01-27 19:17 ` HFP support into BlueZ and oFono Gustavo F. Padovan
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=20100119080223.GA20493@jh-x301 \
--to=johan.hedberg@gmail.com \
--cc=denkenz@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
--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 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).