Re: [PATCH v1] Bluetooth: Fix packet type for ESCO Link

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Hi Marcel,

On Tue, Apr 3, 2012 at 7:30 PM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote:
> Hi Hemant,
>> >> This patch fixes the packet type for ESCO Link which was incorrectly
>> >> set for EDR ESCO Packet Types. Anding esco_type with ~EDR_ESCO_MASK
>> >> results in loosing information of esco packet type (EDR Packet
>> >> types are inverted as per BT Spec) that resulted in wrong calculation
>> >> of packet type in hci_setup_sync() API causing the HCI Setup Synchronous
>> >> Connection Command to fail.
>> >
>> > any chance you can add a hcidump trace to the commit message to show
>> > this failure?
>> >
>> I am attaching the hcicump Logs, that can be viewed in Capture File
>> Viewer of FTS.
>> In that the problem can be seen in HCI Tab, Frame # 619, where the
>> packet type is
>> incorrect resulting in command failure with error code 0x1a.
> so me installing a Windows to just view the logs is not going to
> happen ;)
> Since you run BlueZ on one side, please just create a hcidump trace. If
> you store them in BTSnoop format, you can open it with FTS as well.
I have uploaded another patch, please have a look. I found some issues in my
fix yesterday, so I have updated the patch with proper commit message.

> Regards
> Marcel

Best Regards
Hemant Gupta
ST-Ericsson India
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at

[Bluez Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Free Online Dating]     [Bluez Devel]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Devices]     [Big List of Linux Books]

Add to Google Powered by Linux