[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200708030957.46320.oliver@neukum.org>
Date: Fri, 3 Aug 2007 09:57:45 +0200
From: Oliver Neukum <oliver@...kum.org>
To: linux-usb-devel@...ts.sourceforge.net
Cc: Matthew Garrett <mjg59@...f.ucam.org>, Greg KH <gregkh@...e.de>,
linux-kernel@...r.kernel.org
Subject: Re: [linux-usb-devel] [PATCH] USB: Only enable autosuspend by default on certain device classes
Am Freitag 03 August 2007 schrieb Matthew Garrett:
> > Also, we have udev rules for SANE that disables their autosuspend
> > settings, which handles the majority of the devices we have seen with
> > problems.
>
> Several printers seem to have the issue as well, and the blacklist seems
> to contain some odd miscellaneous devices like the Blackberry. The main
Then make autosuspend support for the printer driver a config option.
This is not a reason to change the core usb code. The core code needs
to be involved only for device that are driven through usbfs. The major types
are:
- scanners
- PTP devices
- OBEX
Scanners are covered by SANE's latest CVS
PTP are a class and could be covered by a single udev rule
Obex is comm, so the patch wouldn't help.
> concern I have is that kernel developers just don't tend to be the sort
> of people that use webcams, printers or scanners, so we're relying on
> normal users to go to the effort of reporting that their device has
> stopped working.
Kernel developers are a diverser lot than you think ;-)
We don't enable autosuspend in drivers we can't test, except where
the lack of a kernel driver forces us to use a broad swipe. Printers
were tested, too, and most printers seem to work.
Regards
Oliver
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists