Return-Path: From: Gowtham Anandha Babu To: 'Luiz Augusto von Dentz' Cc: linux-bluetooth@vger.kernel.org References: <1417423638-29222-1-git-send-email-luiz.dentz@gmail.com> <1417423638-29222-8-git-send-email-luiz.dentz@gmail.com> <003a01d00d60$d0424590$70c6d0b0$@samsung.com> In-reply-to: Subject: RE: [PATCH BlueZ 8/8] obexd/client: Add supported_features support Date: Mon, 01 Dec 2014 19:01:43 +0530 Message-id: <003b01d00d6b$3723cfe0$a56b6fa0$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=utf-8 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Luiz, > -----Original Message----- > From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth- > owner@vger.kernel.org] On Behalf Of Luiz Augusto von Dentz > Sent: Monday, December 01, 2014 6:22 PM > To: Gowtham Anandha Babu > Cc: linux-bluetooth@vger.kernel.org > Subject: Re: [PATCH BlueZ 8/8] obexd/client: Add supported_features > support > > Hi Gowtham, > > On Mon, Dec 1, 2014 at 2:17 PM, Gowtham Anandha Babu > wrote: > > Hi Luiz, > > > >> -----Original Message----- > >> From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth- > >> owner@vger.kernel.org] On Behalf Of Luiz Augusto von Dentz > >> Sent: Monday, December 01, 2014 2:17 PM > >> To: linux-bluetooth@vger.kernel.org > >> Subject: [PATCH BlueZ 8/8] obexd/client: Add supported_features > >> support > >> > >> From: Luiz Augusto von Dentz > >> > >> This adds supported_features support to obc_driver so driver can > >> provide this information when connecting. > >> > >> This is required by PBAP 1.2 (page 48): > >> > >> 'Mandatory if the PSE advertises a PbapSupportedFeatures attribute in > >> its SDP record, else excluded.' > >> --- > >> obexd/client/driver.h | 1 + > >> obexd/client/pbap.c | 36 > ++++++++++++++++++++++++++++++++++++ > >> obexd/client/session.c | 25 ++++++++++++++++++++++++- > >> 3 files changed, 61 insertions(+), 1 deletion(-) > >> > >> diff --git a/obexd/client/driver.h b/obexd/client/driver.h index > >> f1c0646..0112219 100644 > >> --- a/obexd/client/driver.h > >> +++ b/obexd/client/driver.h > >> @@ -26,6 +26,7 @@ struct obc_driver { > >> const char *uuid; > >> void *target; > >> gsize target_len; > >> + void *(*supported_features) (struct obc_session *session); > >> int (*probe) (struct obc_session *session); > >> void (*remove) (struct obc_session *session); }; diff --git > >> a/obexd/client/pbap.c b/obexd/client/pbap.c index 812a7fb..57632b4 > >> 100644 > >> --- a/obexd/client/pbap.c > >> +++ b/obexd/client/pbap.c > >> @@ -76,6 +76,7 @@ > >> #define PRIMARY_COUNTER_TAG 0X0A > >> #define SECONDARY_COUNTER_TAG 0X0B > >> #define DATABASEID_TAG 0X0D > >> +#define SUPPORTED_FEATURES_TAG 0x10 > >> > >> #define DOWNLOAD_FEATURE 0x00000001 > >> #define BROWSE_FEATURE 0x00000002 > >> @@ -1230,6 +1231,40 @@ static void parse_service_record(struct > >> pbap_data > >> *pbap) > >> > >> } > >> > >> +static void *pbap_supported_features(struct obc_session *session) { > >> + const void *data; > >> + uint16_t version; > >> + > >> + /* Version */ > >> + data = obc_session_get_attribute(session, > >> SDP_ATTR_PFILE_DESC_LIST); > >> + if (!data) > >> + return NULL; > >> + > >> + version = GPOINTER_TO_UINT(data); > >> + > >> + if (version < 0x0102) > >> + return NULL; > >> + > >> + /* Supported Feature Bits */ > >> + data = obc_session_get_attribute(session, > >> + > >> SDP_ATTR_PBAP_SUPPORTED_FEATURES); > >> + if (!data) > >> + return NULL; > >> + > >> + return g_obex_apparam_set_uint32(NULL, > >> SUPPORTED_FEATURES_TAG, > >> + DOWNLOAD_FEATURE | > >> + BROWSE_FEATURE | > >> + DATABASEID_FEATURE | > >> + FOLDER_VERSION_FEATURE > >> | > >> + VCARD_SELECTING_FEATURE > >> | > >> + ENHANCED_CALLS_FEATURE > >> | > >> + UCI_FEATURE | > >> + UID_FEATURE | > >> + REFERENCING_FEATURE | > >> + DEFAULT_IMAGE_FEATURE); > >> +} > >> + > >> static int pbap_probe(struct obc_session *session) { > >> struct pbap_data *pbap; > >> @@ -1274,6 +1309,7 @@ static struct obc_driver pbap = { > >> .uuid = PBAP_UUID, > >> .target = OBEX_PBAP_UUID, > >> .target_len = OBEX_PBAP_UUID_LEN, > >> + .supported_features = pbap_supported_features, > >> .probe = pbap_probe, > >> .remove = pbap_remove > >> }; > >> diff --git a/obexd/client/session.c b/obexd/client/session.c index > >> 9bba6c6..d2ae4fd 100644 > >> --- a/obexd/client/session.c > >> +++ b/obexd/client/session.c > >> @@ -345,6 +345,7 @@ static void transport_func(GIOChannel *io, GError > >> *err, gpointer user_data) > >> struct obc_driver *driver = session->driver; > >> struct obc_transport *transport = session->transport; > >> GObex *obex; > >> + GObexApparam *apparam; > >> GObexTransportType type; > >> int tx_mtu = -1; > >> int rx_mtu = -1; > >> @@ -370,7 +371,29 @@ static void transport_func(GIOChannel *io, > >> GError *err, gpointer user_data) > >> > >> g_io_channel_set_close_on_unref(io, TRUE); > >> > >> - if (driver->target != NULL) > >> + apparam = NULL; > >> + > >> + if (driver->supported_features) > >> + apparam = driver->supported_features(session); > >> + > >> + if (apparam) { > >> + uint8_t buf[1024]; > >> + ssize_t len; > >> + > >> + len = g_obex_apparam_encode(apparam, buf, sizeof(buf)); > >> + if (driver->target) > >> + g_obex_connect(obex, connect_cb, callback, &err, > >> + G_OBEX_HDR_TARGET, > >> + driver->target, driver->target_len, > >> + G_OBEX_HDR_APPARAM, > >> + buf, len, > >> + G_OBEX_HDR_INVALID); > >> + else > >> + g_obex_connect(obex, connect_cb, callback, &err, > >> + G_OBEX_HDR_APPARAM, buf, len, > >> + G_OBEX_HDR_INVALID); > >> + g_obex_apparam_free(apparam); > >> + } else if (driver->target) > >> g_obex_connect(obex, connect_cb, callback, &err, > >> G_OBEX_HDR_TARGET, driver->target, driver- > >> >target_len, > >> G_OBEX_HDR_INVALID); > >> -- > >> 1.9.3 > >> > >> -- > >> 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 > > > > > > > > I applied this patch locally and tested with PTS. > > The PTS test case: TP/SSM/BV-09-C [PCE Shares PbapSupportedFeature > > bits] is failing. > > According to PBAP V1.2 Spec - Section 6.4 - OBEX Connect request > > format, the PbapSupportedFeatures are shared during obex connect(after > > applying the this patch). > > Still the above test case is failing. > > Am I mapping anything wrong? > > Have you applied the full set? Maybe the PCE record needs updating as well, > but normally it is not mandatory. > > > -- > Luiz Augusto von Dentz > -- > 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 Yes, I applied the complete set and tested it. Still that TC is failing. Just want to know how it is failing, because I tried 1) Applying this patch set - not passing 2) Added below attribute in the src/profile PCE record at the end as below - not passing May be it’s a PTS Issue. Anyway it's not mandatory. Regards, Gowtham Anandha Babu