All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Jones <davej@redhat.com>
To: Oliver Neukum <oliver@neukum.org>
Cc: Matthew Garrett <mjg59@srcf.ucam.org>,
	linux-usb-devel@lists.sourceforge.net, Greg KH <gregkh@suse.de>,
	linux-kernel@vger.kernel.org,
	David Brownell <david-b@pacbell.net>,
	Alan Stern <stern@rowland.harvard.edu>,
	Rogan Dawes <lists@dawes.za.net>
Subject: Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
Date: Fri, 3 Aug 2007 16:03:36 -0400	[thread overview]
Message-ID: <20070803200336.GA20376@redhat.com> (raw)
In-Reply-To: <200708031808.11903.oliver@neukum.org>

On Fri, Aug 03, 2007 at 06:08:11PM +0200, Oliver Neukum wrote:
 > Am Freitag 03 August 2007 schrieb Matthew Garrett:
 > > > Which is why I didn't suggest doing that, of course.  The only
 > > > one making that kind of straw man argument seems to be you.
 > > 
 > > But however you phrase it, that's effectively what it is. "Does your 
 > > device work?" just makes users wonder why the damn computer doesn't know 
 > > already. "This option may prevent your device from working. Click here 
 > > to switch it off" results in them wondering why it was switched on in 
 > > the first place. Many of our users aren't technical - they don't care 
 > > about saving 200mW, they just care about their printer working when they 
 > > plug it in.
 > 
 > Devices rarely simply crash. Although Windows doesn't do runtime power
 > management, it certainly will suspend all devices when the system goes
 > into suspension. Buggy devices typically disconnect and reconnect when
 > resumed. This is testable for in software without user intervention.

The printer I mentioned earlier this thread wouldn't work again
until I physically unplugged and replugged the usb cable.
It spewed descriptor read errors every time I tried to talk to it.
Even unloading and reloading the usb modules didn't bring it back
to life.

	Dave

-- 
http://www.codemonkey.org.uk

  parent reply	other threads:[~2007-08-03 20:10 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
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 [this message]
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=20070803200336.GA20376@redhat.com \
    --to=davej@redhat.com \
    --cc=david-b@pacbell.net \
    --cc=gregkh@suse.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.