Hi Johan, > -----Original Message----- > From: Johan Hedberg [mailto:johan.hedberg@xxxxxxxxx] > Sent: Wednesday, May 30, 2012 1:18 PM > To: Joohi RASTOGI > Cc: User Name > Subject: Re: [PATCH] Sets the information attribute for the service > > Hi Joohi, > > On Tue, May 29, 2012, joohi.rastogi@xxxxxxxxxxxxxx wrote: > > From: Joohi Rastogi <joohi.rastogi@xxxxxxxxxxxxxx> > > > > This patch adds the service name for > > Device Identification Profile > > --- > > src/sdpd-service.c | 2 ++ > > 1 files changed, 2 insertions(+), 0 deletions(-) > > > > diff --git a/src/sdpd-service.c b/src/sdpd-service.c > > index 39e05ab..1fa82a6 100644 > > --- a/src/sdpd-service.c > > +++ b/src/sdpd-service.c > > @@ -228,6 +228,8 @@ void register_device_id(void) > > source_data = sdp_data_alloc(SDP_UINT16, &main_opts.did_source); > > sdp_attr_add(record, 0x0205, source_data); > > > > + sdp_set_info_attr(record, "Device Identification", 0, 0); > > + > > update_db_timestamp(); > > 0 is an integer, NULL is a pointer. The function call you're adding > doesn't take any integer arguments so you should be passing NULL and not > 0. > I will do the required change and re-push the patch. > That said, I can't find any mention of the service name attribute in the > Device ID specification (version 1.3). In fact the table (8.1 on page > 17) that lists all mandatory and optional attributes doesn't even have > this attribute mentioned as an optional one. So why do you need to have > it added? Is there some client that breaks if it's not present? > I added the "Service Name" because while doing sdptool browse, every profile displays its Service Name along with other attributes like Service Class ID List, Protocol Descriptor List and Service RecHandle. But, Device ID Profile does not display its Service Name. So, I had added the Service Name attribute as well to the profile. For example: For AVRCP TG the sdptool browse response is: Service Name: AVRCP TG Service RecHandle: 0x10005 Service Class ID List: "AV Remote Target" (0x110c) Protocol Descriptor List: "L2CAP" (0x0100) PSM: 23 "AVCTP" (0x0017) uint16: 0x103 Profile Descriptor List: "AV Remote" (0x110e) Version: 0x0103 And for Device ID Profile sdptool browse response is: Service RecHandle: 0x10000 Service Class ID List: "PnP Information" (0x1200) Profile Descriptor List: "PnP Information" (0x1200) Version: 0x0103 Please let me know your view on this. > Johan Thanks, Joohi Rastogi -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
