linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: Anderson Lizardo <anderson.lizardo@openbossa.org>
Cc: anderson.briglia@openbossa.org, linux-bluetooth@vger.kernel.org,
	Bruna Moreira <bruna.moreira@openbossa.org>
Subject: Re: [RFC 7/7] Update Management API documentation
Date: Wed, 15 Jun 2011 19:29:13 -0700	[thread overview]
Message-ID: <1308191356.2196.9.camel@aeonflux> (raw)
In-Reply-To: <BANLkTikuyizJeF9xhAo22dv8LJ+JxeTskg@mail.gmail.com>

Hi Anderson,

> > so why are these two separate commands. Just reading them in one kernel
> > call might be a lot better. Including everything else you need to know
> > about the connection. Also TX power can be local and remote TX power if
> > I remember this correctly.
> 
> The issue I see with your proposal is that for Proximity (currently
> the only user of this information) we read TX power from server
> (reporter) using GATT, while RSSI is read on client (monitor) side. By
> reading both, we would need to issue two separate HCI commands (and
> wait for both responses), but will always use only one value,
> depending on local BlueZ role.
> 
> Also note that on latest Proximity profile drafts, it is mentioned
> that the TX power on reporter will never change during a connection,
> and thus it should be read only once when the connection is
> established. RSSI (which is read "locally" on monitor side), on the
> other hand, should be read periodically for path loss calculation.

if TX power is only read once than the kernel should just do it once and
be done with it.

And for RSSI, it would be better if the kernel read this periodically
based on current sniff mode etc. Userspace can not trigger this with
proper timing anyway. And it will potentially at some point start
blocking the controller. Only the kernel really knows when it is
acceptable to read the RSSI value.

Regards

Marcel



  reply	other threads:[~2011-06-16  2:29 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1308171784-24442-1-git-send-email-y>
2011-06-15 21:02 ` [RFC 1/7] Add current tx power read on hciops anderson.briglia
2011-06-15 21:02 ` [RFC 2/7] Add read RSSI " anderson.briglia
2011-06-15 21:03 ` [RFC 3/7] Add RSSI read callbacks anderson.briglia
2011-06-15 21:03 ` [RFC 4/7] Add transmission power callbacks anderson.briglia
2011-06-15 21:03 ` [RFC 5/7] Implement mgmt_read_rssi command anderson.briglia
2011-06-15 21:03 ` [RFC 6/7] Implement mgmt_read_tx_power command anderson.briglia
2011-06-15 21:03 ` [RFC 7/7] Update Management API documentation anderson.briglia
2011-06-15 22:10   ` Marcel Holtmann
2011-06-15 23:47     ` Anderson Lizardo
2011-06-16  2:29       ` Marcel Holtmann [this message]
2011-06-16  6:07         ` Ganir, Chen
2011-06-16  7:54           ` Luiz Augusto von Dentz
2011-06-16 12:35             ` Claudio Takahasi
2011-06-16 14:47               ` Marcel Holtmann
2011-06-16 15:12                 ` Claudio Takahasi
2011-06-16 16:08                   ` Marcel Holtmann
2011-06-16 20:45                   ` Gustavo F. Padovan
2011-06-20 15:47                     ` Anderson Briglia
2011-06-20 16:47                       ` Marcel Holtmann
2011-06-16 13:49             ` Anderson Lizardo
2011-06-16 14:08               ` Johan Hedberg
2011-06-16 14:17                 ` Anderson Lizardo
2011-06-16  7:15         ` GATT and GATT based Profiles architecture - Query Ajay Pillai
2011-06-16 14:04           ` Anderson Lizardo
2011-06-17  9:30             ` Ajay Pillai
2011-06-17 12:10               ` Anderson Lizardo
2011-06-17 12:22                 ` Santiago Carot
2011-06-17 12:39                   ` Anderson Lizardo
2011-06-17 12:53                     ` Santiago Carot
2011-06-20  8:42                 ` Ajay Pillai
2011-06-16  6:03       ` [RFC 7/7] Update Management API documentation Ganir, Chen
2011-06-16 13:44         ` Anderson Lizardo
2011-06-16 14:19           ` Ganir, Chen

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=1308191356.2196.9.camel@aeonflux \
    --to=marcel@holtmann.org \
    --cc=anderson.briglia@openbossa.org \
    --cc=anderson.lizardo@openbossa.org \
    --cc=bruna.moreira@openbossa.org \
    --cc=linux-bluetooth@vger.kernel.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).