Linux bluetooth development
 help / color / mirror / Atom feed
* Add SDP discovery support to gatttool with BR
@ 2014-05-08  8:44 Vikrampal
  2014-05-14  6:13 ` Dmitry Kasatkin
  2014-05-14  7:56 ` Johan Hedberg
  0 siblings, 2 replies; 4+ messages in thread
From: Vikrampal @ 2014-05-08  8:44 UTC (permalink / raw)
  To: linux-bluetooth; +Cc: Dmitry Kasatkin, bharat.panda, p.sinha, cpgs

Hi,

I am new to BlueZ community. I'd like to contribute to BlueZ community. I
went through BlueZ's TODO items. 
To get started, I have selected adding SDP discovery support to gatttool
with BR as my first work.

- Add sdp discovery support to gatttool with BR (--sdp, default is 0x1f)

There are several options available under SDP support. However, this TODO
particular item seeks to support only search option.

Please provide me your valuable suggestion for the same. Thanks!

Regards,
Vikram


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Add SDP discovery support to gatttool with BR
  2014-05-08  8:44 Add SDP discovery support to gatttool with BR Vikrampal
@ 2014-05-14  6:13 ` Dmitry Kasatkin
  2014-05-14  7:56 ` Johan Hedberg
  1 sibling, 0 replies; 4+ messages in thread
From: Dmitry Kasatkin @ 2014-05-14  6:13 UTC (permalink / raw)
  To: Vikrampal, linux-bluetooth, Marcel Holtmann, Johan Hedberg
  Cc: bharat.panda, p.sinha, cpgs

On 08/05/14 11:44, Vikrampal wrote:
> Hi,
>
> I am new to BlueZ community. I'd like to contribute to BlueZ community. I
> went through BlueZ's TODO items. 
> To get started, I have selected adding SDP discovery support to gatttool
> with BR as my first work.
>
> - Add sdp discovery support to gatttool with BR (--sdp, default is 0x1f)
>
> There are several options available under SDP support. However, this TODO
> particular item seeks to support only search option.
>
> Please provide me your valuable suggestion for the same. Thanks!
>
> Regards,
> Vikram
>
>

Hi Marcel and Johan,

Vikrant is keen to contribute to BlueZ.
Does it make sense to start with what Vikrant suggest?
Anybody working on that?

BR,

Dmitry

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Add SDP discovery support to gatttool with BR
  2014-05-08  8:44 Add SDP discovery support to gatttool with BR Vikrampal
  2014-05-14  6:13 ` Dmitry Kasatkin
@ 2014-05-14  7:56 ` Johan Hedberg
  2014-05-15 10:49   ` Vikrampal
  1 sibling, 1 reply; 4+ messages in thread
From: Johan Hedberg @ 2014-05-14  7:56 UTC (permalink / raw)
  To: Vikrampal; +Cc: linux-bluetooth, Dmitry Kasatkin, bharat.panda, p.sinha, cpgs

Hi Vikram,

On Thu, May 08, 2014, Vikrampal wrote:
> I am new to BlueZ community. I'd like to contribute to BlueZ community. I
> went through BlueZ's TODO items. 
> To get started, I have selected adding SDP discovery support to gatttool
> with BR as my first work.
> 
> - Add sdp discovery support to gatttool with BR (--sdp, default is 0x1f)

Not sure what exactly you mean here. ATT over BR/EDR uses a fixed PSM so
separate discovery of this is not necessary. The core spec says:

"The use of a fixed PSM allows rapid reconnection of the L2CAP channel
for Attribute Protocol as a preliminary SDP query is not required."

Based on git blame this item was added 5th Oct 2010, i.e. in the very
early days of the LE specification. I suspect that there might have been
some confusion regarding the ATT PSM back then. While this kind of PSM
discovery it's not completely pointless, we don't do it for any other
fixed PSM profiles either (e.g. A2DP & AVRCP) so I think it's better to
spend time on more valuable tasks instead.

I've now removed the ATT PSM related items from the TODO file (which btw
could use much more cleanups in general).

Johan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: Add SDP discovery support to gatttool with BR
  2014-05-14  7:56 ` Johan Hedberg
@ 2014-05-15 10:49   ` Vikrampal
  0 siblings, 0 replies; 4+ messages in thread
From: Vikrampal @ 2014-05-15 10:49 UTC (permalink / raw)
  To: 'Johan Hedberg'
  Cc: linux-bluetooth, 'Dmitry Kasatkin', bharat.panda, p.sinha,
	cpgs

Hi Johan,

>-----Original Message-----
>From: Johan Hedberg [mailto:johan.hedberg@gmail.com] 
>Sent: Wednesday, May 14, 2014 1:26 PM
>To: Vikrampal
>Cc: linux-bluetooth@vger.kernel.org; Dmitry Kasatkin;
bharat.panda@samsung.com; p.sinha@samsung.com; cpgs@samsung.com
>Subject: Re: Add SDP discovery support to gatttool with BR
>
>Hi Vikram,
>
>On Thu, May 08, 2014, Vikrampal wrote:
>> I am new to BlueZ community. I'd like to contribute to BlueZ 
>> community. I went through BlueZ's TODO items.
>> To get started, I have selected adding SDP discovery support to 
>> gatttool with BR as my first work.
>> 
>> - Add sdp discovery support to gatttool with BR (--sdp, default is 
>> 0x1f)
>
>Not sure what exactly you mean here. ATT over BR/EDR uses a fixed PSM so
separate discovery of this is not necessary. The core spec says:
>
>"The use of a fixed PSM allows rapid reconnection of the L2CAP channel for
Attribute Protocol as a preliminary SDP query is not required."
>
>Based on git blame this item was added 5th Oct 2010, i.e. in the very early
days of the LE specification. I suspect that there might have been some
confusion regarding >the ATT PSM back then. While this kind of PSM discovery
it's not completely pointless, we don't do it for any other fixed PSM
profiles either (e.g. A2DP & AVRCP) so I >think it's better to spend time on
more valuable tasks instead.
>
>I've now removed the ATT PSM related items from the TODO file (which btw
could use much more cleanups in general).
>
>Johan

Thanks for your kind advice. Could you please suggest me as to which all
topics are of more value currently so that
I may pick up one of them as my maiden contribution. 

Regards,
Vikram


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-05-15 10:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-08  8:44 Add SDP discovery support to gatttool with BR Vikrampal
2014-05-14  6:13 ` Dmitry Kasatkin
2014-05-14  7:56 ` Johan Hedberg
2014-05-15 10:49   ` Vikrampal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox