From: "Adam Kropelin" <akropel1@rochester.rr.com>
To: "Dave Jones" <davej@redhat.com>, "Jiri Kosina" <jikos@jikos.cz>
Cc: "Matthew Garrett" <mjg59@srcf.ucam.org>,
"Oliver Neukum" <oliver@neukum.org>,
"Rogan Dawes" <lists@dawes.za.net>, "Greg KH" <gregkh@suse.de>,
<linux-kernel@vger.kernel.org>,
"David Brownell" <david-b@pacbell.net>,
"Alan Stern" <stern@rowland.harvard.edu>,
<linux-usb-devel@lists.sourceforge.net>
Subject: Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
Date: Fri, 3 Aug 2007 11:25:19 -0400 [thread overview]
Message-ID: <036a01c7d5e2$80add6b0$03c8a8c0@kroptech.com> (raw)
In-Reply-To: 20070803145321.GB17138@redhat.com
Dave Jones wrote:
> On Fri, Aug 03, 2007 at 04:43:16PM +0200, Jiri Kosina wrote:
>> On Fri, 3 Aug 2007, Matthew Garrett wrote:
>>
>>> Windows will autosuspend hubs, bluetooth devices, HID devices
>>
>> Hi Matthew,
>>
>> are you sure about windows suspending the HID devices in runtime? I
>> have never seen LEDs of USB keyboard connected to windows host to go
>> off after some time of not using it.
>
> Not so sure about keyboards, but I've seen the LEDs on USB mice dim
> or go off after a few seconds of inactivity under Windows, but under
> Linux they stay on.
>
>> We have been playing with runtime autosuspend of HID devices, are
>> currently postponed the full support, as it turns out that many
>> devices don't support this feature properly (probably due to not
>> being tested in Windows).
>
> Interesting. Which devices did you notice failing?
> Was it a case that they would sleep and not come back out of that
> state?
Although I have no proof immediately at hand, I suspect at a minimum HID
power class devices should be blacklisted from autosuspend. Given the
spotty USB implementations on many such devices I'd be surprised if
suspend worked reliably. Plus if you're connected to such a device for
monitoring purposes you're probably powered by it as well, so you have
little to gain from suspend even if it works.
--Adam
next prev parent reply other threads:[~2007-08-03 15:26 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-02 23:56 [PATCH] USB: Only enable autosuspend by default on certain device classes Matthew Garrett
2007-08-03 1:15 ` Greg KH
2007-08-03 1:47 ` Matthew Garrett
2007-08-03 2:46 ` [linux-usb-devel] " Alan Stern
2007-08-03 6:01 ` David Brownell
2007-08-03 11:28 ` Matthew Garrett
2007-08-03 11:44 ` Oliver Neukum
2007-08-03 12:04 ` Matthew Garrett
2007-08-03 12:26 ` Rogan Dawes
2007-08-03 12:32 ` Matthew Garrett
2007-08-03 14:01 ` David Brownell
2007-08-03 14:09 ` Matthew Garrett
2007-08-03 14:28 ` Alan Stern
2007-08-03 14:33 ` Matthew Garrett
2007-08-03 14:41 ` Alan Stern
2007-08-03 15:10 ` Matthew Garrett
2007-08-03 14:43 ` Jiri Kosina
2007-08-03 14:53 ` [linux-usb-devel] " Dave Jones
2007-08-03 14:58 ` Jiri Kosina
2007-08-03 15:24 ` David Brownell
2007-08-03 15:29 ` Jiri Kosina
2007-08-03 17:47 ` Alan Stern
2007-08-03 15:25 ` Adam Kropelin [this message]
2007-08-03 15:31 ` Jiri Kosina
2007-08-03 15:48 ` Dave Jones
2007-08-03 16:49 ` David Brownell
2007-08-03 16:55 ` Adam Kropelin
2007-08-03 15:06 ` Matthew Garrett
2007-08-03 14:37 ` [linux-usb-devel] " David Brownell
2007-08-03 15:03 ` Matthew Garrett
2007-08-03 16:08 ` Oliver Neukum
2007-08-03 17:49 ` Alan Stern
2007-08-03 20:03 ` Dave Jones
2007-08-03 16:29 ` David Brownell
2007-08-03 16:50 ` Matthew Garrett
2007-08-03 17:49 ` Greg KH
2007-08-03 17:44 ` Greg KH
2007-08-03 17:48 ` Matthew Garrett
2007-08-03 19:29 ` Chuck Ebbert
2007-08-03 19:56 ` Pete Zaitcev
2007-08-07 9:14 ` Amit Kucheria
2007-08-03 20:12 ` [linux-usb-devel] " Dave Jones
2007-08-03 21:17 ` Alan Stern
2007-08-03 21:31 ` Dave Jones
2007-08-03 21:33 ` Matthew Garrett
2007-08-03 12:34 ` Felipe Balbi
2007-08-03 6:06 ` David Brownell
2007-08-03 14:22 ` Dave Jones
2007-08-03 14:52 ` David Brownell
2007-08-03 7:57 ` Oliver Neukum
2007-08-03 14:24 ` Dave Jones
2007-08-03 14:32 ` Oliver Neukum
2007-08-03 14:36 ` [linux-usb-devel] [PATCH] USB: Only enable autosuspend by?default " Dave Jones
2007-08-03 19:34 ` [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default " Pete Zaitcev
2007-08-03 19:45 ` Dave Jones
2007-08-03 20:04 ` Pete Zaitcev
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='036a01c7d5e2$80add6b0$03c8a8c0@kroptech.com' \
--to=akropel1@rochester.rr.com \
--cc=davej@redhat.com \
--cc=david-b@pacbell.net \
--cc=gregkh@suse.de \
--cc=jikos@jikos.cz \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb-devel@lists.sourceforge.net \
--cc=lists@dawes.za.net \
--cc=mjg59@srcf.ucam.org \
--cc=oliver@neukum.org \
--cc=stern@rowland.harvard.edu \
/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