Return-Path: From: Gowtham Anandha Babu To: 'Arman Uguray' Cc: 'BlueZ development' , 'Bharat Bhusan Panda' , cpgs@samsung.com References: <1424336266-17437-1-git-send-email-gowtham.ab@samsung.com> In-reply-to: Subject: RE: [PATCH] src/gatt-dbus: Add GATT Extended props bit fields Date: Thu, 19 Feb 2015 15:12:53 +0530 Message-id: <000001d04c28$728ab480$57a01d80$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=utf-8 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Arman, > -----Original Message----- > From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth- > owner@vger.kernel.org] On Behalf Of Arman Uguray > Sent: Thursday, February 19, 2015 2:37 PM > To: Gowtham Anandha Babu > Cc: BlueZ development; Bharat Bhusan Panda; cpgs@samsung.com > Subject: Re: [PATCH] src/gatt-dbus: Add GATT Extended props bit fields > > Hi Gowtham, > > > On Thu, Feb 19, 2015 at 12:57 AM, Gowtham Anandha Babu > wrote: > > Removed the TODO and added the GATT Characteristic Extended > > Properties Bitfield values as per core spec 4.2 - page 2227. > > I don't know if there is any point of developing attrib/gatt any further, > especially since we can call it deprecated now. > Yes, you are right. Even I don’t see any value in updating attrib/gatt, because it's going to be deprecated as you said. Still my concern is about src/gatt-dbus. > Also, I'm in the process of heavily rewriting src/gatt-dbus and I'm planning to > upload some patches next week, so would it be OK to hold off on this? > Yes, fine. I will rebase this on top of your patches and send it again later. Regards, Gowtham Anandha Babu > > --- > > attrib/gatt.h | 4 ++++ > > src/gatt-dbus.c | 6 ++++-- > > 2 files changed, 8 insertions(+), 2 deletions(-) > > > > diff --git a/attrib/gatt.h b/attrib/gatt.h index f6db10f..c26a286 > > 100644 > > --- a/attrib/gatt.h > > +++ b/attrib/gatt.h > > @@ -42,6 +42,10 @@ > > #define GATT_CHR_PROP_AUTH 0x40 > > #define GATT_CHR_PROP_EXT_PROP 0x80 > > > > +/* GATT Characteristic Extended Properties Bitfield values */ > > +#define GATT_CHRC_EXT_PROP_RELIABLE_WRITE 0x01 > > +#define GATT_CHRC_EXT_PROP_WRITABLE_AUX 0x02 > > + > > /* Client Characteristic Configuration bit field */ > > #define GATT_CLIENT_CHARAC_CFG_NOTIF_BIT 0x0001 > > #define GATT_CLIENT_CHARAC_CFG_IND_BIT 0x0002 > > diff --git a/src/gatt-dbus.c b/src/gatt-dbus.c index c22e8af..a5a9faf > > 100644 > > --- a/src/gatt-dbus.c > > +++ b/src/gatt-dbus.c > > @@ -154,11 +154,13 @@ static uint8_t flags_string2int(const char *proper) > > value = GATT_CHR_PROP_INDICATE; > > else if (!strcmp("authenticated-signed-writes", proper)) > > value = GATT_CHR_PROP_AUTH; > > + else if (!strcmp("reliable-write", proper)) > > + value = GATT_CHRC_EXT_PROP_RELIABLE_WRITE; > > + else if (!strcmp("writable-auxiliaries", proper)) > > + value = GATT_CHRC_EXT_PROP_WRITABLE_AUX ; > > else > > value = 0; > > > > - /* TODO: Extended properties. Ref core spec 4.1 page 2185 */ > > - > > return value; > > } > > > > -- > > 1.9.1 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe > > linux-bluetooth" in the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > Thanks, > Arman > -- > To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in > the body of a message to majordomo@vger.kernel.org More majordomo > info at http://vger.kernel.org/majordomo-info.html