Re: [PATCHv2] Bluetooth: btusb: Fix Intel controller hang after shutdown

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

 



Hi Ethan,

I am helping Tedd to test this patch now, but I have a question about this patch.

For your comment It looks like you want to send HCI RESET command to controller first when BT enabled, then I found below patch also do same thing as your patch. May I know any difference for your patch? Thanks.

commit dffd30ee9619ccd7153f1861ba0436bbc4400f36
Author: Tedd Ho-Jeong An <tedd.an@xxxxxxxxx>
Date:   Fri Apr 19 09:57:43 2013 -0700

     Bluetooth: Add support for Intel Bluetooth device [8087:07dc]

setup is only called for the first open/up to initialize the device.
shutdown is called before each regular HCI close/down to perform vendor HCI epilog.

The Intel shutdown procedure consist in sending the 0xfc3f command which seems
to cause an internal reset with old firmwares (LED off with new one).
So, we need to HCI reset the controller before any other command in order to avoid
the "number of completed commands" issue.
To be sure that this HCI reset will be perform, I just put the HCI reset after the 0xfc3f
in the shutdown callback.

Since I only have old version, I can only test that the problem is fixed for me but not that
no regression is caused with the new fw/hw version.
So we need feedback, to know if the expected behavior (LED OFF on shutdown) is preserved.

Regards,
Loic

--
Intel Open Source Technology Center
http://oss.intel.com/

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux