From: Marcel Holtmann <marcel@holtmann.org>
To: Andrzej Kaczmarek <andrzej.kaczmarek@tieto.com>
Cc: linux-bluetooth@vger.kernel.org,
par-gunnar.p.hjalmdahl@stericsson.com,
ulrik.lauren@stericsson.com
Subject: Re: [PATCH 0/4] Add support for custom (e)SCO parameters
Date: Wed, 15 Jun 2011 04:08:05 -0700 [thread overview]
Message-ID: <1308136088.2518.3.camel@aeonflux> (raw)
In-Reply-To: <1308059827-15095-1-git-send-email-andrzej.kaczmarek@tieto.com>
Hi Andrzej,
> Following patches add possibility to specify custom parameters for
> outgoing (e)SCO link. This is implemented as SCO socket options.
>
> This change allows to use reference parameters as defined in HFP 1.5
> specification as well as support for future enchancements as wideband
> speech as defined in HFP 1.6 draft (uses different air coding format).
>
> Additional socket option (patch 3) allows to disable automatic retry
> in case of failure which fallbacks from eSCO to SCO connection in
> current implementation. Such fallback should not happen when using
> wideband speech which mandates using of eSCO transport.
I am not a big fan of just stupidly exposing all possible HCI value to
userspace and let userspace set them. Especially since in the end there
are no real variation here anyway. There are just a 2-3 use cases for
your SCO socket.
I would rather have simple mode to hand to the kernel and let it do the
right thing (including negotiation and downgrading to SCO) than having
userspace opening sockets over sockets again.
So what are the actual modes for HFP that need to be supported and what
is the range of possible SCO and eSCO options?
Regards
Marcel
next prev parent reply other threads:[~2011-06-15 11:08 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-14 13:57 [PATCH 0/4] Add support for custom (e)SCO parameters Andrzej Kaczmarek
2011-06-14 13:57 ` [PATCH 1/4] Bluetooth: Change bit logic in pkt_type for SCO links Andrzej Kaczmarek
2011-06-14 13:57 ` [PATCH 2/4] Bluetooth: Add BT_SCO_PARAMETERS SCO socket option Andrzej Kaczmarek
2011-06-14 13:57 ` [PATCH 3/4] Bluetooth: Add BT_NO_AUTORETRY " Andrzej Kaczmarek
2011-06-14 13:57 ` [PATCH 4/4] Bluetooth: Update mapping for eSCO related error codes Andrzej Kaczmarek
2011-06-15 11:08 ` Marcel Holtmann [this message]
2011-06-15 13:53 ` [PATCH 0/4] Add support for custom (e)SCO parameters Andrzej Kaczmarek
2011-06-15 14:25 ` Marcel Holtmann
2011-06-16 11:33 ` Andrzej Kaczmarek
2011-06-16 14:00 ` Luiz Augusto von Dentz
2011-06-16 14:41 ` Marcel Holtmann
2011-06-29 13:41 ` Andrzej Kaczmarek
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=1308136088.2518.3.camel@aeonflux \
--to=marcel@holtmann.org \
--cc=andrzej.kaczmarek@tieto.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=par-gunnar.p.hjalmdahl@stericsson.com \
--cc=ulrik.lauren@stericsson.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).