* Fwd: Re: Re: Fwd: Fwd: [PATCH 1/1] shared/gatt-client: Fix the "Find Information req" error
@ 2015-06-11 10:23 Nagaraj D R
2015-06-11 13:40 ` Luiz Augusto von Dentz
0 siblings, 1 reply; 2+ messages in thread
From: Nagaraj D R @ 2015-06-11 10:23 UTC (permalink / raw)
To: Luiz Augusto von Dentz; +Cc: linux-bluetooth@vger.kernel.org
Kytjb3JyZWN0aW9uDQpEZXNjcmlwdG9ycyBhcmUgc2VhcmNoZWQgYmV0d2VlbiBDaGFyYWN0ZXJp
c3RpYyB2YWx1ZSBkZWNsYXJhdGlvbiBhbmQgY2hhcmFjdGVyaXN0aWMgZGVmaW5pdGlvbiBlbmQg
aGFuZGxlDQoNCi0tLS0tLS0gT3JpZ2luYWwgTWVzc2FnZSAtLS0tLS0tDQpTZW5kZXIgOiBOYWdh
cmFqIEQgUjxuYWdhcmFqLmRyQHNhbXN1bmcuY29tPiBMZWFkIEVuZ2luZWVyL1NSSS1CYW5nYWxv
cmUtU3lzdGVtICYgQ29ubmVjdGl2aXR5L1NhbXN1bmcgRWxlY3Ryb25pY3MNCkRhdGUgOiBKdW4g
MTEsIDIwMTUgMTU6NDYgKEdNVCswNTozMCkNClRpdGxlIDogUmU6IFJlOiBGd2Q6IEZ3ZDogW1BB
VENIIDEvMV0gc2hhcmVkL2dhdHQtY2xpZW50OiBGaXggdGhlICJGaW5kIEluZm9ybWF0aW9uIHJl
cSIgZXJyb3INCg0KSGVsbG8gTHVpeiwNClRoZSBwcm9ibGVtIHRoYXQgSSB3YXMgdHJ5aW5nIHRv
IGFkZHJlc3MgaXMgZm9yIHRoZSBjaGFyYWN0ZXJpc3RpYyBhdHRyaWJ1dGUgYXQgMFhmZmZlIGFu
ZCBjaGFyYWN0ZXJpc3RpYyB2YWx1ZSBkZWNsYXJhdGlvbiBhdCBveGZmZmYuDQpUaGlzIGlzIGEg
dmFsaWQgc2NlbmFyaW8uIENoYXJhY3RlcmlzdGljIG5lZWQgdG8gaGF2ZSBhIGNoYXJhY3Rlcmlz
dGljIHZhbHVlIGRlY2xhcmF0aW9uIGJ1dCBuZWVkIG5vdCBoYXZlIGNoYXJhY3RlcmlzdGljIGRl
c2NyaXB0b3JzLg0KDQpTaW5jZSBwcmVzZW50IGRlc2lnbiBzZWFyY2ggdGhlIGRlc2NyaXB0b3Jz
IGIvdyB0aGUgY2hhcmFjdGVyaXN0aWMgdmFsdWUgZGVjbGFyYXRpb24gYW5kIHNlcnZpY2UgZW5k
IGhhbmRsZSwNCmlmIHNlcnZpY2UgZW5kIGhhbmRsZSBhbmQgY2hhcmFjdGVyaXN0aWMgdmFsdWUg
ZGVjbGFyYXRpb24gZW5kIGhhbmRsZSBhcmUgc2FtZSAoaS5lIDB4ZmZmZiBpbiBhIHBhcnRpY3Vs
YXIgY2FzZSkNCnRoZW4gZGVzY3JpcHRvciBzZWFyY2ggaGFuZGxlIHdpbGwgYmUgaW5pdGlhbGl6
ZWQgdG8gMFgwMDAgKGkuZSBieSBpbmNyZW1lbnRpbmcgY2hhcmFjdGVyaXN0aWMgdmFsdWUgaGFu
ZGxlICgweGZmZmYpICkNCjB4MDAwMCBoYW5kbGUgaXMgcmVzZXJ2ZWQgaGFuZGxlICsgaW50ZW50
aW9uIG9mIHNlYXJjaGluZyBkZXNjcmlwdG9yIHdpbGwgbm90IGJlIGFjaGlldmVkLg0KDQpSZWdh
cmRzDQpOYWdhcmFqIEQgUg0KDQotLS0tLS0tIE9yaWdpbmFsIE1lc3NhZ2UgLS0tLS0tLQ0KU2Vu
ZGVyIDogTHVpeiBBdWd1c3RvIHZvbiBEZW50eg0KRGF0ZSA6IEp1biAxMSwgMjAxNSAxNToyNCAo
R01UKzA1OjMwKQ0KVGl0bGUgOiBSZTogRndkOiBGd2Q6IFtQQVRDSCAxLzFdIHNoYXJlZC9nYXR0
LWNsaWVudDogRml4IHRoZSAiRmluZCBJbmZvcm1hdGlvbiByZXEiIGVycm9yDQoNCkhpIE5hZ2Fy
YWosDQoNCk9uIFRodSwgSnVuIDExLCAyMDE1IGF0IDc6MDkgQU0sIE5hZ2FyYWogRCBSIHdyb3Rl
Og0KPiBwaW5nDQo+DQo+IC0tLS0tLS0gT3JpZ2luYWwgTWVzc2FnZSAtLS0tLS0tDQo+IFNlbmRl
ciA6IE5hZ2FyYWogRCBSIExlYWQgRW5naW5lZXIvU1JJLUJhbmdhbG9yZS1TeXN0ZW0gJiBDb25u
ZWN0aXZpdHkvU2Ftc3VuZyBFbGVjdHJvbmljcw0KPiBEYXRlIDogTWF5IDE1LCAyMDE1IDExOjUw
IChHTVQrMDU6MzApDQo+IFRpdGxlIDogRndkOiBbUEFUQ0ggMS8xXSBzaGFyZWQvZ2F0dC1jbGll
bnQ6IEZpeCB0aGUgIkZpbmQgSW5mb3JtYXRpb24gcmVxIiBlcnJvcg0KPg0KPiBwaW5nDQo+DQo+
IC0tLS0tLS0gT3JpZ2luYWwgTWVzc2FnZSAtLS0tLS0tDQo+IFNlbmRlciA6IE5hZ2FyYWogRCBS
IExlYWQgRW5naW5lZXIvU1JJLUJhbmdhbG9yZS1TeXN0ZW0gJiBDb25uZWN0aXZpdHkvU2Ftc3Vu
ZyBFbGVjdHJvbmljcw0KPiBEYXRlIDogTWF5IDEzLCAyMDE1IDE3OjA1IChHTVQrMDU6MzApDQo+
IFRpdGxlIDogW1BBVENIIDEvMV0gc2hhcmVkL2dhdHQtY2xpZW50OiBGaXggdGhlICJGaW5kIElu
Zm9ybWF0aW9uIHJlcSIgZXJyb3INCj4NCj4gRnJvbTogIm5hZ2FyYWouZHIiDQo+DQo+IGNoYXJh
Y3RlcmlzdGljIGRlc2NyaXB0b3IgaXMgc2VhcmNoZWQgYi93IHRoZQ0KPiBjaGFyYWN0ZXJpc3Rp
YyAidmFsdWVfaGFuZGxlICsgMSIgYW5kDQo+IGNoYXJhY3RlcmlzdGljICJlbmRfaGFuZGxlIiB1
c2luZyAiRmluZCBJbmZvcm1hdGlvbiByZXEiDQo+DQo+IGlmIHJlbW90ZSBkZXZpY2UgaGFkIGRl
Y2xhcmVkIHRoZSBjaGFyYWN0ZXJpc3RpYyB2YWx1ZV9oYW5kbGUNCj4gYXQgMFhGRkZGICh3aGlj
aCBhbHNvIG1lYW5zIHRoZXJlIGlzIG5vIGNoYXJhY3RlcmlzdGljIGRlc2NyaXB0b3JzKSx0aGVu
DQo+IHByZXNlbnQgY29kZSB3b3VsZCBtYWtlIHN0YXJ0X2hhbmRsZSBoYXMgMFgwMDAwKGJlY2F1
c2Ugb2YgMTYgYml0DQo+IGludGVnZXIgb3ZlcmZsb3cpIGZvciAiRmluZCBJbmZvcm1hdGlvbiBy
ZXEiDQo+DQo+IGRlc2Nfc3RhcnQgPSBjaHJjX2RhdGEtPnZhbHVlX2hhbmRsZSArIDE7DQo+DQo+
IENvbnNlcXVlbmNlOiAgQmVsb3cgcmVxdWVzdCB3aWxsIGJlIHNlbnQsDQo+ICAgICAgICAgQVRU
OiBGaW5kIEluZm9ybWF0aW9uIHJlcSAoMHgwNCkNCj4gICAgICAgICBzdGFydCAweDAwMDAsIGVu
ZCAweGZmZmYNCj4NCj4gICAgICAgICBhbmQgYmVsb3cgd2lsbCBiZSB0aGUgcHJvcGVyIHJlc3Bv
bnNlIGZyb20gdGhlIHJlbW90ZSBkZXZpY2UNCj4gICAgICAgICBBVFQ6IEVycm9yICgweDAxKQ0K
PiAgICAgICAgIEVycm9yOiBJbnZhbGlkIGhhbmRsZSAoMSkNCj4gICAgICAgICBGaW5kIEluZm9y
bWF0aW9uIHJlcSAoMHgwNCkgb24gaGFuZGxlIDB4MDAwMA0KPiAtLS0NCj4gc3JjL3NoYXJlZC9n
YXR0LWNsaWVudC5jIHwgICAgNiArKystLS0NCj4gMSBmaWxlIGNoYW5nZWQsIDMgaW5zZXJ0aW9u
cygrKSwgMyBkZWxldGlvbnMoLSkNCj4NCj4gZGlmZiAtLWdpdCBhL3NyYy9zaGFyZWQvZ2F0dC1j
bGllbnQuYyBiL3NyYy9zaGFyZWQvZ2F0dC1jbGllbnQuYw0KPiBpbmRleCA3YmMzYjcxLi45Yjk0
NGVlIDEwMDY0NA0KPiAtLS0gYS9zcmMvc2hhcmVkL2dhdHQtY2xpZW50LmMNCj4gKysrIGIvc3Jj
L3NoYXJlZC9nYXR0LWNsaWVudC5jDQo+IEBAIC02MDEsMTMgKzYwMSwxMyBAQCBzdGF0aWMgYm9v
bCBkaXNjb3Zlcl9kZXNjcyhzdHJ1Y3QgZGlzY292ZXJ5X29wICpvcCwgYm9vbCAqZGlzY292ZXJp
bmcpDQo+IGNocmNfZGF0YS0+dmFsdWVfaGFuZGxlKQ0KPiBnb3RvIGZhaWxlZDsNCg0KQWN0dWFs
bHkgdGhpcyBjaGVjayBzaG91bGQgaGFwcGVuIGluDQpnYXR0X2RiX3NlcnZpY2VfaW5zZXJ0X2No
YXJhY3RlcmlzdGljIHdoaWNoIGlzIHByb2JhYmx5IGNyZWF0aW5nDQpzb21ldGhpbmcgd2l0aCBo
YW5kbGUgMCBpbiB0aGUgZGF0YWJhc2Ugd2hpY2ggaXMgd3JvbmcsIGFjdHVhbGx5DQpwZXJoYXBz
IGl0IHNob3VsZCBqdXN0IGZhaWwgdG8gY3JlYXRlIGEgY2hhcmFjdGVyaXN0aWMgaW4gMHhmZmZm
LA0KY2hhcmFjdGVyaXN0aWNzIHNoYWxsIGFsd2F5cyBoYXZlIGEgdmFsdWUgZGVzY3JpcHRvcjoN
Cg0KMy4zLjIgQ2hhcmFjdGVyaXN0aWMgVmFsdWUgRGVjbGFyYXRpb24NClRoZSBDaGFyYWN0ZXJp
c3RpYyBWYWx1ZSBkZWNsYXJhdGlvbiBjb250YWlucyB0aGUgdmFsdWUgb2YgdGhlDQpjaGFyYWN0
ZXJpc3RpYy4gSXQNCmlzIHRoZSBmaXJzdCBBdHRyaWJ1dGUgYWZ0ZXIgdGhlIGNoYXJhY3Rlcmlz
dGljIGRlY2xhcmF0aW9uLiBBbGwgY2hhcmFjdGVyaXN0aWMNCmRlZmluaXRpb25zIHNoYWxsIGhh
dmUgYSBDaGFyYWN0ZXJpc3RpYyBWYWx1ZSBkZWNsYXJhdGlvbi4NCg0KU28gSSB3b3VsZCBzdWdn
ZXN0IHNvbWV0aGluZyBsaWtlIHRoaXM6DQoNCmlmZiAtLWdpdCBhL3NyYy9zaGFyZWQvZ2F0dC1k
Yi5jIGIvc3JjL3NoYXJlZC9nYXR0LWRiLmMNCmluZGV4IDVlMTUzN2UuLjdmODYzZWYgMTAwNjQ0
DQotLS0gYS9zcmMvc2hhcmVkL2dhdHQtZGIuYw0KKysrIGIvc3JjL3NoYXJlZC9nYXR0LWRiLmMN
CkBAIC03MDYsNiArNzA2LDE4IEBAIHNlcnZpY2VfaW5zZXJ0X2NoYXJhY3RlcmlzdGljKHN0cnVj
dA0KZ2F0dF9kYl9zZXJ2aWNlICpzZXJ2aWNlLA0KICAgICAgICBpZiAoaGFuZGxlICYmIGhhbmRs
ZSA8PSBzZXJ2aWNlLT5hdHRyaWJ1dGVzWzBdLT5oYW5kbGUpDQogICAgICAgICAgICAgICAgcmV0
dXJuIE5VTEw7DQoNCisgICAgICAgLyoNCisgICAgICAgICogSXQgaXMgbm90IHBvc3NpYmxlIHRv
IGFsbG9jYXRlIGxhc3QgaGFuZGxlIGZvciBhIENoYXJhY3RlcmlzdGljDQorICAgICAgICAqIHNp
bmNlIGl0IHdvdWxkIG5vdCBoYXZlIHNwYWNlIGZvciBpdHMgdmFsdWU6DQorICAgICAgICAqIDMu
My4yIENoYXJhY3RlcmlzdGljIFZhbHVlIERlY2xhcmF0aW9uDQorICAgICAgICAqIFRoZSBDaGFy
YWN0ZXJpc3RpYyBWYWx1ZSBkZWNsYXJhdGlvbiBjb250YWlucyB0aGUgdmFsdWUgb2YgdGhlDQor
ICAgICAgICAqIGNoYXJhY3RlcmlzdGljLiBJdCBpcyB0aGUgZmlyc3QgQXR0cmlidXRlIGFmdGVy
IHRoZSBjaGFyYWN0ZXJpc3RpYw0KKyAgICAgICAgKiBkZWNsYXJhdGlvbi4gQWxsIGNoYXJhY3Rl
cmlzdGljIGRlZmluaXRpb25zIHNoYWxsIGhhdmUgYQ0KKyAgICAgICAgKiBDaGFyYWN0ZXJpc3Rp
YyBWYWx1ZSBkZWNsYXJhdGlvbi4NCisgICAgICAgICovDQorICAgICAgIGlmIChoYW5kbGUgPT0g
VUlOVDE2X01BWCkNCisgICAgICAgICAgICAgICByZXR1cm4gTlVMTDsNCisNCiAgICAgICAgaSA9
IGdldF9hdHRyaWJ1dGVfaW5kZXgoc2VydmljZSwgMSk7DQogICAgICAgIGlmICghaSkNCiAgICAg
ICAgICAgICAgICByZXR1cm4gTlVMTDsNCg0KLS0gDQpMdWl6IEF1Z3VzdG8gdm9uIERlbnR6
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Re: Re: Fwd: Fwd: [PATCH 1/1] shared/gatt-client: Fix the "Find Information req" error
2015-06-11 10:23 Fwd: Re: Re: Fwd: Fwd: [PATCH 1/1] shared/gatt-client: Fix the "Find Information req" error Nagaraj D R
@ 2015-06-11 13:40 ` Luiz Augusto von Dentz
0 siblings, 0 replies; 2+ messages in thread
From: Luiz Augusto von Dentz @ 2015-06-11 13:40 UTC (permalink / raw)
To: nagaraj.dr; +Cc: linux-bluetooth@vger.kernel.org
Hi Nagaraj,
On Thu, Jun 11, 2015 at 1:23 PM, Nagaraj D R <nagaraj.dr@samsung.com> wrote:
> ++correction
> Descriptors are searched between Characteristic value declaration and characteristic definition end handle
No Top posting on this mailing list please.
> ------- Original Message -------
> Sender : Nagaraj D R<nagaraj.dr@samsung.com> Lead Engineer/SRI-Bangalore-System & Connectivity/Samsung Electronics
> Date : Jun 11, 2015 15:46 (GMT+05:30)
> Title : Re: Re: Fwd: Fwd: [PATCH 1/1] shared/gatt-client: Fix the "Find Information req" error
>
> Hello Luiz,
> The problem that I was trying to address is for the characteristic attribute at 0Xfffe and characteristic value declaration at oxffff.
> This is a valid scenario. Characteristic need to have a characteristic value declaration but need not have characteristic descriptors.
>
> Since present design search the descriptors b/w the characteristic value declaration and service end handle,
> if service end handle and characteristic value declaration end handle are same (i.e 0xffff in a particular case)
> then descriptor search handle will be initialized to 0X000 (i.e by incrementing characteristic value handle (0xffff) )
> 0x0000 handle is reserved handle + intention of searching descriptor will not be achieved.
Sound like a different problem than I thought, anyway it is valid so
could you please add this explanation to the patch description and
perhaps a comment to the code so we have it crystal clear why it is
necessary.
--
Luiz Augusto von Dentz
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-06-11 13:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-11 10:23 Fwd: Re: Re: Fwd: Fwd: [PATCH 1/1] shared/gatt-client: Fix the "Find Information req" error Nagaraj D R
2015-06-11 13:40 ` Luiz Augusto von Dentz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox