From: Oliver Neukum <oliver@neukum.org>
To: Mario Limonciello <mario_limonciello@dell.com>,
linux-usb@vger.kernel.org
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
Marcel Holtmann <marcel@holtmann.org>,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Explicitly disable BT radio using rfkill interface on suspend
Date: Wed, 20 May 2009 00:16:48 +0200 [thread overview]
Message-ID: <200905200016.49110.oliver@neukum.org> (raw)
In-Reply-To: <4A1328D6.5090404@dell.com>
Am Dienstag, 19. Mai 2009 23:47:02 schrieb Mario Limonciello:
> Hi Oliver:
> Attached are both the lsusb listings. As you can see, the missing
> device post-suspend is that Bluetooth radio device (which is why I
> proposed this patch in the first place).
This explains the error in your log.
> There is a udev removal event for this device, but as I said, there are
> no add/remove/change events for the keyboard or mouse devices. I'm
> attaching udevadm monitor ran from right before the suspend to right after.
That is correct the devices have been successfully resumed and thus are
present. Generally the idea of suspened/resume is to have the same system
after resumption. It is correct to not generate events. Usbcore cannot know
that you use udev to switch the device from pure HID to HID + bluetooth.
> Sending the appropriate hid report to the mouse device will re-attach
> this Bluetooth radio device to the bus.
Yes, the question is whether it is the clean thing to do.
Marcel, are the HID devices usable after hid2hci has run? Should we
choose to not resume from power loss any hid device which hid2hci
operated on?
Regards
Oliver
next prev parent reply other threads:[~2009-05-19 22:16 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-18 20:12 [PATCH] Explicitly disable BT radio using rfkill interface on suspend Mario Limonciello
2009-05-18 21:02 ` Matthew Garrett
2009-05-18 21:36 ` Mario Limonciello
2009-05-19 5:03 ` Matthew Garrett
2009-05-19 5:14 ` Marcel Holtmann
2009-05-19 6:27 ` Oliver Neukum
2009-05-19 6:45 ` Marcel Holtmann
2009-05-19 7:36 ` Oliver Neukum
2009-05-19 9:06 ` Matthew Garrett
2009-05-19 12:47 ` Oliver Neukum
2009-05-19 12:58 ` Matthew Garrett
2009-05-19 16:30 ` Mario Limonciello
2009-05-19 16:33 ` Matthew Garrett
2009-05-19 16:52 ` Mario Limonciello
2009-05-19 19:58 ` Oliver Neukum
2009-05-19 20:36 ` Mario Limonciello
2009-05-19 21:26 ` Oliver Neukum
2009-05-19 21:47 ` Mario Limonciello
2009-05-19 22:16 ` Oliver Neukum [this message]
2009-05-19 22:19 ` Mario Limonciello
2009-05-20 2:17 ` Alan Stern
2009-05-20 12:29 ` Oliver Neukum
2009-05-20 14:17 ` Alan Stern
2009-05-20 17:52 ` Mario Limonciello
2009-05-20 18:31 ` Alan Stern
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200905200016.49110.oliver@neukum.org \
--to=oliver@neukum.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=marcel@holtmann.org \
--cc=mario_limonciello@dell.com \
--cc=mjg59@srcf.ucam.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox