From: Gustavo Padovan <padovan@profusion.mobi>
To: Santiago Carot-Nemesio <sancane@gmail.com>
Cc: linux-bluetooth@vger.kernel.org
Subject: Re: [PATCH 07/19] Read temperature type characteristic
Date: Tue, 20 Sep 2011 17:32:58 -0300 [thread overview]
Message-ID: <20110920203258.GE11558@joana> (raw)
In-Reply-To: <1316540841-14713-8-git-send-email-sancane@gmail.com>
Hi Santiago,
* Santiago Carot-Nemesio <sancane@gmail.com> [2011-09-20 19:47:19 +0200]:
> ---
> thermometer/thermometer.c | 54 ++++++++++++++++++++++++++++++++++++++++++++-
> 1 files changed, 53 insertions(+), 1 deletions(-)
>
> diff --git a/thermometer/thermometer.c b/thermometer/thermometer.c
> index f0b977e..2dcc4ba 100644
> --- a/thermometer/thermometer.c
> +++ b/thermometer/thermometer.c
> @@ -39,6 +39,14 @@
> #define INTERMEDIATE_TEMPERATURE_UUID "00002a1e-0000-1000-8000-00805f9b34fb"
> #define MEASUREMENT_INTERVAL_UUID "00002a21-0000-1000-8000-00805f9b34fb"
>
> +struct properties {
> + gboolean intermediate;
> + guint8 *type;
> + guint16 *interval;
> + guint16 *max;
> + guint16 *min;
> +};
> +
> struct thermometer {
> DBusConnection *conn; /* The connection to the bus */
> struct btd_device *dev; /* Device reference */
> @@ -47,6 +55,7 @@ struct thermometer {
> gint attioid; /* Att watcher id */
> gint attindid; /* Att incications id */
> GSList *chars; /* Characteristics */
> + struct properties properties; /* Thermometer's properties */
fold type, interval, max and min here seems a better idea.
> };
>
> struct characteristic {
> @@ -63,6 +72,24 @@ struct descriptor {
>
> static GSList *thermometers = NULL;
>
> +static gchar* temptype2str(uint8_t value)
> +{
> + switch (value) {
> + case 1: return "Armpit";
> + case 2: return "Body";
> + case 3: return "Ear";
> + case 4: return "Finger";
> + case 5: return "Intestines";
> + case 6: return "Mouth";
> + case 7: return "Rectum";
> + case 8: return "Toe";
> + case 9: return "Tympanum";
> + default:
> + error("Temperature type %d reserved for future use", value);
> + return NULL;
> + };
> +}
> +
> static void destroy_char(gpointer user_data)
> {
> struct characteristic *c = user_data;
> @@ -87,6 +114,11 @@ static void destroy_thermometer(gpointer user_data)
> if (t->chars)
> g_slist_free_full(t->chars, destroy_char);
>
> + g_free(t->properties.type);
> + g_free(t->properties.interval);
> + g_free(t->properties.max);
> + g_free(t->properties.min);
> +
> dbus_connection_unref(t->conn);
> btd_device_unref(t->dev);
> g_free(t->svc_range);
> @@ -113,7 +145,27 @@ static void discover_desc_cb(guint8 status, const guint8 *pdu, guint16 len,
> static void read_temp_type_cb(guint8 status, const guint8 *pdu, guint16 len,
> gpointer user_data)
> {
> - /* TODO */
> + struct characteristic *ch = user_data;
> + struct thermometer *t = ch->t;
> + uint8_t value;
> + int vlen;
> +
> + if (status != 0) {
> + DBG("Temperature Type value read failed: %s",
> + att_ecode2str(status));
> + return;
> + }
> +
> + if (!dec_read_resp(pdu, len, &value, &vlen)) {
> + DBG("Protocol error");
> + return;
> + }
> +
> + DBG("TEMPERATURE TYPE: %s", temptype2str(value));
> + if (!t->properties.type)
> + t->properties.type = g_new0(guint8, 1);
Is there a good reason why this is allocated memory? I'm no seeing the point.
Gustavo
next prev parent reply other threads:[~2011-09-20 20:32 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-20 17:47 Health Thermomete Profile support Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 01/19] Get thermometer service range to load the driver Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 02/19] Register Health Thermometer Interface Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 03/19] Unregister " Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 04/19] Add functions to manage attio callbacks Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 05/19] Add handler function to manage GATT indications Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 06/19] Get all characteristics in thermometer service Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 07/19] Read temperature type characteristic Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 08/19] Read measurement interval characteristic Santiago Carot-Nemesio
2011-09-20 17:47 ` [PATCH 09/19] Set Intermediate property if intermediate temp. characteristic is supported Santiago Carot-Nemesio
2011-09-20 20:28 ` Santiago Carot
2011-09-20 20:32 ` Gustavo Padovan [this message]
2011-09-20 20:40 ` [PATCH 07/19] Read temperature type characteristic Santiago Carot
2011-09-20 20:46 ` Gustavo Padovan
2011-09-20 20:53 ` Santiago Carot
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=20110920203258.GE11558@joana \
--to=padovan@profusion.mobi \
--cc=linux-bluetooth@vger.kernel.org \
--cc=sancane@gmail.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).