On Wed, Mar 14, 2018 at 11:06:02PM +0100, Hans de Goede wrote: > This reverts commit 43fff7683468 ("Bluetooth: hci_bcm: Streamline runtime > PM code"). The commit msg for this commit states "No functional change > intended.", but replacing: > > pm_runtime_get(); > pm_runtime_mark_last_busy(); > pm_runtime_put_autosuspend(); > > with: > > pm_request_resume(); > > Does result in a functional change, pm_request_resume() only calls > pm_runtime_mark_last_busy() if the device was suspended before the call. Yes, Robert Howell (cc) reported this a few days ago: https://bugzilla.kernel.org/show_bug.cgi?id=198953 I've worked with him to develop a fix which is better IMHO than a revert, namely he's replacing the pm_request_resume() in bcm_recv() with pm_runtime_mark_last_busy(), and the pm_request_resume() in the interrupt handler can stay. He says that fixes the issue for him. I hope he'll submit the patch shortly. Thanks, Lukas -- 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