Re: [Bluez-devel] hcidump EDR packet type

[Marcel Holtmann <marcel@holtmann.org>]

Hi Mayank,

> > actually it is totally wrong. Sending the 3-DH5 bit only means that you
> > wanna enable DM1 and all EDR packets except 3-DH5, because setting that
> > bit means you disable that packet type. The corresponding response
> > should be DM1 + 3-DH5 bit set.
> > 
> > Since this is custom firmware, it is interpreting the bitmask of the
> > change connection packet type in the wrong way.
> > 
> > And to be clear on this, I don't care about the packet mask at all and
> > nobody should mess with it (except for testing). Let the link manager
> > decide which packet type to use. BlueZ can't make any intelligent
> > decision on which is the best packet type. It doesn't know anything
> > about the piconet it is using besides its handle.
> 
> OK. But my question is why display different packet lists in the command
> and event?
> < HCI Command: Change Connection Packet Type (0x01|0x000f) plen 4
>     handle 1 ptype 0x2000
>     Packet type: 3-DH5
> 
> ...
> 
> > HCI Event: Connection Packet Type Changed (0x1d) plen 5
>     status 0x00 handle 1 ptype 0x1306
>     Packet type: 2-DH1 2-DH3 2-DH5 3-DH1 3-DH3
> 
> Both of them should show the same packet list. Isn't it?

that must be a bug in their firmware. They should be the same with the
only exception of DM1.

My assumption is that change connection packet type is treating this
command as enable 3-DH5 only. However that is not what that command
means. It means enable DM1 and all 2-* and 3-* packets except 3-DH5.

Regards

Marcel



-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel