From: Chen Ganir <chen.ganir@ti.com>
To: Claudio Takahasi <claudio.takahasi@openbossa.org>
Cc: <linux-bluetooth@vger.kernel.org>
Subject: Re: [PATCH v2 0/9] Add GATT Client Battery Service
Date: Mon, 6 Aug 2012 08:52:57 +0300 [thread overview]
Message-ID: <501F5BB9.30406@ti.com> (raw)
In-Reply-To: <CAKT1EBdUCSLZm=_FodNeT1UuquYVQVx8oZTOJw9ei2ca8WCrqA@mail.gmail.com>
Claudio,
On 08/03/2012 03:57 PM, Claudio Takahasi wrote:
> Hi Chen Ganir:
>
> On Wed, Aug 1, 2012 at 3:40 AM, Chen Ganir <chen.ganir@ti.com> wrote:
>> On 07/25/2012 08:42 AM, Chen Ganir wrote:
>>>
>>> Add suupport for LE GATT Client Battery Service.
>>>
>>> This plugin adds battery list to the btd_device, exposes DBUS API to list
>>> the
>>> device batteries, and allows querying for battery information. In addition
>>> this
>>> patch allows getting notifications for battery level changes.
>>>
>>> Look at doc/device-api.txt and doc/battery-api.txt for more information.
>>>
>>> This is version 2 of this patch set, rebased on top of the latest sources
>>> and
>>> fixes some issues found during testing and in the ML comments.
>>>
>>> Chen Ganir (9):
>>> Battery: Add Battery Service GATT Client
>>> Battery: Add connection logic
>>> Battery: Discover Characteristic Descriptors
>>> Battery: Get Battery ID
>>> Battery: Add Battery list to btd_device
>>> Battery: Add Battery D-BUS API
>>> Battery: Read Battery level characteristic
>>> Battery: Add support for notifications
>>> Battery: Emit property changed on first read
>>>
>>> Makefile.am | 10 +-
>>> doc/battery-api.txt | 38 +++
>>> doc/device-api.txt | 5 +
>>> profiles/batterystate/batterystate.c | 518
>>> ++++++++++++++++++++++++++++++++++
>>> profiles/batterystate/batterystate.h | 24 ++
>>> profiles/batterystate/main.c | 67 +++++
>>> profiles/batterystate/manager.c | 93 ++++++
>>> profiles/batterystate/manager.h | 24 ++
>>> src/device.c | 66 +++++
>>> src/device.h | 3 +
>>> 10 files changed, 846 insertions(+), 2 deletions(-)
>>> create mode 100644 doc/battery-api.txt
>>> create mode 100644 profiles/batterystate/batterystate.c
>>> create mode 100644 profiles/batterystate/batterystate.h
>>> create mode 100644 profiles/batterystate/main.c
>>> create mode 100644 profiles/batterystate/manager.c
>>> create mode 100644 profiles/batterystate/manager.h
>>>
>>
>> Ping anyone ? Did anyone get to review this ?
>>
>> Thanks,
>> Chen Ganir
>
>
> not yet.
> We have an INTERNAL release in two weeks, next week we will send
> comments in the ML.
>
Thanks. I'll be waiting.
> BR,
> Claudio
>
Chen Ganir.
next prev parent reply other threads:[~2012-08-06 5:52 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-25 5:42 [PATCH v2 0/9] Add GATT Client Battery Service Chen Ganir
2012-07-25 5:42 ` [PATCH v2 1/9] Battery: Add Battery Service GATT Client Chen Ganir
2012-07-25 5:42 ` [PATCH v2 2/9] Battery: Add connection logic Chen Ganir
2012-08-07 16:29 ` Joao Paulo Rechi Vita
2012-08-08 5:50 ` Chen Ganir
2012-07-25 5:42 ` [PATCH v2 3/9] Battery: Discover Characteristic Descriptors Chen Ganir
2012-07-25 5:42 ` [PATCH v2 4/9] Battery: Get Battery ID Chen Ganir
2012-08-07 16:29 ` Joao Paulo Rechi Vita
2012-08-08 5:51 ` Chen Ganir
2012-07-25 5:42 ` [PATCH v2 5/9] Battery: Add Battery list to btd_device Chen Ganir
2012-08-07 16:29 ` Joao Paulo Rechi Vita
2012-08-08 5:56 ` Chen Ganir
2012-08-08 6:14 ` Chen Ganir
2012-07-25 5:42 ` [PATCH v2 6/9] Battery: Add Battery D-BUS API Chen Ganir
2012-07-25 5:42 ` [PATCH v2 7/9] Battery: Read Battery level characteristic Chen Ganir
2012-07-25 5:42 ` [PATCH v2 8/9] Battery: Add support for notifications Chen Ganir
2012-08-07 16:29 ` Joao Paulo Rechi Vita
2012-08-08 6:03 ` Chen Ganir
2012-07-25 5:42 ` [PATCH v2 9/9] Battery: Emit property changed on first read Chen Ganir
2012-08-07 16:29 ` Joao Paulo Rechi Vita
2012-08-08 6:04 ` Chen Ganir
2012-08-01 6:40 ` [PATCH v2 0/9] Add GATT Client Battery Service Chen Ganir
2012-08-03 12:57 ` Claudio Takahasi
2012-08-06 5:52 ` Chen Ganir [this message]
2012-08-07 16:29 ` Joao Paulo Rechi Vita
2012-08-08 5:45 ` Chen Ganir
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=501F5BB9.30406@ti.com \
--to=chen.ganir@ti.com \
--cc=claudio.takahasi@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).