register HID with SDP error

[Maksim Yevmenkin]

Hello,

[...]

> I think I'm getting close, I finally have a good scan with bthidcontrol
>
> Note that bthidcontrol doesn't seem to connect after sdpd is restarted
> on the server, unless I do sdpcontrol browse first.

hmm... i'm sorry, i do not follow. both bthidcontrol(8) and
sdpcontrol(8) are using exactly the same way to obtain sdp records. it
should not make any difference.


> #sdpcontrol -a 00:1b:dc:06:94:d3 browse

[...]

> # bthidcontrol -a 00:1b:dc:06:94:d3 query

[...]

this looks better to me.

> latest version at https://github.com/waitman/sdpd/blob/master/hid.c

thanks! i will take a look. few minor nits: attribute list must be
sorted. when generating protocol descriptor list and additional
protocol descriptor list, you may re-use one function, i.e. additional
protocol descriptor list is just a protocol descriptor list wrapped
into a sequence. you hardwired lots of hid related things. while its
ok for testing, i feel like it should be moved to parameters so
clients can set those up as needed.

> I didn't see a "bool(8)" defined in sdp.h, I was thinking maybe it was
> the same as UINT8 but then I noticed Iain's post that showed a
> 'bool(8)' as 0x28 instead of 0x08, thank you.

huh? from /usr/include/sdp.h

/* Unsigned integer */
#define SDP_DATA_UINT8                                  0x08

/* Boolean */
#define SDP_DATA_BOOL                                   0x28

thanks,
max