linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arik Nemtsov <arik@wizery.com>
To: Anderson Lizardo <anderson.lizardo@openbossa.org>
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH RFC BlueZ 0/5] Time Profile (server) improvements
Date: Thu, 2 Feb 2012 15:32:01 +0200	[thread overview]
Message-ID: <CA+XVXfeyCUMP0V02N7VQ-KOchTOKptTCYa1kbeue_+OO-ybB9A@mail.gmail.com> (raw)
In-Reply-To: <CAJdJm_P2nKg9EWCyX4T1qG6zcOtggb-GqueaF4worq6dfbw2_Q@mail.gmail.com>

On Thu, Feb 2, 2012 at 15:07, Anderson Lizardo
<anderson.lizardo@openbossa.org> wrote:
> Hi Arik,
>
> On Thu, Feb 2, 2012 at 8:19 AM, Arik Nemtsov <arik@wizery.com> wrote:
>> Does this mean the server-application operating over D-Bus would be
>> able to register read/write callbacks on specific attributes?
>
> Actually the API will be more high level, and hopefully Profile
> agnostic, so it can be reused between profiles.

Maybe you have some preliminary suggested documentation? It would be
interesting to take a look

>
>> It would be nice to get a sense of the API soon. We're planning on
>> implementing several GATT servers. Namely - ANS, PASS, IAS, LLS and
>> TPS. This can obviously effect the design.
>
> It would be interesting for us to cooperate on this then. IAS/LLS/TPS
> are implemented on BlueZ upstream already (check proximity/* files).

It would indeed be nice to cooperate :)
As for IAS/LLS/TPS - I think maybe a rewrite using the higher-level
gatt_service_add() API is in order.

For IAS/LLS the notification to an upper-level app is missing
(something the new D-Bus API will probably address). For TPS,
returning the RSSI is missing (in a read_cb).

>
> ANS and PASS have skeleton code and initial implementation (which I
> will send soon to the list once I clean it up) on my development tree
> (warning: it is currently messy and outdated, and needs cleanup):
>
> git://gitorious.org/~lizardo/bluez/lizardo-bluez.git (branch
> for-upstream-phone-alert)

It seems I can't get to gitorious right now (seems to be some planned
maintenance). I'll definitely take a look once I can.

>
> Our current design is to have both ANS and PASS grouped into a
> altert/* directory.

Sounds good.

> I plan to send initial "dummy" implementations of NDCS and Local Time
> Information as well, but if you already have them implemented, feel
> free to send at least some RFC here.
>
> But you can extend with further functionality after we have the
> initial services upstream.

NDCS adds very little code. My main concern here is updating the
DST-change time periodically from the server-app.
Seems to me it's better to wait for your proposed skeleton implementations.

Regards,
Arik

  reply	other threads:[~2012-02-02 13:32 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-02 18:34 [PATCH RFC BlueZ 0/5] Time Profile (server) improvements Anderson Lizardo
2011-12-02 18:34 ` [PATCH RFC BlueZ 1/5] Time Profile: add Reference Time Update Service Anderson Lizardo
2011-12-02 18:34 ` [PATCH RFC BlueZ 2/5] Time Profile: implement generic "time provider" interface Anderson Lizardo
2011-12-02 18:34 ` [PATCH RFC BlueZ 3/5] Time Profile: Add "timed" time provider Anderson Lizardo
2011-12-02 18:34 ` [PATCH RFC BlueZ 4/5] Add support for sending notifications for current time Anderson Lizardo
2011-12-02 18:34 ` [PATCH RFC BlueZ 5/5] Add testing API to dummy Time Server provider Anderson Lizardo
2012-02-01 22:48 ` [PATCH RFC BlueZ 0/5] Time Profile (server) improvements Arik Nemtsov
2012-02-01 23:12   ` Anderson Lizardo
2012-02-02 12:19     ` Arik Nemtsov
2012-02-02 13:07       ` Anderson Lizardo
2012-02-02 13:32         ` Arik Nemtsov [this message]
2012-02-02 14:06           ` Anderson Lizardo
2012-05-16 11:40     ` Arun K. Singh
2012-05-17 14:50       ` Anderson Lizardo

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=CA+XVXfeyCUMP0V02N7VQ-KOchTOKptTCYa1kbeue_+OO-ybB9A@mail.gmail.com \
    --to=arik@wizery.com \
    --cc=anderson.lizardo@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).