* [PATCH] Sets the information attribute for the service
@ 2012-05-29 6:46 joohi.rastogi
2012-05-30 7:48 ` Johan Hedberg
0 siblings, 1 reply; 5+ messages in thread
From: joohi.rastogi @ 2012-05-29 6:46 UTC (permalink / raw)
To: User Name; +Cc: User Name2
From: Joohi Rastogi <joohi.rastogi@stericsson.com>
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();
}
--
1.7.5.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] Sets the information attribute for the service
2012-05-29 6:46 [PATCH] Sets the information attribute for the service joohi.rastogi
@ 2012-05-30 7:48 ` Johan Hedberg
2012-06-01 6:01 ` Joohi RASTOGI
2012-06-12 7:31 ` Joohi RASTOGI
0 siblings, 2 replies; 5+ messages in thread
From: Johan Hedberg @ 2012-05-30 7:48 UTC (permalink / raw)
To: joohi.rastogi; +Cc: User Name
Hi Joohi,
On Tue, May 29, 2012, joohi.rastogi@stericsson.com wrote:
> From: Joohi Rastogi <joohi.rastogi@stericsson.com>
>
> 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.
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?
Johan
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] Sets the information attribute for the service
2012-05-30 7:48 ` Johan Hedberg
@ 2012-06-01 6:01 ` Joohi RASTOGI
2012-06-12 7:31 ` Joohi RASTOGI
1 sibling, 0 replies; 5+ messages in thread
From: Joohi RASTOGI @ 2012-06-01 6:01 UTC (permalink / raw)
To: Johan Hedberg; +Cc: User Name
Hi Johan,
> -----Original Message-----
> From: Johan Hedberg [mailto:johan.hedberg@gmail.com]
> 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@stericsson.com wrote:
> > From: Joohi Rastogi <joohi.rastogi@stericsson.com>
> >
> > 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
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] Sets the information attribute for the service
2012-05-30 7:48 ` Johan Hedberg
2012-06-01 6:01 ` Joohi RASTOGI
@ 2012-06-12 7:31 ` Joohi RASTOGI
1 sibling, 0 replies; 5+ messages in thread
From: Joohi RASTOGI @ 2012-06-12 7:31 UTC (permalink / raw)
To: Johan Hedberg; +Cc: User Name
Hi Johan,
> -----Original Message-----
> From: Joohi RASTOGI
> Sent: Friday, June 01, 2012 11:32 AM
> To: 'Johan Hedberg'
> Cc: User Name
> Subject: RE: [PATCH] Sets the information attribute for the service
>
> Hi Johan,
>
> > -----Original Message-----
> > From: Johan Hedberg [mailto:johan.hedberg@gmail.com]
> > 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@stericsson.com wrote:
> > > From: Joohi Rastogi <joohi.rastogi@stericsson.com>
> > >
> > > 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.
It is not mandatory nor optional but other profiles have this information, so it is good to have for Device ID profile as well. For AVRCP also Service Name is not mandatory, but sdptool browse has this information. So, we have added it for Device ID and it is not breaking any other client.
> 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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] Sets the information attribute for the service
@ 2012-05-21 9:30 joohi.rastogi
0 siblings, 0 replies; 5+ messages in thread
From: joohi.rastogi @ 2012-05-21 9:30 UTC (permalink / raw)
To: User Name; +Cc: User Name2
From: Joohi Rastogi <joohi.rastogi@stericsson.com>
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();
}
--
1.7.5.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-06-12 7:31 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-29 6:46 [PATCH] Sets the information attribute for the service joohi.rastogi
2012-05-30 7:48 ` Johan Hedberg
2012-06-01 6:01 ` Joohi RASTOGI
2012-06-12 7:31 ` Joohi RASTOGI
-- strict thread matches above, loose matches on Subject: below --
2012-05-21 9:30 joohi.rastogi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.